]> granicus.if.org Git - postgresql/commit
Fix misplaced right paren bugs in pgstatfuncs.c.
authorKevin Grittner <kgrittn@postgresql.org>
Fri, 27 Dec 2013 21:26:24 +0000 (15:26 -0600)
committerKevin Grittner <kgrittn@postgresql.org>
Fri, 27 Dec 2013 21:26:24 +0000 (15:26 -0600)
commita133bf7031ad5b62281b21dbd6b2097d3d400f0d
tree6ebbf7aa1532a26c61b27290d6e40c315a18cd29
parenta09e3fd77661e22f5000098e7b0bfcc6751d7c18
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