[Rpm-maint] [rpm-software-management/rpm] Add dbus-announce plugin (#1255)
Florian Festi
notifications at github.com
Wed May 26 13:35:48 UTC 2021
@ffesti commented on this pull request.
> + if (!state->bus) {
+ return RPMRC_OK;
+ }
+
+ msg = dbus_message_new_signal("/org/rpm/Transaction", /* object name */
+ "org.rpm.Transaction", /* interface name */
+ name); /* signal name */
+ if (NULL == msg) {
+ goto err;
+ }
+
+ dbcookie = rpmdbCookie(rpmtsGetRdb(ts));
+ if (!dbus_message_append_args(msg, DBUS_TYPE_STRING, &dbcookie)) {
+ goto err;
+ }
+ dbcookie = _free(dbcookie);
Plugged the leak. Still not a text book example of the central exit point pattern...
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1255#discussion_r639731889
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20210526/6f56ab56/attachment.html>
More information about the Rpm-maint
mailing list