Review Request 123778: Fix crash when DBus call ends with error

David Rosca nowrep at gmail.com
Wed May 13 16:22:53 UTC 2015


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

Review request for KDE Frameworks and Ivan Čukić.


Repository: kactivities


Description
-------

If the DBus call ends with error, reportResult is not called in DBusCallFutureInterface and calling QFuture::result() without actually setting the result leads to crash.

In case of error, continuation callback is not called (maybe it would be good to somehow indicate the error?).


Diffs
-----

  src/utils/dbusfuture_p.h 336235c 

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


Testing
-------

No longer crashes


Thanks,

David Rosca

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150513/a93902fd/attachment.html>


More information about the Kde-frameworks-devel mailing list