]> granicus.if.org Git - postgresql/commit
Fix low-probability memory leak in regex execution.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 18 Sep 2015 17:55:17 +0000 (13:55 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 18 Sep 2015 17:55:17 +0000 (13:55 -0400)
commitd9c0c728afe734b7087dbb9a4bc16d5b682d139d
tree4dfb0df17031d0ae480869b4278a22a4352fbc3d
parentd63a1720fa0b2ca8452c3a2aa2f6ddbb4f05b640
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.
src/backend/regex/regcomp.c
src/backend/regex/regexec.c
src/include/regex/regguts.h