[mythtv-commits] Re: Ticket #196: 6984 changes cause SEGV

[MythTV]

#196: 6984 changes cause SEGV
--------------------------------------------------+-------------------------
 Reporter:  Anduin Withers <awithers at anduin.com>  |        Owner:  ijr   
     Type:  patch                                 |       Status:  closed
 Priority:  minor                                 |    Milestone:        
Component:  mythtv                                |      Version:  head  
 Severity:  medium                                |   Resolution:  fixed 
       Cc:                                        |  
--------------------------------------------------+-------------------------
Changes (by danielk):

  * resolution:  => fixed
  * status:  new => closed

Comment:

 (In [6986]) Closes #196.

 Moving the screensaver stuff to Init() in [6984] did not fix the
 segmentation fault caused by [6977]. Looking deeper at the problem
 it looks like we need neither the lirc nor the joystick lock at
 that point. These are only needed once there is a MythMainWindow.

 So this fix:
  * Makes util.cpp only request the locks when there is a MythContext
    and it has a MythMainWindow.
  * Adds a safety check for the existance of a MythContext in lock().
  * Initializes all the variables in MythContextPrivate as early as
    possible; to make sure all pointers are initialized to NULL if
    they haven't been set yet.

-- 
Ticket URL: <http://cvs.mythtv.org/trac/ticket/196>
MythTV <http://www.mythtv.org/>
MythTV