Opened 12 years ago

Closed 11 years ago

#1655 closed defect (needs_more_info)

mpegts demuxer does not recognize invalid PES header

Reported by: alphamoose Owned by:
Priority: normal Component: avformat
Version: git-master Keywords: mpegts
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

Is there a special reason, why the mpegts demuxer does not give a warning, when the PES header start code is missing? Instead it just skips the invalid TS packet payload. The code comment says, it assumes the data to be a "table". I never heard about "tables" in a PES stream though.
http://git.videolan.org/?p=ffmpeg.git;a=blob;f=libavformat/mpegts.c;hb=HEAD#l854
I think ffmpeg should at least give a warning message or even set the AV_PKT_FLAG_CORRUPT flag, when it skips that data without interpreting it at all.

Change History (2)

comment:1 by Carl Eugen Hoyos, 12 years ago

Keywords: mpegts added
Version: unspecifiedgit-master

Please provide a sample that allows to reproduce the problem.

comment:2 by Carl Eugen Hoyos, 11 years ago

Resolution: needs_more_info
Status: newclosed

Please reopen if you can share a sample to reproduce the problem.

Note: See TracTickets for help on using tickets.