Opened 11 years ago

Closed 10 years ago

Last modified 9 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 11 years ago.
mythcommflag-backtrace2.txt (13.6 KB) - added by otto at kolsi dot fi 11 years ago.
CommDetector2_iter_fix.diff (1.3 KB) - added by Shane Shrybman 10 years ago.

Download all attachments as: .zip

Change History (7)

Changed 11 years ago by otto at kolsi dot fi

Attachment: mythcommflag-backtrace1.txt added

Changed 11 years ago by otto at kolsi dot fi

Attachment: mythcommflag-backtrace2.txt added

comment:1 Changed 10 years ago by Shane Shrybman

Milestone: unknown0.22
Type: defectpatch

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

Changed 10 years ago by Shane Shrybman

Attachment: CommDetector2_iter_fix.diff added

comment:2 in reply to:  1 Changed 10 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 10 years ago by shane

Resolution: fixed
Status: newclosed

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

comment:4 Changed 9 years ago by beirdo

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