Opened 13 years ago

Closed 13 years ago

#11047 closed Bug Report - Crash (Fixed)

Possible mythuiimage decref segfault

Reported by: trebor_s@… Owned by: stuartm
Priority: major Milestone: 0.26.1
Component: Plugin - MythMusic Version: Master Head
Severity: low Keywords:
Cc: Ticket locked: no

Description

The call to im->DecrRef() in mythuiimage.cpp in the customEvent method can cause a segfault. I had one segfault when I was listening to internet radio with the mythmusic plugin and it tried to fetch the Deutschland Radio logo. Adding a check "if (im)" fixed that. Patch attached.

Attachments (1)

mythuiimage_decref.patch​ (540 bytes ) - added by anonymous 13 years ago.

Download all attachments as: .zip

Change History (8)

by anonymous, 13 years ago

Attachment: mythuiimage_decref.patch​ added

comment:1 by stuartm, 13 years ago

Component: MythTV - General → Plugin - MythMusic
Milestone: unknown → 0.26
Owner: set to stuartm
Priority: minor → critical
Status: new → accepted

comment:2 by stuartm, 13 years ago

The patch treats the symptom but not the root cause. That pointer should not be null and I'll need to do some digging to discover exactly how it came to be deleted.

comment:3 by stuartm, 13 years ago

Milestone: 0.26 → 0.26.1

comment:4 by Stuart Morgan <smorgan@…>, 13 years ago

In b4deb185994d6388b55cea18aa3e861c71541101/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:5 by Stuart Morgan <smorgan@…>, 13 years ago

In 178ea5aaa562845e3685ffeff585ea57c6014a64/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:6 by stuartm, 13 years ago

Priority: critical → major
Severity: medium → low

comment:7 by stuartm, 13 years ago

Resolution: → Fixed
Status: accepted → closed

Too many other things to worry about, so closing this.

Note: See TracTickets for help on using tickets.