On Tue, 2014-02-04 at 19:03 -0300, jprvita at gmail.com wrote: > +void hf_audio_agent_done(hf_audio_agent_data *data) { > + hf_audio_agent_data *hfdata = data; What's the point of this extra variable? Why not use data directly? > + > + pa_assert(hfdata); > + > + if (hfdata->connection) { > + dbus_connection_unregister_object_path(pa_dbus_connection_get(hfdata->connection), HF_AUDIO_AGENT_PATH); > + > + pa_dbus_connection_unref(hfdata->connection); > + } > + > + pa_xfree(hfdata); > +} -- Tanu