Opened 18 years ago

Closed 18 years ago

#565 closed patch (fixed)

Merging TS packets into PES packets must account for the payload start offset

Reported by: redmundt@… Owned by: danielk
Priority: minor Milestone: unknown
Component: mythtv Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

The attached patch starts the memory copy in AddTSPacket after the payload start offset. Additionally we now track the pesdata size instead of the number of TS packets. This allows us to start the memory copy in the exact location when merging packets after a start offset has been eliminated.

Overall effect: TS packets that contain PMT packets no longer get merged with the extra offset byte. PMT entries that are split among TS entries now have the audio and video pids properly parsed. This reduces (if not eliminates) the partial locks I've received.

Attachments (1)

offset.diff​ (4.0 KB) - added by redmundt@… 18 years ago.

Download all attachments as: .zip

Change History (3)

Changed 18 years ago by redmundt@…

Attachment: offset.diff​ added

comment:1 Changed 18 years ago by danielk

Milestone: → unknown
Owner: changed from Isaac Richards to danielk

comment:2 Changed 18 years ago by danielk

Resolution: → fixed
Status: new → closed

(In [7678]) Closes #565, by applying patch.

Accounts for size of offset pointer when assembling PES packets.

Note: See TracTickets for help on using tickets.