mirror of
https://git.postgresql.org/git/postgresql.git
synced 2024-09-30 02:41:26 +02:00
b6d8a60aba
Commitscf112c12
anda0dc8271
were a little too hasty in getting rid of the pg_ prefixes where we use pread(), pwrite() and vectored variants. We dropped support for ancient Unixes where we needed to use lseek() to implement replacements for those, but it turns out that Windows also changes the current position even when you pass in an offset to ReadFile() and WriteFile() if the file handle is synchronous, despite its documentation saying otherwise. Switching to asynchronous file handles would fix that, but have other complications. For now let's just put back the pg_ prefix and add some comments to highlight the non-standard side-effect, which we can now describe as Windows-only. Reported-by: Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> Reviewed-by: Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> Discussion: https://postgr.es/m/20220923202439.GA1156054%40nathanxps13
44 lines
812 B
C
44 lines
812 B
C
/*-------------------------------------------------------------------------
|
|
*
|
|
* preadv.c
|
|
* Implementation of preadv(2) for platforms that lack one.
|
|
*
|
|
* Portions Copyright (c) 1996-2022, PostgreSQL Global Development Group
|
|
*
|
|
* IDENTIFICATION
|
|
* src/port/preadv.c
|
|
*
|
|
*-------------------------------------------------------------------------
|
|
*/
|
|
|
|
|
|
#include "c.h"
|
|
|
|
#include <unistd.h>
|
|
|
|
#include "port/pg_iovec.h"
|
|
|
|
ssize_t
|
|
pg_preadv(int fd, const struct iovec *iov, int iovcnt, off_t offset)
|
|
{
|
|
ssize_t sum = 0;
|
|
ssize_t part;
|
|
|
|
for (int i = 0; i < iovcnt; ++i)
|
|
{
|
|
part = pg_pread(fd, iov[i].iov_base, iov[i].iov_len, offset);
|
|
if (part < 0)
|
|
{
|
|
if (i == 0)
|
|
return -1;
|
|
else
|
|
return sum;
|
|
}
|
|
sum += part;
|
|
offset += part;
|
|
if (part < iov[i].iov_len)
|
|
return sum;
|
|
}
|
|
return sum;
|
|
}
|