kernel32: GetPrivateProfileString should set buffer to "" if default value is not set (resend)

Alexandre Julliard julliard at winehq.org
Tue Jul 8 02:45:10 CDT 2008


"Dan Kegel" <dank at kegel.com> writes:

> diff --git a/dlls/kernel32/profile.c b/dlls/kernel32/profile.c
> index 9b6b1a1..e52cd66 100644
> --- a/dlls/kernel32/profile.c
> +++ b/dlls/kernel32/profile.c
> @@ -1114,9 +1114,10 @@ static int PROFILE_GetPrivateProfileString( LPCWSTR section, LPCWSTR entry,
>      } else if (buffer && def_val) {
>         lstrcpynW( buffer, def_val, len );
>         ret = strlenW( buffer );
> -    }
> -    else
> +    } else {
> +       buffer[0] = 0; /* buffer must be a null terminated empty string */
>         ret = 0;

What if len is 0?  What if buffer is NULL?

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list