Re: gdi32 (1/2): fix unchecked NULL ptr in BIDI_Reorder

[Alexandre Julliard]

Alexander Dorofeyev <alexd14(a)hushmail.com> writes:
> > > One problem is that it wasn't assured lpOutString isn't NULL, and NULL
> > > was getting passed to this routine by GetCharacterPlacementW. so it was
> > > crashing in memcpy. This patch creates a temporary buffer when
> > > lpOutString is NULL and it is used for in-place modifications instead in
> > > such case.

You shouldn't need to allocate a new buffer, the output modifications
should be skipped when no output string is requested.

-- 
Alexandre Julliard
julliard(a)winehq.org