tests:va: Fix null ptr dereference in multi-vpp

Dereference the pointer err before null check, which raised a null
pointer dereference issue by Coverity. Modify it to do the null check
of err first, then dereference it.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1143>

diff --git a/subprojects/gst-plugins-bad/tests/examples/va/multiple-vpp.c b/subprojects/gst-plugins-bad/tests/examples/va/multiple-vpp.c
index 05d680e..175327e 100644 (file)
--- a/subprojects/gst-plugins-bad/tests/examples/va/multiple-vpp.c
+++ b/subprojects/gst-plugins-bad/tests/examples/va/multiple-vpp.c
@@ -140,12 +140,15 @@ message_handler (GstBus * bus, GstMessage * msg, gpointer data)
       GError *err = NULL;
 
       gst_message_parse_error (msg, &err, &debug);
-      gst_printerrln ("GStreamer error: %s\n%s", err->message,
-          debug ? debug : "");
+
+      if (err) {
+        gst_printerrln ("GStreamer error: %s\n%s", err->message,
+            debug ? debug : "");
+        g_error_free (err);
+      }
+
       if (debug)
         g_free (debug);
-      if (err)
-        g_error_free (err);
 
       g_main_loop_quit (app->loop);
       break;