[PATCH v2] ole32: Fix memory leak in CompositeMonikerImpl_Construct.
Alexandre Julliard
julliard at winehq.org
Wed Apr 13 00:52:50 CDT 2016
Alex Henrie <alexhenrie24 at gmail.com> writes:
F> Coverity #713613, "Variable This going out of scope leaks the storage it
> points to."
>
> If we need to loop through This->tabMoniker and call IMoniker_Release on
> each of its items, we should do the same in the other three similar
> error paths in this function. Please let me know if this is what you
> want.
Yes, even if Coverity doesn't warn about the reference leaks, it should
be fixed completely, otherwise once the warning is gone we'll forget
about it.
--
Alexandre Julliard
julliard at winehq.org
More information about the wine-devel
mailing list