diff options
author | David Tardon <dtardon@redhat.com> | 2022-08-09 14:35:15 +0200 |
---|---|---|
committer | David Tardon <dtardon@redhat.com> | 2022-09-13 08:13:27 +0200 |
commit | 995340074e554b4bf4a0fdb0cb7436692c5a4ffd (patch) | |
tree | ca3ee001db3e608ee80478f42fea9c3b409f1b7b /src/core/dbus.c | |
parent | 6d64cb0625691e2b9eda8babe07ac8281f9467ee (diff) | |
download | systemd-995340074e554b4bf4a0fdb0cb7436692c5a4ffd.tar.gz |
tree-wide: use ASSERT_PTR more
Diffstat (limited to 'src/core/dbus.c')
-rw-r--r-- | src/core/dbus.c | 30 |
1 files changed, 10 insertions, 20 deletions
diff --git a/src/core/dbus.c b/src/core/dbus.c index ad2230d1b5..141c3ffe12 100644 --- a/src/core/dbus.c +++ b/src/core/dbus.c @@ -97,13 +97,12 @@ int bus_forward_agent_released(Manager *m, const char *path) { static int signal_agent_released(sd_bus_message *message, void *userdata, sd_bus_error *error) { _cleanup_(sd_bus_creds_unrefp) sd_bus_creds *creds = NULL; - Manager *m = userdata; + Manager *m = ASSERT_PTR(userdata); const char *cgroup; uid_t sender_uid; int r; assert(message); - assert(m); /* only accept org.freedesktop.systemd1.Agent from UID=0 */ r = sd_bus_query_sender_creds(message, SD_BUS_CREDS_EUID, &creds); @@ -126,11 +125,10 @@ static int signal_agent_released(sd_bus_message *message, void *userdata, sd_bus } static int signal_disconnected(sd_bus_message *message, void *userdata, sd_bus_error *error) { - Manager *m = userdata; + Manager *m = ASSERT_PTR(userdata); sd_bus *bus; assert(message); - assert(m); assert_se(bus = sd_bus_message_get_bus(message)); if (bus == m->api_bus) @@ -149,13 +147,12 @@ static int signal_disconnected(sd_bus_message *message, void *userdata, sd_bus_e static int signal_activation_request(sd_bus_message *message, void *userdata, sd_bus_error *ret_error) { _cleanup_(sd_bus_error_free) sd_bus_error error = SD_BUS_ERROR_NULL; _cleanup_(sd_bus_message_unrefp) sd_bus_message *reply = NULL; - Manager *m = userdata; + Manager *m = ASSERT_PTR(userdata); const char *name; Unit *u; int r; assert(message); - assert(m); r = sd_bus_message_read(message, "s", &name); if (r < 0) { @@ -315,19 +312,18 @@ static int find_unit(Manager *m, sd_bus *bus, const char *path, Unit **unit, sd_ } static int bus_unit_find(sd_bus *bus, const char *path, const char *interface, void *userdata, void **found, sd_bus_error *error) { - Manager *m = userdata; + Manager *m = ASSERT_PTR(userdata); assert(bus); assert(path); assert(interface); assert(found); - assert(m); return find_unit(m, bus, path, (Unit**) found, error); } static int bus_unit_interface_find(sd_bus *bus, const char *path, const char *interface, void *userdata, void **found, sd_bus_error *error) { - Manager *m = userdata; + Manager *m = ASSERT_PTR(userdata); Unit *u; int r; @@ -335,7 +331,6 @@ static int bus_unit_interface_find(sd_bus *bus, const char *path, const char *in assert(path); assert(interface); assert(found); - assert(m); r = find_unit(m, bus, path, &u, error); if (r <= 0) @@ -349,7 +344,7 @@ static int bus_unit_interface_find(sd_bus *bus, const char *path, const char *in } static int bus_unit_cgroup_find(sd_bus *bus, const char *path, const char *interface, void *userdata, void **found, sd_bus_error *error) { - Manager *m = userdata; + Manager *m = ASSERT_PTR(userdata); Unit *u; int r; @@ -357,7 +352,6 @@ static int bus_unit_cgroup_find(sd_bus *bus, const char *path, const char *inter assert(path); assert(interface); assert(found); - assert(m); r = find_unit(m, bus, path, &u, error); if (r <= 0) @@ -374,7 +368,7 @@ static int bus_unit_cgroup_find(sd_bus *bus, const char *path, const char *inter } static int bus_cgroup_context_find(sd_bus *bus, const char *path, const char *interface, void *userdata, void **found, sd_bus_error *error) { - Manager *m = userdata; + Manager *m = ASSERT_PTR(userdata); CGroupContext *c; Unit *u; int r; @@ -383,7 +377,6 @@ static int bus_cgroup_context_find(sd_bus *bus, const char *path, const char *in assert(path); assert(interface); assert(found); - assert(m); r = find_unit(m, bus, path, &u, error); if (r <= 0) @@ -401,7 +394,7 @@ static int bus_cgroup_context_find(sd_bus *bus, const char *path, const char *in } static int bus_exec_context_find(sd_bus *bus, const char *path, const char *interface, void *userdata, void **found, sd_bus_error *error) { - Manager *m = userdata; + Manager *m = ASSERT_PTR(userdata); ExecContext *c; Unit *u; int r; @@ -410,7 +403,6 @@ static int bus_exec_context_find(sd_bus *bus, const char *path, const char *inte assert(path); assert(interface); assert(found); - assert(m); r = find_unit(m, bus, path, &u, error); if (r <= 0) @@ -428,7 +420,7 @@ static int bus_exec_context_find(sd_bus *bus, const char *path, const char *inte } static int bus_kill_context_find(sd_bus *bus, const char *path, const char *interface, void *userdata, void **found, sd_bus_error *error) { - Manager *m = userdata; + Manager *m = ASSERT_PTR(userdata); KillContext *c; Unit *u; int r; @@ -437,7 +429,6 @@ static int bus_kill_context_find(sd_bus *bus, const char *path, const char *inte assert(path); assert(interface); assert(found); - assert(m); r = find_unit(m, bus, path, &u, error); if (r <= 0) @@ -659,12 +650,11 @@ static int bus_setup_disconnected_match(Manager *m, sd_bus *bus) { static int bus_on_connection(sd_event_source *s, int fd, uint32_t revents, void *userdata) { _cleanup_(sd_bus_close_unrefp) sd_bus *bus = NULL; _cleanup_close_ int nfd = -1; - Manager *m = userdata; + Manager *m = ASSERT_PTR(userdata); sd_id128_t id; int r; assert(s); - assert(m); nfd = accept4(fd, NULL, NULL, SOCK_NONBLOCK|SOCK_CLOEXEC); if (nfd < 0) { |