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 e4d46569ea81cd35f802885a7e650dc89bfe591d..3e974f257e6cc9bb5a489e318f0e02193b92097f 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;