oleaut32: do not crash in logging if string is NULL

Alexandre Julliard julliard at winehq.org
Wed Jan 20 09:51:59 CST 2010

Aric Stewart <aric at 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 at winehq.org

More information about the wine-devel mailing list