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

[Alexandre Julliard]

Alexander Dorofeyev <alexd14 at 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 at winehq.org