Ticket #47 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

mpegts muxer doesn't override time base, but assumes 1/90000

Reported by: divVerent Owned by:
Priority: normal Component: avformat
Version: git Keywords: mpeg
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

The subject says all: the mpegts muxer does not set the stream time base using av_set_pts_info, but expects it to have the default value of 1/90000. If the calling code changes stream pts before writing the header, other muxers override the time base at that point (like mpegenc.c). mpegtsenc.c does not, causing this issue.

In my code, I had to work around this by not setting the stream time base at all.

I attached a patch that adds the missing function call to mpegtsenc.c.

Attachments

fix90000.diff Download (476 bytes) - added by divVerent 2 years ago.

Change History

Changed 2 years ago by divVerent

comment:1 Changed 2 years ago by cehoyos

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

Patch pushed in 0a962e61e956addb7cdc829e41b82fd76bea55db (April 14)

Note: See TracTickets for help on using tickets.