pgsql: Fix two low-probability memory leaks in regular expression parsi - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix two low-probability memory leaks in regular expression parsi
Date
Msg-id E1X8BXL-000074-L2@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix two low-probability memory leaks in regular expression parsing.

If pg_regcomp failed after having invoked markst/cleanst, it would leak any
"struct subre" nodes it had created.  (We've already detected all regex
syntax errors at that point, so the only likely causes of later failure
would be query cancel or out-of-memory.)  To fix, make sure freesrnode
knows the difference between the pre-cleanst and post-cleanst cleanup
procedures.  Add some documentation of this less-than-obvious point.

Also, newlacon did the wrong thing with an out-of-memory failure from
realloc(), so that the previously allocated array would be leaked.

Both of these are pretty low-probability scenarios, but a bug is a bug,
so patch all the way back.

Per bug #10976 from Arthur O'Dwyer.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/1567e659a877d35ab4b85dafff41b2845d50990f

Modified Files
--------------
src/backend/regex/regcomp.c |   31 ++++++++++++++++++++++++-------
1 file changed, 24 insertions(+), 7 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Fix two low-probability memory leaks in regular expression parsi
Next
From: Tom Lane
Date:
Subject: pgsql: Fix two low-probability memory leaks in regular expression parsi