git.postgresql.org Git - postgresql.git/commit

git projects / postgresql.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: b1aebbb) | patch
Another round of Coverity fixes
Mon, 3 Mar 2014 08:18:51 +0000 (03:18 -0500)
Mon, 3 Mar 2014 08:18:51 +0000 (03:18 -0500)
commit 5592ebac55460866da867df5c783c34e3c9a7cae
Another round of Coverity fixes

Additional non-security issues/improvements spotted by Coverity.

In backend/libpq, no sense trying to protect against port->hba being
NULL after we've already dereferenced it in the switch() statement.

Prevent against possible overflow due to 32bit arithmitic in
basebackup throttling (not yet released, so no security concern).

Remove nonsensical check of array pointer against NULL in procarray.c,
looks to be a holdover from 9.1 and earlier when there were pointers
being used but now it's just an array.

Remove pointer check-against-NULL in tsearch/spell.c as we had already
dereferenced it above (in the strcmp()).

Remove dead code from adt/orderedsetaggs.c, isnull is checked
immediately after each tuplesort_getdatum() call and if true we return,
so no point checking it again down at the bottom.

Remove recently added minor error-condition memory leak in pg_regress.
src/backend/libpq/auth.c diff | blob | blame | history
src/backend/replication/basebackup.c diff | blob | blame | history
src/backend/storage/ipc/procarray.c diff | blob | blame | history
src/backend/tsearch/spell.c diff | blob | blame | history
src/backend/utils/adt/orderedsetaggs.c diff | blob | blame | history
src/test/regress/pg_regress.c diff | blob | blame | history
This is the main PostgreSQL git repository.
RSS Atom

AltStyle によって変換されたページ (->オリジナル) /