Ticket #1655 (closed defect: needs_more_info)

Opened 9 months ago

Last modified 7 months ago

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

comment:1 Changed 9 months 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 7 months ago by cehoyos

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

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

Note: See TracTickets for help on using tickets.