diff options
| author | Yu Watanabe <watanabe.yu+github@gmail.com> | 2023-05-06 14:11:08 +0900 |
|---|---|---|
| committer | Daan De Meyer <daan.j.demeyer@gmail.com> | 2023-05-08 09:49:55 +0200 |
| commit | d2132d3d8d68e4f5dbe935c8af7a50eb124bcb8e (patch) | |
| tree | ec6007448f51b50b356fd2116f298327977dab56 /src/shared/varlink.c | |
| parent | b3d12ac0da2a8b25abc9052a4f350f1ea1108985 (diff) | |
| download | systemd-d2132d3d8d68e4f5dbe935c8af7a50eb124bcb8e.tar.gz | |
parse-util: make parse_fd() return -EBADF
The previous error code -ERANGE is slightly ambiguous, and use more
specific one. This also drops unnecessary error handlings.
Follow-up for 754d8b9c330150fdb3767491e24975f7dfe2a203 and
e652663a043cb80936bb12ad5c87766fc5150c24.
Diffstat (limited to 'src/shared/varlink.c')
| -rw-r--r-- | src/shared/varlink.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/shared/varlink.c b/src/shared/varlink.c index a4936bff26..ab97af57e2 100644 --- a/src/shared/varlink.c +++ b/src/shared/varlink.c @@ -3061,9 +3061,6 @@ int varlink_server_deserialize_one(VarlinkServer *s, const char *value, FDSet *f buf = strndupa_safe(v, n); fd = parse_fd(buf); - if (fd == -ERANGE) - return log_debug_errno(SYNTHETIC_ERRNO(EINVAL), - "VarlinkServerSocket varlink-server-socket-fd= has an invalid value: %s", buf); if (fd < 0) return log_debug_errno(fd, "Unable to parse VarlinkServerSocket varlink-server-socket-fd=%s: %m", buf); if (!fdset_contains(fds, fd)) |