Change #254500

Comments

avformat/demux: prevent the loss of packet side data when using an parser
Many parsers will request data until they find what will be the start of the
next assembled packet in order to decide where to cut the current one. If this
happens, the loop in demux.c will, in case the demuxer exports already fully
assembled packets as is sometimes the case for MPEG-TS, discard the already
handled first input packet before it tries to move its side data to the output.

The affected FATE tests reflect this change by no longer dropping the side data
from the first input packet, nor exporting every other side data in the wrong
output packet.

Signed-off-by: James Almer <jamrial@gmail.com>

Changed files

• libavformat/demux.c
• tests/ref/fate/concat-demuxer-simple2-lavf-ts
• tests/ref/fate/segment-mp4-to-ts
• tests/ref/fate/ts-demux
• tests/ref/fate/ts-small-demux