Opened 18 years ago
Closed 18 years ago
Last modified 18 years ago
#3921 closed defect (fixed)
Segfault leaving Inactive Screens in Screen Setup
| Reported by: | anonymous | Owned by: | danielk |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.21 |
| Component: | mythweather | Version: | head |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description
Subversion trunk 14374 has a reproducible (for me at least) segfault when doing the following. Go into MythWeather setup and choose Screens which brings up "Screen Setup" screen. I have never installed MythWeather on this machine before so there are no screens currently defined. "Current Conditions" is highlighted under "Inactive Screens". Down arrow to the last item on the list (18 Hour Forecast) and then down arrow once more to encounter the segfault.
Notes:
- This DOES NOT HAPPEN if you have any active screens defined.
- This DOES NOT HAPPEN if you have had an active screen defined and then removed it, UNTIL you escape back one screen and then re-enter Screen Setup with no active items on the list.
Backtrace will be attached.
Attachments (3)
Change History (13)
by , 18 years ago
| Attachment: | gdb.txt.gz added |
|---|
comment:1 by , 18 years ago
This is supposed to be for mythweather but got put in mythtv. Sorry about that. Could someone please fix.
comment:2 by , 18 years ago
| Component: | mythtv → mythweather |
|---|---|
| Milestone: | unknown → 0.21 |
| Owner: | changed from to |
comment:3 by , 18 years ago
This problem occurs because the active_list is given focus when it has no elements in it. This can be fixed by disabling focus on the active_list after the buildFocusList() function runs.
The attached patch fixes the problem for me.
-- Joe Ripley vitaminjoe@…
by , 18 years ago
| Attachment: | 3921_diff.patch.txt added |
|---|
Disables initial focus for m_active_list if the list contains no items
by , 18 years ago
| Attachment: | 3921_diff.patch added |
|---|
Disables initial focus for m_active_list if the list contains no items
comment:4 by , 18 years ago
Sorry, that '.txt' extension crept in somehow... :) I've re-attached the patch with the correct filename.
-- Joe Ripley vitaminjoe@…
comment:5 by , 18 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:6 by , 18 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
Changeset [14566] calls m_active_list->allowFocus(false) in wireUI(). This sets the allowFocus property before buildFocusList() is called. As a result, it is impossible to access the 'Active Screens' list to set up location information for the active screens.
Kevin Hayes reported this on the mythtv-users mailing list. I can confirm that I have the same behavior with [14566].
-- Joe Ripley vitaminjoe@…
comment:7 by , 18 years ago
Relevant mythtv-users thread: http://www.gossamer-threads.com/lists/mythtv/users/296787
-- Joe Ripley vitaminjoe@…
comment:8 by , 18 years ago
| Owner: | changed from to |
|---|---|
| Status: | reopened → new |
I'm going to try to fix this so we can finish testing #4132.
comment:9 by , 18 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:10 by , 18 years ago
(In [14888]) Refs #4132. Fixes some miscellaneous problems with MythWeather.
There were three problems which made it impossible to test MythWeather dialogs.
- When sources are not defined (see #3921) the text of the error dialog is not visible.
- doListSelect() used an unchecked return value from showButtonPopup as an index into an array.
- connectScreen()/disconnectScreen() segfault when sources are not properly configured.
This fixes these problems so we can test the dialogs.
Backtrace