Created attachment 85531 [details] [review] [PATCH 1/2] Fix pass wrong type of argument to function
Created attachment 85532 [details] [review] [PATCH 2/2] Use the function argument rather than hard-code
Comment on attachment 85531 [details] [review] [PATCH 1/2] Fix pass wrong type of argument to function Review of attachment 85531 [details] [review]: ----------------------------------------------------------------- OK for master. ISO C says the two usages are equivalent, so this is not significant enough for stable branches.
Comment on attachment 85532 [details] [review] [PATCH 2/2] Use the function argument rather than hard-code Review of attachment 85532 [details] [review]: ----------------------------------------------------------------- > This fix to use the function argument, meanwhile, this change doesn't > has any effect in practice since this function only invoked once with > a TRUE argument. This function is public API, so in principle, a libdbus user could have been calling dbus_connection_set_route_peer_messages(c, FALSE) - but that's rather unlikely, and would have had the opposite of the desired effect, so I think we can conclude that nobody does. I think that makes it worth applying this to dbus-1.6 - it's a real bugfix, not just an internal change.
Fixed in git for 1.6.16 (2/2), 1.7.6 (both).
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.