From 583e0dfa7479be45b8a5ab7a53c9de88139683e4 Mon Sep 17 00:00:00 2001 From: Jens Axboe Date: Sun, 8 Apr 2018 15:46:17 -0600 Subject: [PATCH] client: fix bad shadowing of 'ret' Different scopes for ret, introduce 'wrote' to check the return value of write(2). Signed-off-by: Jens Axboe --- client.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/client.c b/client.c index 970974a0..ea1a4d29 100644 --- a/client.c +++ b/client.c @@ -1339,7 +1339,7 @@ static int fio_client_handle_iolog(struct fio_client *client, sprintf(log_pathname, "%s.%s", pdu->name, client->hostname); if (store_direct) { - ssize_t ret; + ssize_t wrote; size_t sz; int fd; @@ -1353,10 +1353,10 @@ static int fio_client_handle_iolog(struct fio_client *client, } sz = cmd->pdu_len - sizeof(*pdu); - ret = write(fd, pdu->samples, sz); + wrote = write(fd, pdu->samples, sz); close(fd); - if (ret != sz) { + if (wrote != sz) { log_err("fio: short write on compressed log\n"); ret = 1; goto out; -- 2.25.1