Created attachment 116556 [details] [review] Proposed fix for memleak in GetConnectionCredentials handler I've run dbus-daemon in valgrind and it reported memleaks after calling GetConnectionCredentials with valid unique id as the parameter. The reason for that seems to be lack of proper reply message unreferencing in GetConnectionCredentials handler. Proposed fix in attachment. I would be grateful to apply this fix also to 1.8.x branch.
Comment on attachment 116556 [details] [review] Proposed fix for memleak in GetConnectionCredentials handler Review of attachment 116556 [details] [review]: ----------------------------------------------------------------- The general idea looks right, the actual change not so much. Please use git format-patch instead of git show, it makes it much easier to preserve attribution when applying patches. ::: bus/driver.c @@ +1755,4 @@ > goto oom; > } > > + bus_message_unref (reply); Did you test this? There is no function of that name. There is a dbus_message_unref() whose use here would fix the bug, though :-)
Fixed in git for 1.8.20 and 1.9.18 (with function name corrected).
Yes, you are right. Sorry for being careless. I tested it on Tizen D-Bus repository with dbus_message_unref and it worked correctly. Then I wrote similar patch in the upstream repo making a typo by the way without testing it again. Thanks for correcting and applying the fix.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.