core: Assert return value of pa_shared_set/_remove() in dbus-shared

it must succeed, or we are leaking memory

Coverity ID: #1380674, #1380673

Signed-off-by: Peter Meerwald-Stadler <pmeerw@pmeerw.net>
This commit is contained in:
Peter Meerwald-Stadler 2017-03-07 15:57:28 +01:00
parent 0ced45265c
commit 976f7e9a61

View file

@ -45,7 +45,7 @@ static pa_dbus_connection* dbus_connection_new(pa_core *c, pa_dbus_wrap_connecti
pconn->property_name = name; pconn->property_name = name;
pconn->connection = conn; pconn->connection = conn;
pa_shared_set(c, name, pconn); pa_assert_se(pa_shared_set(c, name, pconn) >= 0);
return pconn; return pconn;
} }
@ -88,7 +88,7 @@ void pa_dbus_connection_unref(pa_dbus_connection *c) {
pa_dbus_wrap_connection_free(c->connection); pa_dbus_wrap_connection_free(c->connection);
pa_shared_remove(c->core, c->property_name); pa_assert_se(pa_shared_remove(c->core, c->property_name) >= 0);
pa_xfree(c); pa_xfree(c);
} }