NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitdd63a71

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 parent900429d commitdd63a71Copy full SHA for dd63a71

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
`@@ -1082,11 +1082,13 @@ _tarAddFile(ArchiveHandle AH, TAR_MEMBER th)`
`1082`	`1082`	`/*`
`1083`	`1083`	`* Find file len & go back to start.`
`1084`	`1084`	`*/`
`1085`		`-fseeko(tmp,0,SEEK_END);`
	`1085`	`+if (fseeko(tmp,0,SEEK_END)!=0)`
	`1086`	`+fatal("error during file seek: %m");`
`1086`	`1087`	`th->fileLen=ftello(tmp);`
`1087`	`1088`	`if (th->fileLen<0)`
`1088`	`1089`	`fatal("could not determine seek position in archive file: %m");`
`1089`		`-fseeko(tmp,0,SEEK_SET);`
	`1090`	`+if (fseeko(tmp,0,SEEK_SET)!=0)`
	`1091`	`+fatal("error during file seek: %m");`
`1090`	`1092`
`1091`	`1093`	`_tarWriteHeader(th);`
`1092`	`1094`

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!

Commitdd63a71

File tree

1 file changed

1 file changed

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

0 commit comments