Skip to content

Commit c150d2a

Browse files
committed
compat/pread: check initial lseek for errors
git_pread() saves the current file offset via lseek(fd, 0, SEEK_CUR) and later restores it. If the initial lseek fails (e.g., the fd is a pipe or otherwise non-seekable), current_offset is -1. This negative value is later passed to lseek(fd, -1, SEEK_SET) at line 16, which sets the file position to an unintended location (or fails with EINVAL on some platforms). Check the initial lseek return value and return -1 immediately if it fails, consistent with the error handling for the other lseek calls in the same function. Pointed out by Coverity. Assisted-by: Claude Opus 4.6 Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
1 parent 948e4b3 commit c150d2a

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

compat/pread.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,8 @@ ssize_t git_pread(int fd, void *buf, size_t count, off_t offset)
77
ssize_t rc;
88

99
current_offset = lseek(fd, 0, SEEK_CUR);
10+
if (current_offset < 0)
11+
return -1;
1012

1113
if (lseek(fd, offset, SEEK_SET) < 0)
1214
return -1;

0 commit comments

Comments
 (0)