[mythtv-commits] Ticket #4115: Network Control causes segfaults

[MythTV]

#4115: Network Control causes segfaults
---------------------+------------------------------------------------------
 Reporter:  danielk  |        Owner:  danielk
     Type:  patch    |       Status:  new    
 Priority:  minor    |    Milestone:  0.21   
Component:  mythtv   |      Version:  head   
 Severity:  medium   |   Resolution:         
  Mlocked:  0        |  
---------------------+------------------------------------------------------

Comment(by danielk):

 (In [14777]) Refs #4115. Fixes segfault in "All Tuners are Busy" dialog
 handling and adds an "Exit" option.

 I noticed two problems with the handling of this dialog when debugging the
 network control.
  * If you hit escape in this dialog it would select the first recording in
 the list for playback, rather than returning to the main menu.
  * If you send the "return" key to the frontend application when the
 frontend is not focused it segfaults.

 The first problem occurs because the "reject" action of the
 showButtonPopup() function returns '0' the index of the first recording
 until this changeset added an exit option. The second problem occurs
 because there was no range checking on the return from showButtonPopup().
 We really need to move to the new less buggy MythUI dialogs in the long
 term, but a little bit of range checking and adding an "Exit" option to
 this dialog are good ideas anyway.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/4115#comment:2>
MythTV <http://svn.mythtv.org/trac>
MythTV