flacparse: fix off by one in frame size check
authorVincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Tue, 16 Aug 2011 12:16:22 +0000 (13:16 +0100)
committerTim-Philipp Müller <tim.muller@collabora.co.uk>
Tue, 16 Aug 2011 12:25:30 +0000 (13:25 +0100)
Yes, I was tracking another bug and the small test file I generated
to test with improbably just happened to trigger this, with a second
and last frame of 1615 bytes.

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

gst/audioparsers/gstflacparse.c

index 0c6c529..3011bf0 100644 (file)
@@ -581,7 +581,7 @@ gst_flac_parse_frame_is_valid (GstFlacParse * flacparse,
   data = GST_BUFFER_DATA (buffer);
   size = GST_BUFFER_SIZE (buffer);
 
-  if (size <= flacparse->min_framesize)
+  if (size < flacparse->min_framesize)
     goto need_more;
 
   header_ret =