Review Request 119119: bugfix: 332596 - crash when exiting unfinished practice

[Andreas Xavier]

> On July 6, 2014, 8:37 a.m., Inge Wallin wrote:
> > src/practice/practicemainwindow.cpp, lines 64-65
> > <https://git.reviewboard.kde.org/r/119119/diff/1/?file=287039#file287039line64>
> >
> >     I think the comment should be "m_floatingToolBar is a child of this class and will be deleted automatically".
> >     
> >     Or actually just remove this line. There is no need to set it to zero in the destructor because it will never be used again.

This is set to zero here because the subsequent call to toggleFullScreen will otherwise adjust the visibility of the toolBar which has either just been or about to be deleted by m_mainWindow which owns the m_floatingToolBar.  It is a race condition and a bug waiting to happen.


- Andreas


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119119/#review61678
-----------------------------------------------------------


On July 4, 2014, 8:14 p.m., Andreas Xavier wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119119/
> -----------------------------------------------------------
> 
> (Updated July 4, 2014, 8:14 p.m.)
> 
> 
> Review request for KDE Edu.
> 
> 
> Bugs: 332596
>     http://bugs.kde.org/show_bug.cgi?id=332596
> 
> 
> Repository: parley
> 
> 
> Description
> -------
> 
> Problem: The bug was that if the qstyled KLineEdit answerEdit held focus when the WrittenPracticeWidget was deleted, then a focus event passed to the deleted KLineEdit would be passed to the QStyleSheet and return an invalid pointer to an already deleted QStyle.
>     
> Fix: This patch sets the focus to helpLabel before the object is deleted. It does this by calling modeWidgetDestroyed(), which calls objectDestroyed() before the focus changing event will be invalid. 
> 
> 
> Diffs
> -----
> 
>   src/parleymainwindow.cpp 7c688a1 
>   src/practice/abstractwidget.h e9ef394 
>   src/practice/guifrontend.h 60e6fdc 
>   src/practice/guifrontend.cpp 99fc015 
>   src/practice/practicemainwindow.cpp 31349aa 
>   src/practice/writtenpracticewidget.h 9b25987 
>   src/practice/writtenpracticewidget.cpp f20ac6d 
> 
> Diff: https://git.reviewboard.kde.org/r/119119/diff/
> 
> 
> Testing
> -------
> 
> Replicated the GUI conditions of the bug by hand, before and after the fix.
> 
> 
> Thanks,
> 
> Andreas Xavier
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20140706/e32d4427/attachment.html>