<div dir="ltr">On Tue, Nov 20, 2018 at 9:55 AM Huw Davies <<a href="mailto:huw@codeweavers.com" target="_blank">huw@codeweavers.com</a>> wrote:<br><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sat, Nov 17, 2018 at 10:03:09PM +0100, Sven Baars wrote:<br>
> Signed-off-by: Sven Baars <<a href="mailto:sven.wine@gmail.com" target="_blank">sven.wine@gmail.com</a>><br>
> ---<br>
> <br>
> v2: Added NULL checks to PropVariantCopy instead.<br>
> <br>
>  dlls/ole32/ole2.c              |  3 +++<br>
>  dlls/ole32/tests/propvariant.c | 14 ++++++++++++++<br>
>  2 files changed, 17 insertions(+)<br>
<br>
Do you have an app that depends on this behaviour?<br>
<br>
Huw.<br></blockquote><div><br></div><div>Hi Huw,</div><div><br></div><div>No, but Nikolay asked me to implement this here instead of in mfplat. For that I had an app that depended on it for a few days or so, which was obviously a bug since they tried to put something into a NULL pointer, so they fixed that already. I thought I should submit the patch anyway since it matches Windows behaviour.<br></div><div><br></div><div>Sven<br></div></div></div></div>