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