Re: [PATCH 0/1] MR6169: comctl32: fixed first we check index then array access by index is safe

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/comctl32/toolbar.c:
> {
>      TBUTTON_INFO *btnPtr;
>
> -    btnPtr = &infoPtr->buttons[nIndex];
>      if ((nIndex < 0) || (nIndex >= infoPtr->nNumButtons))
> -	return FALSE;
> +        return FALSE;
> +
> +    btnPtr = &infoPtr->buttons[nIndex];
>
> if (lpRect == NULL)
> -	return FALSE;
> +	    return FALSE;
> Please avoid unnecessary whitespace changes.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/6169#note_77257