Modify

Ticket #9654 (accepted Patch - Bug Fix)

Opened 2 years ago

Last modified 13 months ago

[PATCH] Fix SEGV faults in mythfrontend during backend selection

Reported by: Lawrence Rust <lvr@…> Owned by: mdean
Priority: minor Milestone: unknown
Component: MythTV - User Interface Library Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

This patch fixes several segv faults and logic problems with mythfrontend during backend configuration either after a fresh install or by specifying the -p option.

1 Can't select Manual dbase configuration with -p option
2 Can't Cancel
3 Segv after selecting Save on win32
4 Segv in ShowOkPopup? due to stale static data
5 Segv in MythScreenStack::PopScreen? due to null pointer

Attachments

be-select.diff (9.0 KB) - added by Lawrence Rust <lvr@…> 2 years ago.

Change History

Changed 2 years ago by Lawrence Rust <lvr@…>

comment:1 Changed 2 years ago by stuartm

Are these issues reproducible on other platforms? I can't reproduce any of them.

comment:2 Changed 2 years ago by Lawrence Rust <lvr@…>

All apart from 1 are reproducible on both Linux & Windows. The exception is the seg fault when you select Save caused by a NULL m_XML in BackendSelection::Accept

  1. Start "mythfrontend -p", select language, tab to Manual and click. If your setup conforms to defaults you'll go to the root menu. If you have anything unusual you'll get a dialog - Cannot login.
  2. Start "mythfrontend -p", select language, tab to cancel and click, same result as 1.
  3. Win32 only: Start "mythfrontend -p", select language, tab to Save and click - SEGV
  4. Start "mythfrontend -p", select language, tab to cancel and click - Cannot login. Click cancel again - SEGV

comment:3 Changed 2 years ago by mdean

  • Owner changed from stuartm to mdean
  • Status changed from new to accepted

comment:4 Changed 20 months ago by paulh

  • Milestone changed from unknown to 0.25

comment:5 Changed 18 months ago by mdean

Ref: f53cc94e8

comment:6 Changed 14 months ago by stuartm

  • Priority changed from minor to blocker

comment:7 Changed 13 months ago by mdean

  • Priority changed from blocker to minor

Segfaults have been fixed in master (by the above-referenced changeset and potentially other additional changes). (Though I can't test the win32 segfault, those mentioned as #4 and #5 in original description are fixed.)

comment:8 Changed 13 months ago by beirdo

  • Milestone changed from 0.25 to unknown

All that is pending is:

Segv after selecting Save on win32

View

Add a comment

Modify Ticket

Action
as accepted
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.