NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit418414d

committed

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

Coverity pointed out, not unreasonably, that we checked fseeko'sresult at every other call site but these. Failure to seek in thetemp file (note this is NOT pg_dump's output file) seems quiteunlikely, and even if it did happen the file length cross-checkfurther down would probably detect the problem. Still, that's apoor excuse for not checking the result of a system call.

1 parent65a6769 commit418414dCopy full SHA for 418414d

File tree

1 file changed

-2

lines changed

src/bin/pg_dump
- pg_backup_tar.c

1 file changed

-2

lines changed

`‎src/bin/pg_dump/pg_backup_tar.c‎`

Lines changed: 4 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1085,11 +1085,13 @@ _tarAddFile(ArchiveHandle AH, TAR_MEMBER th)`
`1085`	`1085`	`/*`
`1086`	`1086`	`* Find file len & go back to start.`
`1087`	`1087`	`*/`
`1088`		`-fseeko(tmp,0,SEEK_END);`
	`1088`	`+if (fseeko(tmp,0,SEEK_END)!=0)`
	`1089`	`+fatal("error during file seek: %m");`
`1089`	`1090`	`th->fileLen=ftello(tmp);`
`1090`	`1091`	`if (th->fileLen<0)`
`1091`	`1092`	`fatal("could not determine seek position in archive file: %m");`
`1092`		`-fseeko(tmp,0,SEEK_SET);`
	`1093`	`+if (fseeko(tmp,0,SEEK_SET)!=0)`
	`1094`	`+fatal("error during file seek: %m");`
`1093`	`1095`
`1094`	`1096`	`_tarWriteHeader(th);`
`1095`	`1097`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit418414d

File tree

1 file changed

1 file changed

`‎src/bin/pg_dump/pg_backup_tar.c‎`

0 commit comments