Opened 15 years ago

Closed 15 years ago

Last modified 14 years ago

#6117 closed patch (fixed)

Mythcommflag experimental segfaulting for all(?) recordings

Reported by: otto at kolsi dot fi Owned by: Isaac Richards
Priority: minor Milestone: 0.22
Component: mythtv Version: head
Severity: medium Keywords:
Cc: Ticket locked: yes

Description

Mythcommflag with experimental method seems to segfault for almost if not all recordings. I'm not sure if I've had a successful flagging since moving from -fixes to trunk.

Segfault seems to occur in same place, I've attached two backtraces.

Attachments (3)

mythcommflag-backtrace1.txt​ (13.5 KB) - added by otto at kolsi dot fi 15 years ago.
mythcommflag-backtrace2.txt​ (13.6 KB) - added by otto at kolsi dot fi 15 years ago.
CommDetector2_iter_fix.diff​ (1.3 KB) - added by Shane Shrybman 15 years ago.

Download all attachments as: .zip

Change History (7)

Changed 15 years ago by otto at kolsi dot fi

Attachment: mythcommflag-backtrace1.txt​ added

Changed 15 years ago by otto at kolsi dot fi

Attachment: mythcommflag-backtrace2.txt​ added

comment:1 Changed 15 years ago by Shane Shrybman

Milestone: unknown → 0.22
Type: defect → patch

Hi Otto, It looks like the iterator was being invalidated by erasing from the vector while iterating over it.

Changed 15 years ago by Shane Shrybman

Attachment: CommDetector2_iter_fix.diff​ added

comment:2 in reply to:  1 Changed 15 years ago by otto at kolsi dot fi

Replying to gnome42:

Hi Otto, It looks like the iterator was being invalidated by erasing from the vector while iterating over it.

Patch solves the issue, I've commflagged >20 recordings with it and so far none have segfaulted.

comment:3 Changed 15 years ago by shane

Resolution: → fixed
Status: new → closed

(In [20181]) Fixes segfault in experimental commercial detector. Closes #6117

comment:4 Changed 14 years ago by beirdo

Ticket locked: set
Note: See TracTickets for help on using tickets.