Increase FF_INPUT_BUFFER_PADDING_SIZE to 64.
authorAlex Converse <alex.converse@gmail.com>
Sun, 7 Mar 2010 21:16:44 +0000 (21:16 +0000)
committerAlex Converse <alex.converse@gmail.com>
Sun, 7 Mar 2010 21:16:44 +0000 (21:16 +0000)
The purpose of this is to give decoders a reasonable amount of buffer to work
with before needing to check for overreads.

Originally committed as revision 22288 to svn://svn.ffmpeg.org/ffmpeg/trunk

libavcodec/avcodec.h

index 31569d3..3e0e291 100644 (file)
@@ -424,12 +424,14 @@ enum SampleFormat {
 
 /**
  * Required number of additionally allocated bytes at the end of the input bitstream for decoding.
- * This is mainly needed because some optimized bitstream readers read
- * 32 or 64 bit at once and could read over the end.<br>
+ * The first 8 bytes are needed because some optimized bitstream readers read
+ * 32 or 64 bit at once and could read over the end. The remainder is to give
+ * decoders a reasonable amount of distance to work with before checking for
+ * buffer overreads.<br>
  * Note: If the first 23 bits of the additional bytes are not 0, then damaged
  * MPEG bitstreams could cause overread and segfault.
  */
-#define FF_INPUT_BUFFER_PADDING_SIZE 8
+#define FF_INPUT_BUFFER_PADDING_SIZE 64
 
 /**
  * minimum encoding buffer size