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

[Alexandre Julliard]

James Hawkins <truiken(a)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(a)winehq.org