NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit8b13e5c

committed

Fix some bogus direct uses of realloc().

pg_dump/parallel.c was using realloc() directly with no error check.While the odds of an actual failure here seem pretty low, Coveritycomplains about it, so fix by using pg_realloc() instead.While looking for other instances, I noticed a couple of places inpsql that hadn't gotten the memo about the availability of pg_realloc.These aren't bugs, since they did have error checks, but verboselyinconsistent code is not a good thing.Back-patch as far as 9.3. 9.2 did not have pg_dump/parallel.c, nordid it have pg_realloc available in all frontend code.

1 parent606c012 commit8b13e5cCopy full SHA for 8b13e5c

File tree

3 files changed

-15

lines changed

src/bin
- pg_dump
  - parallel.c
- psql
  - command.c
  - tab-complete.c

3 files changed

-15

lines changed

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

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1308,15 +1308,15 @@ readMessageFromPipe(int fd)`
`1308`	`1308`	`{`
`1309`	`1309`	`/* could be any number */`
`1310`	`1310`	`bufsize+=16;`
`1311`		`-msg= (char*)realloc(msg,bufsize);`
	`1311`	`+msg= (char*)pg_realloc(msg,bufsize);`
`1312`	`1312`	`}`
`1313`	`1313`	`}`
`1314`	`1314`
`1315`	`1315`	`/*`
`1316`	`1316`	`* Worker has closed the connection, make sure to clean up before return`
`1317`	`1317`	`* since we are not returning msg (but did allocate it).`
`1318`	`1318`	`*/`
`1319`		`-free(msg);`
	`1319`	`+pg_free(msg);`
`1320`	`1320`
`1321`	`1321`	`returnNULL;`
`1322`	`1322`	`}`

`‎src/bin/psql/command.c`

Lines changed: 1 addition & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -1133,12 +1133,7 @@ exec_command(const char *cmd,`
`1133`	`1133`	`while ((opt=psql_scan_slash_option(scan_state,`
`1134`	`1134`	`OT_NORMAL,NULL, false)))`
`1135`	`1135`	`{`
`1136`		`-newval=realloc(newval,strlen(newval)+strlen(opt)+1);`
`1137`		`-if (!newval)`
`1138`		`-{`
`1139`		`-psql_error("out of memory\n");`
`1140`		`-exit(EXIT_FAILURE);`
`1141`		`-}`
	`1136`	`+newval=pg_realloc(newval,strlen(newval)+strlen(opt)+1);`
`1142`	`1137`	`strcat(newval,opt);`
`1143`	`1138`	`free(opt);`
`1144`	`1139`	`}`

`‎src/bin/psql/tab-complete.c`

Lines changed: 2 additions & 7 deletions

Original file line number	Diff line number	Diff line change
`@@ -4331,13 +4331,8 @@ append_variable_names(char **varnames, int nvars,`
`4331`	`4331`	`if (nvars >=maxvars)`
`4332`	`4332`	`{`
`4333`	`4333`	`maxvars =2;`
`4334`		`-varnames= (char)realloc(varnames,`
`4335`		`- ((maxvars)+1)sizeof(char*));`
`4336`		`-if (!(*varnames))`
`4337`		`-{`
`4338`		`-psql_error("out of memory\n");`
`4339`		`-exit(EXIT_FAILURE);`
`4340`		`-}`
	`4334`	`+varnames= (char)pg_realloc(varnames,`
	`4335`	`+ ((maxvars)+1)sizeof(char*));`
`4341`	`4336`	`}`
`4342`	`4337`
`4343`	`4338`	`(varnames)[(nvars)++]=psprintf("%s%s%s",prefix,varname,suffix);`

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!

Commit8b13e5c

File tree

3 files changed

3 files changed

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

`‎src/bin/psql/command.c`

`‎src/bin/psql/tab-complete.c`

0 commit comments