[BUG] Segmentation Fault part way through extracting from DVB #491

New Issue

claunia · 2026-01-29T16:45:17Z

claunia commented

2026-01-29 16:45:17 +00:00

Originally created by @bcb2000 on GitHub (Apr 28, 2019).

CCExtractor version (using the --version parameter preferably) : 0.87

In raising this issue, I confirm the following (please check boxes, eg [X] - and delete unchecked ones):

I have read and understood the contributors guide.
I have checked that the bug-fix I am reporting can be replicated, or that the feature I am suggesting isn't already present.
I have checked that the issue I'm posting isn't already reported.
I have checked that the issue I'm porting isn't already solved and no duplicates exist in closed issues and in opened issues
I have checked the pull requests tab for existing solutions/implementations to my issue/suggestion.
I have used the latest available version of CCExtractor to verify this issue exists.

My familiarity with the project is as follows (check one, eg [X] - and delete unchecked ones):

I have used CCExtractor just a couple of times.

Necessary information

Is this a regression (did it work before)? [X] NO | [ ] YES - please specify the last known working version
What platform did you use? [ ] Windows - [X] Linux - [ ] Mac
What were the used arguments? none

Video links (replace text below with your links)

http://213.229.121.188/1.srt
http://213.229.121.188/1.ts

Additional information

CCExtractor was installed using the ./build method
OS is Debian 9.1

$ ccextractor /tmp/1.ts
CCExtractor 0.87, Carlos Fernandez Sanz, Volker Quetschke.
Teletext portions taken from Petr Kutalek's telxcc

