From a13518231af95e2c771b8c981068e4138ea320e3 Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Thu, 23 Nov 2023 18:23:40 +0100 Subject: [PATCH] varlink: add an extra assert encoding our assumption that ucred is valid here --- src/shared/varlink.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/shared/varlink.c b/src/shared/varlink.c index 3f022e88bd3..52a946f8c9b 100644 --- a/src/shared/varlink.c +++ b/src/shared/varlink.c @@ -3018,6 +3018,8 @@ static int count_connection(VarlinkServer *server, const struct ucred *ucred) { server->n_connections++; if (FLAGS_SET(server->flags, VARLINK_SERVER_ACCOUNT_UID)) { + assert(uid_is_valid(ucred->uid)); + r = hashmap_ensure_allocated(&server->by_uid, NULL); if (r < 0) return varlink_server_log_errno(server, r, "Failed to allocate UID hash table: %m");