[Okular-devel] Review Request 123876: missing initialisation of PageViewPrivate::aFitWindowToPage

René J.V. Bertin rjvbertin at gmail.com
Thu May 21 21:24:30 UTC 2015


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

Review request for KDE Software on Mac OS X and Okular.


Repository: okular


Description
-------

`PageViewPrivate::aFitWindowToPage` isn't initialised to NULL like `aMouseMagnifier` is, and this leads to jumping to an uninitialised address when `aFitWindowToPage->setEnabled()` is called.

This patch initialises the member to NULL, and adds a check in `PageView::slotRelayoutPages()`.


Diffs
-----

  ui/pageview.cpp b57e6ae 

Diff: https://git.reviewboard.kde.org/r/123876/diff/


Testing
-------

On OS X 10.9 with kdelibs 4.14.7 and Qt 4.8.7 . This patch prevents a crash that would otherwise occur quickly after doing File/Print Preview.


Thanks,

René J.V. Bertin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20150521/101d656f/attachment.html>


More information about the Okular-devel mailing list