vc1parse: Fix seq hdr STRUCT_A handling for advanced profile.
authorSreerenj Balachandran <sreerenj.balachandran@intel.com>
Wed, 7 Aug 2013 11:52:43 +0000 (14:52 +0300)
committerTim-Philipp Müller <tim@centricular.net>
Mon, 12 Aug 2013 15:12:52 +0000 (16:12 +0100)
The Sequence Header Data Structure STRUCT_A for advanced profile
may be eight consecutive zero bytes.Don't try to override the
width and height values in this case.

https://bugzilla.gnome.org/show_bug.cgi?id=705667

gst/videoparsers/gstvc1parse.c

index e4d4656..3e974f2 100644 (file)
@@ -1415,7 +1415,8 @@ gst_vc1_parse_handle_seq_layer (GstVC1Parse * vc1parse,
 
   width = vc1parse->seq_layer.struct_a.vert_size;
   height = vc1parse->seq_layer.struct_a.horiz_size;
-  if (vc1parse->width != width || vc1parse->height != height) {
+  if (width > 0 && height > 0
+      && (vc1parse->width != width || vc1parse->height != height)) {
     vc1parse->update_caps = TRUE;
     vc1parse->width = width;
     vc1parse->height = height;