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?