Opened 16 years ago
Closed 16 years ago
#6536 closed defect (fixed)
Segfault in MythWeather
| Reported by: | Owned by: | Isaac Richards | |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.22 |
| Component: | Plugin - MythWeather | Version: | head |
| Severity: | medium | Keywords: | segfault |
| Cc: | Ticket locked: | no |
Description
1) Enter the MythWeather screen. 2) Press 'MENU'. 3) While the busy dialog is showing press 'ESC'.
The escape key press is not being handled by the busy dialog and is causing the the underlying weather dialog to deallocate while the source manager is still processing.
Attached is a backtrace and a patch which adds an optional parameter to the busy and progress dialogs to make them modal. This will ensure that the processing has completed before deallocating.
Attachments (6)
Change History (10)
by , 16 years ago
| Attachment: | mythweather.backtrace added |
|---|
comment:1 by , 16 years ago
| Type: | patch → defect |
|---|
We don't want modal dialog handling with MythUI so the patch is not useful, but the backtrace is.
by , 16 years ago
| Attachment: | mythweather_fix.2.patch added |
|---|
Alternate patch to fix segfault in MythWeather.
comment:2 by , 16 years ago
I found another segfault in the screen setup window.
To reproduce:
- Enter MythWeather screen setup window
- Select an existing screen on the right side
- Press 'Enter'
- Press 'Esc'
by , 16 years ago
| Attachment: | mythweather.2.backtrace added |
|---|
Backtrace of segfault in the MythWeather screen setup window.
by , 16 years ago
| Attachment: | mythweather_fix.4.patch added |
|---|
An updated patch to fix both segfaults.
comment:3 by , 16 years ago
(In [21104]) Fix a segfault in MythWeather's setup screen when escaping out of the popup menu. Patch from Marc Alban. Refs #6536.
backtrace