]> granicus.if.org Git - postgresql/commit
Fix misplaced right paren bugs in pgstatfuncs.c.
authorKevin Grittner <kgrittn@postgresql.org>
Fri, 27 Dec 2013 21:40:51 +0000 (15:40 -0600)
committerKevin Grittner <kgrittn@postgresql.org>
Fri, 27 Dec 2013 21:40:51 +0000 (15:40 -0600)
commit28b60aa231c1f5ba89495da4f84564ebd9faeeb4
tree930eac63c51da17de83a489280978b369a019409
parent663f8419b6e75b6ebc4c22d10fbe55f133f768e0
Fix misplaced right paren bugs in pgstatfuncs.c.

The bug would only show up if the C sockaddr structure contained
zero in the first byte for a valid address; otherwise it would
fail to fail, which is probably why it went unnoticed for so long.

Patch submitted by Joel Jacobson after seeing an article by Andrey
Karpov in which he reports finding this through static code
analysis using PVS-Studio.  While I was at it I moved a definition
of a local variable referenced in the buggy code to a more local
context.

Backpatch to all supported branches.
src/backend/utils/adt/pgstatfuncs.c