diff options
| author | Nathan Bossart <nathan@postgresql.org> | 2024-11-08 16:11:08 -0600 |
|---|---|---|
| committer | Nathan Bossart <nathan@postgresql.org> | 2024-11-08 16:11:08 -0600 |
| commit | 0fa688406595091c4fb6d65f2d409366cc11394e (patch) | |
| tree | f087901bcd5e58feb9e0a0a9e8db6b428435acaa /src | |
| parent | caca6d8d276ce57473e97050d19dfe84e59482c8 (diff) | |
| download | postgresql-0fa688406595091c4fb6d65f2d409366cc11394e.tar.gz postgresql-0fa688406595091c4fb6d65f2d409366cc11394e.zip | |
Fix sign-compare warnings in pg_iovec.h.
The code in question (pg_preadv() and pg_pwritev()) has been around
for a while, but commit 15c9ac3629 moved it to a header file. If
third-party code that includes this header file is built with
-Wsign-compare on a system without preadv() or pwritev(), warnings
ensue. This commit fixes said warnings by casting the result of
pg_pread()/pg_pwrite() to size_t, which should be safe because we
will have already checked for a negative value.
Author: Wolfgang Walther
Discussion: https://postgr.es/m/16989737-1aa8-48fd-8dfe-b7ada06509ab%40technowledgy.de
Backpatch-through: 17
Diffstat (limited to 'src')
| -rw-r--r-- | src/include/port/pg_iovec.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/include/port/pg_iovec.h b/src/include/port/pg_iovec.h index 7255c1bd911..e5fe677b371 100644 --- a/src/include/port/pg_iovec.h +++ b/src/include/port/pg_iovec.h @@ -68,7 +68,7 @@ pg_preadv(int fd, const struct iovec *iov, int iovcnt, off_t offset) } sum += part; offset += part; - if (part < iov[i].iov_len) + if ((size_t) part < iov[i].iov_len) return sum; } return sum; @@ -107,7 +107,7 @@ pg_pwritev(int fd, const struct iovec *iov, int iovcnt, off_t offset) } sum += part; offset += part; - if (part < iov[i].iov_len) + if ((size_t) part < iov[i].iov_len) return sum; } return sum; |