Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#8223 closed defect (fixed)

MythWeather segfaults opening busy popup when scanning for new scripts

Reported by: Raymond Wagner Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: Plugin - MythWeather Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Using a fresh install of trunk, revision 23815, built on FreeBSD. MythWeather? crashes when trying to open its busy popup, when it scans for scripts prior to opening 'Screen Settings' in setup. Commenting out the respective code allows the screen to open without issue. Backtrace is attached.

Attachments (1)

mw_backtrace.txt (2.9 KB) - added by Raymond Wagner 9 years ago.

Download all attachments as: .zip

Change History (3)

Changed 9 years ago by Raymond Wagner

Attachment: mw_backtrace.txt added

comment:1 Changed 9 years ago by sphery

Resolution: fixed
Status: newclosed

(In [23817]) Fixes MythWeather? setup segfault.

The fix for #8169 assumed that the MythWeather? plugin was running when it changed the SourceManager::findScripts() busy popup to appear on the weather stack. When entering MythWeather? setup through Utilities/Setup?|Setup|Info Center Settings|Weather Settings|Screen Settings, this resulted in a NULL MythScreenStack?. Therefore, if the MythScreenStack? for the weather stack is null, just use the popup stack.

Fixes #8223. Refs #8169.

comment:2 Changed 9 years ago by sphery

(In [23818]) Fix segfault in MythWeather? screen setup. Backports [23817] from trunk. Refs #8223.

Note: See TracTickets for help on using tickets.