diff options
author | Thomas Haller <thaller@redhat.com> | 2016-09-02 17:54:36 +0200 |
---|---|---|
committer | Thomas Haller <thaller@redhat.com> | 2016-09-02 17:54:36 +0200 |
commit | 3c8ba23fa43f15968b5fc433b5e98a62f85cc0e4 (patch) | |
tree | 3e8bb2ba60954504df6184fe5691fbf90a6117a3 | |
parent | 58953c5860d404e035a945a05bf3d69b9f71451d (diff) | |
download | NetworkManager-3c8ba23fa43f15968b5fc433b5e98a62f85cc0e4.tar.gz |
fixup! dbus: fix emitting D-Bus NetworkManager's old-style PropertiesChange signalth/dbus-property-changed-source-iface-bgo770629-v2
-rw-r--r-- | src/nm-exported-object.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/nm-exported-object.c b/src/nm-exported-object.c index c55d9b0021..32b5ec05dd 100644 --- a/src/nm-exported-object.c +++ b/src/nm-exported-object.c @@ -943,14 +943,14 @@ vtype_found: ifdata = &priv->interfaces[i]; if ( ifdata->property_changed_signal_id && !NMDBUS_IS_DEVICE_STATISTICS_SKELETON (ifdata->interface)) { - if (j++ > 0) - g_variant_ref (value_variant); + j++; g_hash_table_insert (ifdata->pending_notifies, (gpointer) dbus_property_name, - value_variant); + g_variant_ref (value_variant)); } } nm_assert (j > 0); + g_variant_unref (value_variant); } else if (ifdata->property_changed_signal_id) { /* @dbus_property_name is inside classinfo and never freed, thus we don't clone it. * Also, we do a pointer, not string comparison. */ |