[mythtv-firehose] mythtv/master commit: 098ead81b by Chris Pinkham (cpinkham)

MythTV noreply at mythtv.org
Sun Oct 23 03:12:46 UTC 2011


      Author:  Chris Pinkham <cpinkham at mythtv.org>
 Change Date:  2011-10-22T20:05:00-07:00
   Push Date:  2011/10/22 20:12:33 -0700
  Repository:  mythtv
      Branch:  master
New Revision:  098ead81b8c75f596b128d5830ecbb8ae10a83e9
   Changeset:  https://github.com/MythTV/mythtv/commit/098ead81b

Log:

Fix potential segfault in MythUI teardown.

When the main screen stack is deleted, NULL out the pointer
tracking it in MythMainWindowPrivate.

In MythScreenStack::PopScreen(), check that the pointer
returned by MythMainWindow::GetMainStack() is non-NULL before trying
to use it to check the top screen on the main stack.

I believe this cures the segfault reported in IRC by several people
that occurred in QRect::width() when called by MythUIType::SetRedraw()
from MythScreenStack::PopScreen().  It was hard to get decent debugging
output on this, but it appears we were trying to access the already-
deleted main screen stack when deleting the background screen stack.

Modified:

   mythtv/libs/libmythui/mythmainwindow.cpp
   mythtv/libs/libmythui/mythscreenstack.cpp



More information about the mythtv-firehose mailing list