Crash on PMT parsing #48

Closed
opened 2026-01-29 16:33:54 +00:00 by claunia · 4 comments
Owner

Originally created by @cfsmp3 on GitHub (Apr 12, 2015).

"http://www.worldtrad.org/Bones - 16.ts"

That file causes a crash here:
int write_section(struct lib_ccx_ctx _ctx, struct ts_payload *payload, unsigned char_buf, int size, int pos)
{
if (payload->pesstart)
{
memcpy(payload->section_buf, buf, size);

(size == -3)

Originally created by @cfsmp3 on GitHub (Apr 12, 2015). "http://www.worldtrad.org/Bones - 16.ts" That file causes a crash here: int write_section(struct lib_ccx_ctx _ctx, struct ts_payload *payload, unsigned char_buf, int size, int pos) { if (payload->pesstart) { memcpy(payload->section_buf, buf, size); (size == -3)
Author
Owner

@uajain commented on GitHub (Apr 14, 2015):

http://www.worldtrad.org/Bones : Link not working (here) !
If @cfsmp3 finds it working, can he/anyone upload to developer repository ?

@uajain commented on GitHub (Apr 14, 2015): http://www.worldtrad.org/Bones : Link not working (here) ! If @cfsmp3 finds it working, can he/anyone upload to developer repository ?
Author
Owner

@canihavesomecoffee commented on GitHub (Apr 14, 2015):

It's not broken, it's the total link you need: [http://www.worldtrad.org/Bones - 16.ts](http://www.worldtrad.org/Bones - 16.ts) - That's the reason why Carlos put it between quotes.

But I believe he fixed it already, though it would be better for @cfsmp3 to confirm this :)

@canihavesomecoffee commented on GitHub (Apr 14, 2015): It's not broken, it's the total link you need: [http://www.worldtrad.org/Bones - 16.ts](http://www.worldtrad.org/Bones - 16.ts) - That's the reason why Carlos put it between quotes. But I believe he fixed it already, though it would be better for @cfsmp3 to confirm this :)
Author
Owner

@uajain commented on GitHub (Apr 14, 2015):

okay, got you :)

@uajain commented on GitHub (Apr 14, 2015): okay, got you :)
Author
Owner

@cfsmp3 commented on GitHub (Apr 14, 2015):

Yes, this is fixed.

@cfsmp3 commented on GitHub (Apr 14, 2015): Yes, this is fixed.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ccextractor#48