]> granicus.if.org Git - postgresql/commit
Fix some bogus direct uses of realloc().
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 18 Nov 2014 18:28:09 +0000 (13:28 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 18 Nov 2014 18:28:09 +0000 (13:28 -0500)
commit3aa3ae8e1d959681d3396fe176cd8086e3b26c55
tree3fc56c9b065d997f67a2bda50b673ac55a8c8f23
parentd5bea1fbcc24a9880ddc2ede29a6fedb41de9292
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, Coverity
complains about it, so fix by using pg_realloc() instead.

While looking for other instances, I noticed a couple of places in
psql that hadn't gotten the memo about the availability of pg_realloc.
These aren't bugs, since they did have error checks, but verbosely
inconsistent code is not a good thing.

Back-patch as far as 9.3.  9.2 did not have pg_dump/parallel.c, nor
did it have pg_realloc available in all frontend code.
src/bin/pg_dump/parallel.c
src/bin/psql/command.c
src/bin/psql/tab-complete.c