client: check return of dup(2)
authorJens Axboe <axboe@kernel.dk>
Fri, 15 Jun 2018 15:10:48 +0000 (09:10 -0600)
committerJens Axboe <axboe@kernel.dk>
Fri, 15 Jun 2018 15:10:48 +0000 (09:10 -0600)
If dup(2) fails, we can't pass it to fdopen.

Fixes: b4f5e72f1383 ("client: parse env variables before sending job-file contents to server")
Signed-off-by: Jens Axboe <axboe@kernel.dk>
client.c

index 60f7c6a..611ee2f 100644 (file)
--- a/client.c
+++ b/client.c
@@ -123,8 +123,13 @@ static int read_ini_data(int fd, void *data, size_t size)
        char *p = data;
        int ret = 0;
        FILE *fp;
+       int dupfd;
 
-       fp = fdopen(dup(fd), "r");
+       dupfd = dup(fd);
+       if (dupfd < 0)
+               return errno;
+
+       fp = fdopen(dupfd, "r");
        if (!fp)
                return errno;