Opened 15 years ago
Closed 15 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)
Changed 15 years ago by
| Attachment: | mythweather.backtrace added |
|---|
Changed 15 years ago by
| Attachment: | mythweather_fix.patch added |
|---|
patch to fix seg fault in MythWeather?
comment:1 Changed 15 years ago by
| Type: | patch → defect |
|---|
We don't want modal dialog handling with MythUI so the patch is not useful, but the backtrace is.
Changed 15 years ago by
| Attachment: | mythweather_fix.2.patch added |
|---|
Alternate patch to fix segfault in MythWeather?.
comment:2 Changed 15 years ago by
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'
Changed 15 years ago by
| Attachment: | mythweather.2.backtrace added |
|---|
Backtrace of segfault in the MythWeather? screen setup window.
Changed 15 years ago by
| Attachment: | mythweather_fix.4.patch added |
|---|
An updated patch to fix both segfaults.
comment:3 Changed 15 years ago by
(In [21104]) Fix a segfault in MythWeather?'s setup screen when escaping out of the popup menu. Patch from Marc Alban. Refs #6536.
backtrace