Re: oleaut32: do not crash in logging if string is NULL

[Alexandre Julliard]

Aric Stewart <aric(a)codeweavers.com> writes:
> Meaning the place where I am finding this is different bug?
>
> In deserialize_param in the VT_BSTR case if the byte_length of the
> BSTR that we are deserializing is -1  then BSTR_UserUnmarshal places
> NULL into *arg.
>
> I will admit I know very little about this code but having it crash
> when trying to log seemed very wrong.
>
> Should I change the
> if (debugout) TRACE_(olerelay)("%s",relaystr(*(BSTR *)arg));
> to a simple debugstr_w or is this revealing a deeper problem somewhere?

It's an arbitrary string so yes, it should be using debugstr_w.

-- 
Alexandre Julliard
julliard(a)winehq.org