D17067: do not leak resolver in remoteservice
Harald Sitter
noreply at phabricator.kde.org
Wed Nov 21 09:26:43 GMT 2018
sitter created this revision.
sitter added a reviewer: broulik.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
sitter requested review of this revision.
REVISION SUMMARY
avahi's dbus api expects us to control the live time of objects. to that
end we need to hold our wrapper and free it when we no longer need it.
this already has cleanup code for m_resolver, unfortunately the member was
never set.
TEST PLAN
builds
REPOSITORY
R272 KDNSSD
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D17067
AFFECTED FILES
src/avahi-remoteservice.cpp
To: sitter, broulik
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181121/cc0ccf59/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list