From 6772d7e5a199f786a9dc9ffb4ccb557292e5b6b8 Mon Sep 17 00:00:00 2001 From: Andrew Wood Date: Sun, 10 May 2026 20:17:11 +0100 Subject: [PATCH] Read the EOF values in the debug statements by dereferencing the eof_in and eof_out pointers, not by using the pointers as a truth value. --- src/pv/transfer.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/pv/transfer.c b/src/pv/transfer.c index 5cd3c9b..d2bffec 100644 --- a/src/pv/transfer.c +++ b/src/pv/transfer.c @@ -1341,8 +1341,8 @@ ssize_t pv_transfer(pvstate_t state, int fd, bool *eof_in, bool *eof_out, off_t if (ready_to_read) { if (!pv__transfer_read(state, fd, eof_in, eof_out, allowed)) { debug("%s %d: %s (%s=%s, %s=%s, %s=%lu)", "fd", fd, - "early return 0 - pv__transfer_read returned false", "eof_in", eof_in ? "true" : "false", - "eof_out", eof_out ? "true" : "false", "allowed", (unsigned long) allowed); + "early return 0 - pv__transfer_read returned false", "eof_in", *eof_in ? "true" : "false", + "eof_out", *eof_out ? "true" : "false", "allowed", (unsigned long) allowed); return 0; } } @@ -1378,8 +1378,9 @@ ssize_t pv_transfer(pvstate_t state, int fd, bool *eof_in, bool *eof_out, off_t && (NULL != lineswritten)) { if (!pv__transfer_write(state, eof_in, eof_out, lineswritten)) { debug("%s %d: %s (%s=%s, %s=%s, %s=%lu)", "fd", fd, - "early return 0 - pv__transfer_write returned false", "eof_in", eof_in ? "true" : "false", - "eof_out", eof_out ? "true" : "false", "lineswritten", (unsigned long) lineswritten); + "early return 0 - pv__transfer_write returned false", "eof_in", + *eof_in ? "true" : "false", "eof_out", *eof_out ? "true" : "false", "lineswritten", + (unsigned long) lineswritten); return 0; } }