flacdec: bail on reserved value
authorVincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Tue, 16 Aug 2011 14:32:07 +0000 (15:32 +0100)
committerTim-Philipp Müller <tim.muller@collabora.co.uk>
Tue, 16 Aug 2011 23:02:38 +0000 (00:02 +0100)
Now that we look at the right bits, we can test against the reserved
value as we do for other fields.

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

ext/flac/gstflacdec.c

index 8fdd1091b467213a047d6117b86930e12ee5529d..94f77899f34fbcb3c152470b8d3dd162e893565f 100644 (file)
@@ -424,7 +424,7 @@ gst_flac_dec_scan_got_frame (GstFlacDec * flacdec, guint8 * data, guint size,
   GST_LOG_OBJECT (flacdec,
       "got sync, bs=%x,sr=%x,ca=%x,ss=%x,pb=%x", bs, sr, ca, ss, pb);
 
-  if (sr == 0x0F || ca >= 0x0B || ss == 0x03 || ss == 0x07) {
+  if (bs == 0 || sr == 0x0F || ca >= 0x0B || ss == 0x03 || ss == 0x07) {
     return FALSE;
   }