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.

This commit is contained in:
Andrew Wood
2026-05-10 20:17:11 +01:00
parent f6b2196369
commit 6772d7e5a1
+5 -4
View File
@@ -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;
}
}