Opened 6 years ago

Closed 6 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 Changed 6 years ago by cehoyos

  • Keywords mpegts added
  • Version changed from unspecified to git-master

Please provide a sample that allows to reproduce the problem.

comment:2 Changed 6 years ago by cehoyos

  • Resolution set to needs_more_info
  • Status changed from new to closed

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

Note: See TracTickets for help on using tickets.