[Freeipa-devel] [PATCH][SSSD] Fixing memory leak in GetUserAttributes

Martin Nagy mnagy at redhat.com
Wed Mar 4 07:43:28 UTC 2009


Stephen Gallagher wrote:
> Trivial fix. I forgot to unref the reply message after sending it (or
> if construction of the message failed).

Hm, this seems to be a bit dangerous.. I think a better idea would be
to initialize reply to NULL and then do a test after the done label:

if (reply)
    dbus_message_unref(reply);

Martin




More information about the Freeipa-devel mailing list