Review Request 123860: Handle backend being deleted during GetConfigOperation

David Edmundson david at davidedmundson.co.uk
Wed May 20 13:09:36 UTC 2015


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

(Updated May 20, 2015, 1:09 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma and Solid.


Changes
-------

Submitted with commit d062d40582e00c8f34b8e0a1fd1e457d71b70ed9 by David Edmundson to branch Plasma/5.3.


Bugs: 347626
    http://bugs.kde.org/show_bug.cgi?id=347626


Repository: libkscreen


Description
-------

GetConfigOperation stores a raw pointer to the backend in a property of
the QDBusPendingCallWatcher

BackendManager owns this backend and can delete it at will if it becomes
invalidated, leaving a dangling pointer in the watcher's property.

BUG: 347626


Diffs
-----

  src/getconfigoperation.cpp dac7a33edd87adb53d7b773acfe7f7dfe2229394 

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


Testing
-------

Never made the crash happen myself, but looking at the backtrace and the code it seems the a way it could have happened.

Tested behaviour as before still works.


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150520/fd725a13/attachment.html>


More information about the Plasma-devel mailing list