[Ffmpeg-cvslog] r5487 - trunk/libavcodec/parser.c

mru subversion
Fri Jun 16 22:41:34 CEST 2006

Author: mru
Date: Fri Jun 16 22:41:33 2006
New Revision: 5487


The AAC frame header uses 13 bits for the frame size, so the buffer should
have room for such a frame.  A frame that large seems unlikely in a valid
file, but leaving room for it spares us the need to check it elsewhere.
Moving the buffer to the end of the struct made debugging this easier.

Modified: trunk/libavcodec/parser.c
--- trunk/libavcodec/parser.c	(original)
+++ trunk/libavcodec/parser.c	Fri Jun 16 22:41:33 2006
@@ -729,12 +729,12 @@
 /* also used for ADTS AAC */
 typedef struct AC3ParseContext {
-    uint8_t inbuf[4096]; /* input buffer */
     uint8_t *inbuf_ptr;
     int frame_size;
     int header_size;
     int (*sync)(const uint8_t *buf, int *channels, int *sample_rate,
                 int *bit_rate, int *samples);
+    uint8_t inbuf[8192]; /* input buffer */
 } AC3ParseContext;
 #define AC3_HEADER_SIZE 7

More information about the ffmpeg-cvslog mailing list