validate: do not check for NULL before g_free()
authorReynaldo H. Verdejo Pinochet <reynaldo@osg.samsung.com>
Tue, 21 Mar 2017 23:22:50 +0000 (16:22 -0700)
committerReynaldo H. Verdejo Pinochet <reynaldo@osg.samsung.com>
Thu, 23 Mar 2017 22:21:58 +0000 (15:21 -0700)
g_free() is NULL-safe.

validate/gst-libs/gst/video/gssim.c
validate/gst/validate/gst-validate-reporter.c
validate/gst/validate/gst-validate-scenario.c
validate/gst/validate/media-descriptor.c

index 434c14a..52d531b 100644 (file)
@@ -376,9 +376,8 @@ gssim_configure (Gssim * self, gint width, gint height)
 
   g_free (self->priv->windows);
   self->priv->windows = NULL;
-  if (self->priv->orgmu)
-    g_free (self->priv->orgmu);
 
+  g_free (self->priv->orgmu);
   self->priv->orgmu = g_new (gfloat, width * height);
 
   return TRUE;
index 0cb3431..ea6e284 100644 (file)
@@ -320,8 +320,7 @@ gst_validate_reporter_set_name (GstValidateReporter * reporter, gchar * name)
 {
   GstValidateReporterPrivate *priv = gst_validate_reporter_get_priv (reporter);
 
-  if (priv->name)
-    g_free (priv->name);
+  g_free (priv->name);
 
   priv->name = name;
 }
index 75ce586..71c70df 100644 (file)
@@ -2257,8 +2257,7 @@ _execute_set_debug_threshold (GstValidateScenario * scenario,
 
   gst_debug_set_threshold_from_string (threshold_str, reset);
 
-  if (str)
-    g_free (str);
+  g_free (str);
 
   return TRUE;
 }
@@ -2791,10 +2790,8 @@ gst_validate_scenario_load (GstValidateScenario * scenario,
     }
     /* else check scenario */
   check_scenario:
-    if (tldir)
-      g_free (tldir);
-    if (lfilename)
-      g_free (lfilename);
+    g_free (tldir);
+    g_free (lfilename);
 
     if (!is_config) {
       if (found_actions == TRUE)
index c5e891e..a48f9f0 100644 (file)
@@ -76,12 +76,8 @@ free_streamnode (GstValidateMediaStreamNode * streamnode)
   if (streamnode->tags)
     free_tagsnode (streamnode->tags);
 
-  if (streamnode->padname)
-    g_free (streamnode->padname);
-
-  if (streamnode->id)
-    g_free (streamnode->id);
-
+  g_free (streamnode->padname);
+  g_free (streamnode->id);
   g_free (streamnode->str_open);
   g_free (streamnode->str_close);
   g_slice_free (GstValidateMediaStreamNode, streamnode);
@@ -94,8 +90,7 @@ gst_validate_filenode_free (GstValidateMediaFileNode * filenode)
   if (filenode->tags)
     free_tagsnode (filenode->tags);
 
-  if (filenode->uri)
-    g_free (filenode->uri);
+  g_free (filenode->uri);
 
   if (filenode->caps)
     gst_caps_unref (filenode->caps);