Revert "winegstreamer: Remove redundant null check before g_error_free.".

This reverts commit 976ccb92. 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 <alexhenrie24@gmail.com> Signed-off-by: Andrew Eikum <aeikum@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

Revert "winegstreamer: Remove redundant null check before g_error_free.".
5c549634 · Alex Henrie · Alexandre Julliard · 424c86ad · 5c549634
Commit 5c549634 authored Nov 28, 2016 by Alex Henrie Committed by Alexandre Julliard Nov 30, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

gstdemux.c dlls/winegstreamer/gstdemux.c +2 -1

No files found.
--- 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;
 }