isomp4: Fix mfro and tfra atom dumping
authorEdward Hervey <bilboed@bilboed.com>
Wed, 26 Nov 2014 14:59:36 +0000 (15:59 +0100)
committerEdward Hervey <bilboed@bilboed.com>
Wed, 26 Nov 2014 15:36:39 +0000 (16:36 +0100)
mfro was skipping the version/flags
tfra had wrong byte_reader return value checks

gst/isomp4/qtdemux_dump.c

index 08425fd..749dbcc 100644 (file)
@@ -568,6 +568,7 @@ qtdemux_dump_mfro (GstQTDemux * qtdemux, GstByteReader * data, int depth)
   if (!qt_atom_parser_has_remaining (data, 4))
     return FALSE;
 
+  GST_LOG ("%*s  version/flags: %08x", depth, "", GET_UINT32 (data));
   GST_LOG ("%*s  size: %d", depth, "", GET_UINT32 (data));
   return TRUE;
 }
@@ -596,8 +597,8 @@ qtdemux_dump_tfra (GstQTDemux * qtdemux, GstByteReader * data, int depth)
   GST_LOG ("%*s  version/flags: %08x", depth, "", ver_flags);
 
   if (!gst_byte_reader_get_uint32_be (data, &track_id) ||
-      gst_byte_reader_get_uint32_be (data, &len) ||
-      gst_byte_reader_get_uint32_be (data, &num_entries))
+      !gst_byte_reader_get_uint32_be (data, &len) ||
+      !gst_byte_reader_get_uint32_be (data, &num_entries))
     return FALSE;
 
   GST_LOG ("%*s  track ID:      %u", depth, "", track_id);