Opened 13 years ago

Closed 13 years ago

#4383 closed defect (fixed)

Mythbackend resched segfault with UPnP code

Reported by: otto at kolsi dot fi Owned by: dblain
Priority: minor Milestone: unknown
Component: upnp Version: head
Severity: medium Keywords: UPnpNotifyTask
Cc: Ticket locked: no

Description

About 40-50% of the "mythbackend --resched" executions end up with segfault in UPnP code. I've attached a backtrace. This is with FC5 dual-core machine.

Attachments (4)

backtrace.txt (13.8 KB) - added by otto at kolsi dot fi 13 years ago.
bactrace-v2.txt (14.0 KB) - added by otto at kolsi dot fi 13 years ago.
mythtv-backtrace.txt (14.3 KB) - added by otto at kolsi dot fi 13 years ago.
4383.patch (1.7 KB) - added by Anduin Withers 13 years ago.

Download all attachments as: .zip

Change History (12)

Changed 13 years ago by otto at kolsi dot fi

Attachment: backtrace.txt added

comment:1 Changed 13 years ago by Ash Bostock <abostock@…>

Would this cause the same thing that I'm seeing with mythwelcome, mythfrontend and also mythtv-setup? They segfault 40-50% of the time on first load, last terminal output is about deleting upnp. They'll either segfault straight away or if not will run perfectly fine afterwards, only occurs on initial loading.

comment:2 Changed 13 years ago by Nigel

Yes, and the same as #4346. The more GDB traces we get, the more likely we will be able to find the problem.

comment:3 Changed 13 years ago by otto at kolsi dot fi

Just tested with SVN rev 15284 to see if [15283] helped for this problem, but it does not. Attached is a new file: bactrace-v2.txt just in case more info is needed.

Changed 13 years ago by otto at kolsi dot fi

Attachment: bactrace-v2.txt added

Changed 13 years ago by otto at kolsi dot fi

Attachment: mythtv-backtrace.txt added

comment:4 Changed 13 years ago by otto at kolsi dot fi

I've attached one more backtrace, this time it is from running simply "mythtv" without any arguments. It looks like a bit different this time..

Changed 13 years ago by Anduin Withers

Attachment: 4383.patch added

comment:5 Changed 13 years ago by Anduin Withers

(In [15286]) References #4383

Make m_addressList local to SendNotifyMsg?, this is the last stop before true avoidance with a switch to QWaitCondition and letting the task queue send the disable (would also prevent the double that can/will happen now).

comment:6 Changed 13 years ago by Anduin Withers

Keywords: UPnpNotifyTask added

comment:7 Changed 13 years ago by otto at kolsi dot fi

This seems to be fixed in [15286] or some other lately change.

comment:8 Changed 13 years ago by Nigel

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.