[FFmpeg-devel] [PATCH 1/3] avformat/rtpdec_mpeg4: reassemble fragmented AAC frames

Gilles Chanteperdrix gilles.chanteperdrix at xenomai.org
Sat Feb 21 22:35:14 CET 2015


On Sat, Feb 21, 2015 at 04:54:12PM +0100, Thomas Volkert wrote:
> >+    if (len < data->au_headers[0].size) {
> >+        av_log(ctx, AV_LOG_ERROR, "First AU larger than packet size\n");
> 
> Can this really happen?
> In the lines above you already check for the case that data is missing while
> you have already received the last packet of the frame.

The first test tests for 
data->nb_au_headers == 1 && len < data->au_headers[0].size

We could still have:
data->nb_au_headers >= 2 && len < data->au_headers[0].size

Which is illegal per the RFC (a packet contains either a piece of a
fragmented frame, or one or several entire frames, but never both),
but some malformed packets could happen.

-- 
					    Gilles.


More information about the ffmpeg-devel mailing list