Fix one-byte buffer overrun in contrib/test_parser.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 10 Jan 2012 00:56:27 +0000 (19:56 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 10 Jan 2012 00:57:21 +0000 (19:57 -0500)
commitd427e75e51d9e7d76fc2a4bc142ba7be852a1078
treeeb5ed273d8b0df0635b4f60b3a87b160a27f7441
parent068e08eebbb2204f525647daad3fe15063b77820
Fix one-byte buffer overrun in contrib/test_parser.

The original coding examined the next character before verifying that
there *is* a next character.  In the worst case with the input buffer
right up against the end of memory, this would result in a segfault.

Problem spotted by Paul Guyot; this commit extends his patch to fix an
additional case.  In addition, make the code a tad more readable by not
overloading the usage of *tlen.
contrib/test_parser/test_parser.c