[PATCH] devenum: Avoid an invalid free (Valgrind).
Alexandre Julliard
julliard at winehq.org
Thu Dec 13 03:15:19 CST 2018
Zebediah Figura <z.figura12 at gmail.com> writes:
> Signed-off-by: Zebediah Figura <z.figura12 at gmail.com>
> ---
> dlls/devenum/mediacatenum.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/dlls/devenum/mediacatenum.c b/dlls/devenum/mediacatenum.c
> index 5fa2f008dc..efa69195eb 100644
> --- a/dlls/devenum/mediacatenum.c
> +++ b/dlls/devenum/mediacatenum.c
> @@ -419,7 +419,8 @@ static ULONG WINAPI DEVENUM_IMediaCatMoniker_Release(IMoniker *iface)
> TRACE("(%p) ref=%d\n", This, ref);
>
> if (ref == 0) {
> - CoTaskMemFree(This->name);
> + if (This->type != DEVICE_DMO)
> + CoTaskMemFree(This->name);
I'd suggest getting rid of the union instead, and simply keep name as
NULL when not needed.
--
Alexandre Julliard
julliard at winehq.org
More information about the wine-devel
mailing list