Input: /tmp/1.ts
[Extract: 1] [Stream mode: Autodetect]
[Program : Auto ] [Hauppage mode: No] [Use MythTV code: Auto]
[Timing mode: Auto] [Debug: No] [Buffer input: No]
[Use pic_order_cnt_lsb for H.264: No] [Print CC decoder traces: No]
[Target format: .srt] [Encoding: UTF-8] [Delay: 0] [Trim lines: No]
[Add font color data: Yes] [Add font typesetting: Yes]
[Convert case: No] [Video-edit join: No]
[Extraction start time: not set (from start)]
[Extraction end time: not set (to end)]
[Live stream: No] [Clock frequency: 90000]
[Teletext page: Autodetect]
[Start credits text: None]
[Quantisation-mode: CCExtractor's internal function]

Opening file: /tmp/1.ts
File seems to be a transport stream, enabling TS mode
Analyzing data in general mode
18% | 11:11Segmentation fault

The resulting .srt is complete as far as to 11:44.

Originally created by @bcb2000 on GitHub (Apr 28, 2019). CCExtractor version (using the --version parameter preferably) : **0.87** **In raising this issue, I confirm the following (please check boxes, eg [X] - and delete unchecked ones):** - [X] I have read and understood the [contributors guide](https://github.com/CCExtractor/ccextractor/blob/master/.github/CONTRIBUTING.md). - [X] I have checked that the bug-fix I am reporting can be replicated, or that the feature I am suggesting isn't already present. - [X] I have checked that the issue I'm posting isn't already reported. - [X] I have checked that the issue I'm porting isn't already solved and no duplicates exist in [closed issues](https://github.com/CCExtractor/ccextractor/issues?q=is%3Aissue+is%3Aclosed) and in [opened issues](https://github.com/CCExtractor/ccextractor/issues) - [X] I have checked the pull requests tab for existing solutions/implementations to my issue/suggestion. - [X] I have used the latest available version of CCExtractor to verify this issue exists. **My familiarity with the project is as follows (check one, eg [X] - and delete unchecked ones):** - [X] I have used CCExtractor just a couple of times. **Necessary information** - Is this a regression (did it work before)? [X] NO | [ ] YES - *please specify the last known working version* - What platform did you use? [ ] Windows - [X] Linux - [ ] Mac - What were the used arguments? none **Video links (replace text below with your links)** http://213.229.121.188/1.srt http://213.229.121.188/1.ts **Additional information** - CCExtractor was installed using the `./build` method - OS is Debian 9.1 > $ ccextractor /tmp/1.ts > CCExtractor 0.87, Carlos Fernandez Sanz, Volker Quetschke. > Teletext portions taken from Petr Kutalek's telxcc > > -------------------------------------------------------------------------- > > Input: /tmp/1.ts > [Extract: 1] [Stream mode: Autodetect] > [Program : Auto ] [Hauppage mode: No] [Use MythTV code: Auto] > [Timing mode: Auto] [Debug: No] [Buffer input: No] > [Use pic_order_cnt_lsb for H.264: No] [Print CC decoder traces: No] > [Target format: .srt] [Encoding: UTF-8] [Delay: 0] [Trim lines: No] > [Add font color data: Yes] [Add font typesetting: Yes] > [Convert case: No] [Video-edit join: No] > [Extraction start time: not set (from start)] > [Extraction end time: not set (to end)] > [Live stream: No] [Clock frequency: 90000] > [Teletext page: Autodetect] > [Start credits text: None] > [Quantisation-mode: CCExtractor's internal function] > > ----------------------------------------------------------------- > > Opening file: /tmp/1.ts > File seems to be a transport stream, enabling TS mode > Analyzing data in general mode > 18% | 11:11Segmentation fault The resulting .srt is complete as far as to 11:44.

claunia closed this issue

2026-01-29 16:45:17 +00:00

claunia commented

2026-01-29 16:45:18 +00:00

@thelastpolaris commented on GitHub (Apr 28, 2019):

I can confirm this issue and will look at it in more details soon.

@thelastpolaris commented on GitHub (Apr 28, 2019): I can confirm this issue and will look at it in more details soon.

claunia commented

2026-01-29 16:45:18 +00:00

@thelastpolaris commented on GitHub (Apr 28, 2019):

The issue at 11:44 is due to the absence of check whether last_font_tag_end in ocr.c:662 actually contains text or not. However, I get another SEGFAULT with your file in the middle of processing. Will look at it and commit the changes.

@thelastpolaris commented on GitHub (Apr 28, 2019): The issue at 11:44 is due to the absence of check whether `last_font_tag_end` in ocr.c:662 actually contains text or not. However, I get another SEGFAULT with your file in the middle of processing. Will look at it and commit the changes.

claunia commented

2026-01-29 16:45:18 +00:00

@bcb2000 commented on GitHub (Apr 28, 2019):

Thanks for your swift analysis and investigation!

For background, the sample .ts file is the output from Plex, whose source is a HDHomeRun network tuner. I initially thought the issue was related to HD streams (the sample I provided is HD) but I have also seen the same issue with SD streams. From a small sample, it seems that at least half of the .ts files I've grabbed can demonstrate this behaviour in CCExtractor.

I'm happy to provide more samples if required, and will test any published updates against my backlog of .ts files.

Thanks again!

@bcb2000 commented on GitHub (Apr 28, 2019): Thanks for your swift analysis and investigation! For background, the sample .ts file is the output from Plex, whose source is a HDHomeRun network tuner. I initially thought the issue was related to HD streams (the sample I provided is HD) but I have also seen the same issue with SD streams. From a small sample, it seems that at least half of the .ts files I've grabbed can demonstrate this behaviour in CCExtractor. I'm happy to provide more samples if required, and will test any published updates against my backlog of .ts files. Thanks again!

claunia commented

2026-01-29 16:45:18 +00:00

@thelastpolaris commented on GitHub (Apr 30, 2019):

Ok, so I made a pull request with changes that solve the bug you filed, but it doesn't solve all the problems that we have in DVB module. I'm getting a segfault at 33:53 with free(): invalid next size (normal) description. Kindly ask you to test extensively CCExtractor on your files with DVB. The problem, which you described, should be solved by this commit, but there can be other problems with DVB support. Please, tell me if you will encounter any more segfaults.

While waiting for pull request to be pulled to master you can try the new changes by downloading and building CCExtractor from this repo

@thelastpolaris commented on GitHub (Apr 30, 2019): Ok, so I made a pull request with changes that solve the bug you filed, but it doesn't solve all the problems that we have in DVB module. I'm getting a segfault at 33:53 with `free(): invalid next size (normal)` description. Kindly ask you to test extensively CCExtractor on your files with DVB. The problem, which you described, should be solved by this commit, but there can be other problems with DVB support. Please, tell me if you will encounter any more segfaults. While waiting for pull request to be pulled to master you can try the new changes by downloading and building CCExtractor from this [repo](https://github.com/thelastpolaris/ccextractor/tree/issue_1084)

claunia commented

2026-01-29 16:45:19 +00:00

@bcb2000 commented on GitHub (May 1, 2019):

Thanks again.

My testing is continuing. I have set master and issue_1084 branches away on my full set of 220 .ts files, most (if not all?) seem to have dvb subtitles.

So far, master has processed 190, and had 110 segfaults
issue_1084 has processed 125, and had 1 segfault.

The single segfault for issue_1084 branch is on this file:
http://213.229.121.188/2.ts [836M]

I will post final test results when both versions have processed all files.

@bcb2000 commented on GitHub (May 1, 2019): Thanks again. My testing is continuing. I have set master and issue_1084 branches away on my full set of 220 .ts files, most (if not all?) seem to have dvb subtitles. So far, master has processed 190, and had 110 segfaults issue_1084 has processed 125, and had 1 segfault. The single segfault for issue_1084 branch is on this file: http://213.229.121.188/2.ts [836M] I will post final test results when both versions have processed all files.

claunia commented

2026-01-29 16:45:19 +00:00

@thelastpolaris commented on GitHub (May 1, 2019):

@bcb2000 Thx for testing! It looks like this patch solved most of the issues but let's wait for the end results of your testing. Patch is available in master now.

Also I processed 2.ts file that you provided and got no segfault. Some issues with DVB module look pretty random.

@thelastpolaris commented on GitHub (May 1, 2019): @bcb2000 Thx for testing! It looks like this patch solved most of the issues but let's wait for the end results of your testing. Patch is available in master now. Also I processed 2.ts file that you provided and got no segfault. Some issues with DVB module look pretty random.

claunia commented

2026-01-29 16:45:19 +00:00

@bcb2000 commented on GitHub (May 2, 2019):

All of my ts files have now been processed by the issue_1084 branch of ccextractor.
master: 129 Segfaults from 220 .ts files. 5 ended abruptly with "Aborted"
fix: 1 Segfault (as previously reported - I tried again and still get a Segfault) 17 "Aborted"

The fix seems very good to me.

On a point of order, would you like me to raise a new bug for the 17 "Aborted" scenarios? There's something not right about the files because ffmpeg also has problems with them (if I use -c:s copy arguments) and also aborts without completing processing.

@bcb2000 commented on GitHub (May 2, 2019): All of my ts files have now been processed by the issue_1084 branch of ccextractor. **master**: 129 Segfaults from 220 .ts files. 5 ended abruptly with "Aborted" **fix**: 1 Segfault (as previously reported - I tried again and still get a Segfault) 17 "Aborted" The fix seems very good to me. On a point of order, would you like me to raise a new bug for the 17 "Aborted" scenarios? There's something not right about the files because ffmpeg also has problems with them (if I use ```-c:s copy``` arguments) and also aborts without completing processing.

claunia commented

2026-01-29 16:45:19 +00:00

@thelastpolaris commented on GitHub (May 2, 2019):

@bcb2000 I'm happy to hear that it helped :)

On a point of order, would you like me to raise a new bug for the 17 "Aborted" scenarios?

Sure. Please, add sample files with which you get "aborted" scenario to the issue.

@thelastpolaris commented on GitHub (May 2, 2019): @bcb2000 I'm happy to hear that it helped :) > On a point of order, would you like me to raise a new bug for the 17 "Aborted" scenarios? Sure. Please, add sample files with which you get "aborted" scenario to the issue.

claunia referenced this issue

2026-01-29 17:15:39 +00:00

[PR #491] [CLOSED] Split to sentences implementation #1306

claunia referenced this issue

2026-01-29 17:16:01 +00:00

[PR #537] [MERGED] Split to sentences implementation #1366

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/ccextractor#491