asfdemux: Don't call strlen() on NULL pointers
authorBenjamin Gaignard <benjamin@gaignard.net>
Wed, 18 Nov 2009 08:58:39 +0000 (09:58 +0100)
committerSebastian Dröge <sebastian.droege@collabora.co.uk>
Wed, 18 Nov 2009 08:58:39 +0000 (09:58 +0100)
Fixes bug #602280.

gst/asfdemux/gstasfdemux.c

index f2d0d99529f606b7d1026b63803540ae8226455c..9757e7d384b94dff0dcfc6975925d41356e3253a 100644 (file)
@@ -2396,7 +2396,7 @@ gst_asf_demux_get_gst_tag_from_tag_name (const gchar * name_utf8)
     "WM/Year", GST_TAG_DATE}
     /* { "WM/Composer", GST_TAG_COMPOSER } */
   };
-  gsize out = strlen (name_utf8);
+  gsize out;
   guint i;
 
   if (name_utf8 == NULL) {
@@ -2404,6 +2404,8 @@ gst_asf_demux_get_gst_tag_from_tag_name (const gchar * name_utf8)
     return NULL;
   }
 
+  out = strlen (name_utf8);
+
   for (i = 0; i < G_N_ELEMENTS (tags); ++i) {
     if (strncmp (tags[i].asf_name, name_utf8, out) == 0) {
       GST_LOG ("map tagname '%s' -> '%s'", name_utf8, tags[i].gst_name);