On May 15, 2016 2:30 AM, "Nikolay Sivov" <nsivov@codeweavers.com> wrote:
>
> Signed-off-by: Nikolay Sivov <nsivov@codeweavers.com>
> ---
>  dlls/kernel32/locale.c       | 3 +++
>  dlls/kernel32/tests/locale.c | 2 ++
>  2 files changed, 5 insertions(+)
>
> diff --git a/dlls/kernel32/locale.c b/dlls/kernel32/locale.c
> index a69e76f..636187f 100644
> --- a/dlls/kernel32/locale.c
> +++ b/dlls/kernel32/locale.c
> @@ -2656,6 +2656,9 @@ BOOL WINAPI IsValidLocaleName( LPCWSTR locale )
>  {
>      struct locale_name locale_name;
>
> +    if (!locale)
> +        return FALSE;
> +
>      /* string parsing */
>      parse_locale_name( locale, &locale_name );
>
> diff --git a/dlls/kernel32/tests/locale.c b/dlls/kernel32/tests/locale.c
> index 20f52d2..70944a2 100644
> --- a/dlls/kernel32/tests/locale.c
> +++ b/dlls/kernel32/tests/locale.c
> @@ -4179,6 +4179,8 @@ static void test_IsValidLocaleName(void)
>      ok(!ret, "IsValidLocaleName should have failed\n");
>      ret = pIsValidLocaleName(LOCALE_NAME_INVARIANT);
>      ok(ret, "IsValidLocaleName failed\n");
> +    ret = pIsValidLocaleName(NULL);
> +    ok(!ret, "IsValidLocaleName should have failed\n");
>  }
>
>  static void test_CompareStringOrdinal(void)
> --
> 2.8.1

Hi Nikolay,

I'm just curious what application this fixes? Or was this something you found while writing tests for something else?