On Tue, Nov 20, 2018 at 9:55 AM Huw Davies <huw@codeweavers.com> wrote:
On Sat, Nov 17, 2018 at 10:03:09PM +0100, Sven Baars wrote:
> Signed-off-by: Sven Baars <sven.wine@gmail.com>
> ---
>
> v2: Added NULL checks to PropVariantCopy instead.
>
>  dlls/ole32/ole2.c              |  3 +++
>  dlls/ole32/tests/propvariant.c | 14 ++++++++++++++
>  2 files changed, 17 insertions(+)

Do you have an app that depends on this behaviour?

Huw.

Hi Huw,

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.

Sven