From 5c54963482bcd701c825b7936056a49b113410e5 Mon Sep 17 00:00:00 2001 From: Alex Henrie Date: Mon, 28 Nov 2016 22:35:53 -0700 Subject: [PATCH] Revert "winegstreamer: Remove redundant null check before g_error_free.". This reverts commit 976ccb92ea745d8a79a2d0a25d37268bb4dc62a6. This is what I get for not reading the documentation...if you call g_error_free(NULL), it prints a nasty error message, which I am now getting frequently. Signed-off-by: Alex Henrie Signed-off-by: Andrew Eikum Signed-off-by: Alexandre Julliard --- dlls/winegstreamer/gstdemux.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/dlls/winegstreamer/gstdemux.c b/dlls/winegstreamer/gstdemux.c index 20acd0e4599..03d8d5b2f92 100644 --- a/dlls/winegstreamer/gstdemux.c +++ b/dlls/winegstreamer/gstdemux.c @@ -1083,7 +1083,8 @@ static GstBusSyncReply watch_bus(GstBus *bus, GstMessage *msg, gpointer data) WARN("%s: %s\n", GST_OBJECT_NAME(msg->src), err->message); WARN("%s\n", dbg_info); } - g_error_free(err); + if (err) + g_error_free(err); g_free(dbg_info); return GST_BUS_DROP; }