summaryrefslogtreecommitdiff
path: root/src/shared/varlink.c
diff options
context:
space:
mode:
authorLennart Poettering <lennart@poettering.net>2021-10-11 15:25:14 +0200
committerLennart Poettering <lennart@poettering.net>2021-10-11 15:37:37 +0200
commita995ce4768928843b4d37ecf46f32fe9e635e38a (patch)
treed39f5ca7fa86da5bd89f815924a44d3fa117e5c4 /src/shared/varlink.c
parent599be274c13c503806c85073d7beb1a155ac27bd (diff)
downloadsystemd-a995ce4768928843b4d37ecf46f32fe9e635e38a.tar.gz
util: define initializer for 'struct ucred' that properly invalidates all fields
i.e. let's make sure to invalid uid/gid to UID_INVAID + GID_INVALID instead of zero.
Diffstat (limited to 'src/shared/varlink.c')
-rw-r--r--src/shared/varlink.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/src/shared/varlink.c b/src/shared/varlink.c
index 07a1b96f60..984dea1ff5 100644
--- a/src/shared/varlink.c
+++ b/src/shared/varlink.c
@@ -258,8 +258,7 @@ static int varlink_new(Varlink **ret) {
.state = _VARLINK_STATE_INVALID,
- .ucred.uid = UID_INVALID,
- .ucred.gid = GID_INVALID,
+ .ucred = UCRED_INVALID,
.timestamp = USEC_INFINITY,
.timeout = VARLINK_DEFAULT_TIMEOUT_USEC
@@ -2106,8 +2105,8 @@ static int count_connection(VarlinkServer *server, struct ucred *ucred) {
int varlink_server_add_connection(VarlinkServer *server, int fd, Varlink **ret) {
_cleanup_(varlink_unrefp) Varlink *v = NULL;
+ struct ucred ucred = UCRED_INVALID;
bool ucred_acquired;
- struct ucred ucred;
int r;
assert_return(server, -EINVAL);