[Libreoffice] [REVIEW][PATCH] crash releaseTimer on NULL mpPresTimer

Caolán McNamara caolanm at redhat.com
Tue Dec 6 15:10:37 PST 2011


http://cgit.freedesktop.org/libreoffice/core/commit/?id=d57887eaad534ca51fc4975918fab66b2f93633d

Reproducer at https://bugzilla.redhat.com/show_bug.cgi?id=759647

I see that the mpPresTimer in slideshow is reset when dispose is called.
However a scopeGuard can still try and call releaseTimer on that
mpPresTimer which was disposed since it the guard ctored. The
boost::cref that scopeGuard takes doesn't increment the shared_ptr, its
effectively just a weak reference.

Fix looks right to me anyway ? Anyone know different, and if not is it
worth cherry-picking for 3-5 ?

C.



More information about the LibreOffice mailing list