pgsql: Fix low-probability memory leak in regex execution. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix low-probability memory leak in regex execution.
Date
Msg-id E1Zczsu-0005NJ-NT@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix low-probability memory leak in regex execution.

After an internal failure in shortest() or longest() while pinning down the
exact location of a match, find() forgot to free the DFA structure before
returning.  This is pretty unlikely to occur, since we just successfully
ran the "search" variant of the DFA; but it could happen, and it would
result in a session-lifespan memory leak since this code uses malloc()
directly.  Problem seems to have been aboriginal in Spencer's library,
so back-patch all the way.

In passing, correct a thinko in a comment I added awhile back about the
meaning of the "ntree" field.

I happened across these issues while comparing our code to Tcl's version
of the library.

Branch
------
REL9_0_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/ba9eb293fa6466e5dd8dbcf1e51f8126d9d02bb0

Modified Files
--------------
src/backend/regex/regexec.c |    6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)


pgsql-committers by date:

Previous
From: Teodor Sigaev
Date:
Subject: Re: pgsql: Fix oversight in tsearch type check
Next
From: Tom Lane
Date:
Subject: pgsql: Fix low-probability memory leak in regex execution.