[PATCH 5/6] kernel32: Fix an off-by-one error.

Alexandre Julliard julliard at winehq.org
Tue Mar 24 05:42:36 CDT 2009


James Hawkins <truiken at gmail.com> writes:

> ---
>  dlls/kernel32/profile.c       |    2 +-
>  dlls/kernel32/tests/profile.c |   30 ++++++------------------------
>  2 files changed, 7 insertions(+), 25 deletions(-)
>
> diff --git a/dlls/kernel32/profile.c b/dlls/kernel32/profile.c
> index 5d563a5..b6ad6d0 100644
> --- a/dlls/kernel32/profile.c
> +++ b/dlls/kernel32/profile.c
> @@ -396,7 +396,7 @@ static PROFILESECTION *PROFILE_Load(HANDLE hFile, ENCODING * pEncoding)
>      first_section->name[0] = 0;
>      first_section->key  = NULL;
>      first_section->next = NULL;
> -    next_section = &first_section->next;
> +    next_section = &first_section;

This cannot be right. What happens to the original first_section then?

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list