Modify
Warning Please read the Ticket HowTo before creating or commenting on a ticket. Failure to do so may cause your ticket to be rejected or result in a slower response.

Opened 3 years ago

Closed 3 years ago

#9716 closed Bug Report - Crash (fixed)

assert triggered at cc708decoder.cpp:268 with ATSC

Reported by: jpoet Owned by: danielk
Priority: minor Milestone: 0.25
Component: MythTV - ATSC Version: Master Head
Severity: medium Keywords: mythcommflag mythfrontend playback segfault crash
Cc: Ticket locked: no

Description

Over the past couple of week mythcommflag has been dying on about a quarter of the shows recorded from my local CBS station.

BT attached.

Attachments (2)

commflag-gdb.txt (7.4 KB) - added by jpoet 3 years ago.
BT of abort
sample-runs (3.5 KB) - added by km@… 3 years ago.
successive flag runs on same file

Download all attachments as: .zip

Change History (11)

Changed 3 years ago by jpoet

BT of abort

comment:1 Changed 3 years ago by mdean

See, also, #9741 for additional logging and backtrace information.

comment:2 Changed 3 years ago by beirdo

  • Component changed from MythTV - Mythcommflag to MythTV - ATSC
  • Owner changed from cpinkham to danielk
  • Status changed from new to assigned

This is an assert in cc708decoder.cpp. As this is danielk's code, it's likely better off in his queue. Hopefully, a way to recode this without an assert can be found.

comment:3 Changed 3 years ago by km@…

I'm seeing the same error on some recordings from different channels.

comment:4 Changed 3 years ago by robertm

  • Ticket locked set

READ THE TICKET HOWTO. ME TOO IS NOT HELPFUL.

comment:5 Changed 3 years ago by danielk

  • Milestone changed from unknown to 0.25
  • Status changed from assigned to infoneeded
  • Ticket locked unset

I need a sample file that produces this crash, ideally a short sample.

Changed 3 years ago by km@…

successive flag runs on same file

comment:6 Changed 3 years ago by km@…

I don't have a small file right now. I'm attaching the terminal output from some successive runs on a 2GB file which shows that its non-deterministic. Most of the time it aborts in cc708decoder.cpp while searching for logo. A few of the runs get past that, and I abort them manually.

comment:7 Changed 3 years ago by mdean

  • Keywords mythfrontend playback crash added
  • Summary changed from mythcommflag aborting at cc708decoder.cpp:268 with ATSC to assert triggered at cc708decoder.cpp:268 with ATSC

Modifying title so that it's clear (as shown in #9741 and its backtraces) that this can occur during any playback (for example in mythfrontend), and is not limited to only mythcommflag

comment:8 Changed 3 years ago by km@…

I found a small file (20MB), still too large to attach. It's available as

http://www.mathcs.emory.edu/~km/sample.mpg

comment:9 Changed 3 years ago by Github

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

Fixes #9716. for \(int i... shadowed the int i we had declared earlier for that loop so we could check whether the for loop exited prematurely.

Branch: master
Changeset: 8a469fc0bc284b137c244fe0c32efb8a6c6078cc

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'new'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.