From d2132d3d8d68e4f5dbe935c8af7a50eb124bcb8e Mon Sep 17 00:00:00 2001 From: Yu Watanabe Date: Sat, 6 May 2023 14:11:08 +0900 Subject: 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. --- src/core/main.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'src/core') diff --git a/src/core/main.c b/src/core/main.c index f36f6c4c9e..5274bcc106 100644 --- a/src/core/main.c +++ b/src/core/main.c @@ -1003,10 +1003,8 @@ static int parse_argv(int argc, char *argv[]) { FILE *f; fd = parse_fd(optarg); - if (fd == -ERANGE) - return log_error_errno(fd, "Invalid serialization fd: %s", optarg); if (fd < 0) - return log_error_errno(fd, "Failed to parse deserialize option \"%s\": %m", optarg); + return log_error_errno(fd, "Failed to parse serialization fd \"%s\": %m", optarg); (void) fd_cloexec(fd, true); -- cgit v1.2.1