<p dir="ltr">On May 15, 2016 2:30 AM, "Nikolay Sivov" <<a href="mailto:nsivov@codeweavers.com">nsivov@codeweavers.com</a>> wrote:<br>
><br>
> Signed-off-by: Nikolay Sivov <<a href="mailto:nsivov@codeweavers.com">nsivov@codeweavers.com</a>><br>
> ---<br>
>  dlls/kernel32/locale.c       | 3 +++<br>
>  dlls/kernel32/tests/locale.c | 2 ++<br>
>  2 files changed, 5 insertions(+)<br>
><br>
> diff --git a/dlls/kernel32/locale.c b/dlls/kernel32/locale.c<br>
> index a69e76f..636187f 100644<br>
> --- a/dlls/kernel32/locale.c<br>
> +++ b/dlls/kernel32/locale.c<br>
> @@ -2656,6 +2656,9 @@ BOOL WINAPI IsValidLocaleName( LPCWSTR locale )<br>
>  {<br>
>      struct locale_name locale_name;<br>
><br>
> +    if (!locale)<br>
> +        return FALSE;<br>
> +<br>
>      /* string parsing */<br>
>      parse_locale_name( locale, &locale_name );<br>
><br>
> diff --git a/dlls/kernel32/tests/locale.c b/dlls/kernel32/tests/locale.c<br>
> index 20f52d2..70944a2 100644<br>
> --- a/dlls/kernel32/tests/locale.c<br>
> +++ b/dlls/kernel32/tests/locale.c<br>
> @@ -4179,6 +4179,8 @@ static void test_IsValidLocaleName(void)<br>
>      ok(!ret, "IsValidLocaleName should have failed\n");<br>
>      ret = pIsValidLocaleName(LOCALE_NAME_INVARIANT);<br>
>      ok(ret, "IsValidLocaleName failed\n");<br>
> +    ret = pIsValidLocaleName(NULL);<br>
> +    ok(!ret, "IsValidLocaleName should have failed\n");<br>
>  }<br>
><br>
>  static void test_CompareStringOrdinal(void)<br>
> --<br>
> 2.8.1</p>
<p dir="ltr">Hi Nikolay,</p>
<p dir="ltr">I'm just curious what application this fixes? Or was this something you found while writing tests for something else?</p>