mpegts: No need to check for NULL before calling g_free()
authorSebastian Rasmussen <sebras@hotmail.com>
Sat, 5 Jul 2014 23:55:50 +0000 (01:55 +0200)
committerEdward Hervey <bilboed@bilboed.com>
Wed, 9 Jul 2014 10:04:17 +0000 (12:04 +0200)
Fixes https://bugzilla.gnome.org/show_bug.cgi?id=732789

gst-libs/gst/mpegts/gst-atsc-section.c
gst-libs/gst/mpegts/gst-dvb-descriptor.c
gst-libs/gst/mpegts/gstmpegtsdescriptor.c
gst-libs/gst/mpegts/gstmpegtssection.c

index 021c8e607a252d23cc3bbcd788dbf2f2cf0d0005..459a323e0f4523194eff6d82036fe15209e6272a 100644 (file)
@@ -411,8 +411,7 @@ _gst_mpegts_atsc_string_segment_copy (GstMpegtsAtscStringSegment * seg)
 static void
 _gst_mpegts_atsc_string_segment_free (GstMpegtsAtscStringSegment * seg)
 {
-  if (seg->cached_string)
-    g_free (seg->cached_string);
+  g_free (seg->cached_string);
   g_slice_free (GstMpegtsAtscStringSegment, seg);
 }
 
index c1f2c30962ee52996466ecbf3c06005dbb18c1d2..29b24424d37c251b00143f0b84c3b349f3357b02 100644 (file)
@@ -522,10 +522,8 @@ gst_mpegts_descriptor_from_dvb_service (GstMpegtsDVBServiceType service_type,
     memcpy (data, conv_service_name, service_size);
 
 beach:
-  if (conv_service_name)
-    g_free (conv_service_name);
-  if (conv_provider_name)
-    g_free (conv_provider_name);
+  g_free (conv_service_name);
+  g_free (conv_provider_name);
 
   return descriptor;
 }
@@ -1138,12 +1136,8 @@ _gst_mpegts_dvb_component_descriptor_copy (GstMpegtsComponentDescriptor *
 void
 gst_mpegts_dvb_component_descriptor_free (GstMpegtsComponentDescriptor * source)
 {
-  if (source->language_code)
-    g_free (source->language_code);
-
-  if (source->text)
-    g_free (source->text);
-
+  g_free (source->language_code);
+  g_free (source->text);
   g_slice_free (GstMpegtsComponentDescriptor, source);
 }
 
index 44a4ef968cf47f1f6119b68824ee6ef363cd757b..246aee45f7b509dc8cbea8d8ba40ffa94d8fab19 100644 (file)
@@ -338,8 +338,7 @@ out:
   if (error) {
     GST_WARNING ("Could not convert from utf-8: %s", error->message);
     g_error_free (error);
-    if (out_text)
-      g_free (out_text);
+    g_free (out_text);
     return NULL;
   }
 
@@ -571,8 +570,7 @@ get_encoding_and_convert (const gchar * text, guint length)
       giconv, is_multibyte, &error);
   if (error != NULL) {
     GST_WARNING ("Could not convert string: %s", error->message);
-    if (converted_str)
-      g_free (converted_str);
+    g_free (converted_str);
     g_error_free (error);
     error = NULL;
 
index 1006576dd2b01b92483a51a99e8f39c59f3556b2..527ff38af40cef029728411d886a2bd8a11a5fed 100644 (file)
@@ -184,8 +184,7 @@ _gst_mpegts_section_free (GstMpegtsSection * section)
   if (section->cached_parsed && section->destroy_parsed)
     section->destroy_parsed (section->cached_parsed);
 
-  if (section->data)
-    g_free (section->data);
+  g_free (section->data);
 
   g_slice_free (GstMpegtsSection, section);
 }