]> granicus.if.org Git - postgresql/commit
Add missing error check in regexp parser.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 27 Feb 2013 15:40:26 +0000 (10:40 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 27 Feb 2013 15:40:26 +0000 (10:40 -0500)
commit4aa15467264caf118614827b25db260ec85438ba
treeaee94f9751e160ff6b38eeec3845aa9ff1756dda
parent4fd25a367f24b2bb374393db7c53e4f048ff6bf2
Add missing error check in regexp parser.

parseqatom() failed to check for an error return (NULL result) from its
recursive call to parsebranch(), and in consequence could crash with a
null-pointer dereference after an error return.  This bug has been there
since day one, but wasn't noticed before, probably because most error cases
in parsebranch() didn't actually lead to returning NULL.  Add the missing
error check, and also tweak parsebranch() to exit in a less indirect
fashion after a call to parseqatom() fails.

Report by Tomasz Karlik, fix by me.
src/backend/regex/regcomp.c