rdelaage/postgresql
1
0
Fork 0
mirror of https://git.postgresql.org/git/postgresql.git synced 2024-10-06 20:26:52 +02:00
Code Issues Packages Projects Releases Wiki Activity

Check for fseeko() failure in pg_dump's _tarAddFile().

Browse Source 
Coverity pointed out, not unreasonably, that we checked fseeko's
result at every other call site but these.  Failure to seek in the
temp file (note this is NOT pg_dump's output file) seems quite
unlikely, and even if it did happen the file length cross-check
further down would probably detect the problem.  Still, that's a
poor excuse for not checking the result of a system call.
This commit is contained in:
Tom Lane 2020-08-09 12:39:08 -04:00
parent 65a6769152
commit 418414daaa
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/bin/pg_dump/pg_backup_tar.c
View File

@ -1085,11 +1085,13 @@ _tarAddFile(ArchiveHandle *AH, TAR_MEMBER *th)
/*
* Find file len & go back to start.
*/
fseeko(tmp, 0, SEEK_END);
if (fseeko(tmp, 0, SEEK_END) != 0)
fatal("error during file seek: %m");
th->fileLen = ftello(tmp);
if (th->fileLen < 0)
fatal("could not determine seek position in archive file: %m");
fseeko(tmp, 0, SEEK_SET);
if (fseeko(tmp, 0, SEEK_SET) != 0)
fatal("error during file seek: %m");
_tarWriteHeader(th);