Re: [PATCH v7 0/2] MR6737: comctl32/toolbar: forward unhandled WM_NOTIFY

[Zhiyi Zhang (＠zhiyi)]

On Thu Nov  7 19:56:04 2024 +0000, Alanas wrote:
> <div><a target="_blank"
> href="https://gitlab.winehq.org/wine/wine/-/blob/6db2812411d45ca4b4d15dea9a00577ea17b508e/dlls/comctl32/pager.c#L1099-1110"><code>pager.c</code>:<pre><code
> lang="c">static BOOL PAGER_AdjustBuffer(PAGER_INFO *infoPtr, INT
> size)&#10;{&#10;    if (!infoPtr->pwszBuffer)&#10;
>        infoPtr->pwszBuffer = Alloc(size);&#10;    else if
> (infoPtr->nBufferSize &lt; size)&#10;        infoPtr->pwszBuffer =
> ReAlloc(infoPtr->pwszBuffer, size);&#10;&#10;    if
> (!infoPtr->pwszBuffer) return FALSE;&#10;    if (infoPtr->nBufferSize
> &lt; size) infoPtr->nBufferSize = size;&#10;&#10;    return
> TRUE;&#10;}</code></pre></a>Why is this function designed to leak old
> <code>infoPtr->pwszBuffer</code> when <code>ReAlloc(infoPtr->pwszBuffer,
> size)</code> returns <code>NULL</code>?</div>

No. It's a bug.