diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2014-03-19 11:09:24 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2014-03-19 11:09:59 -0400 |
| commit | 473194c09cb294151231374d2628ce624da346ba (patch) | |
| tree | 96061523c7ccb3a28d06c6d36a31f83b21149fec /src | |
| parent | f07692e1906c5f78d4dc4f777ade31cbfdb0069e (diff) | |
| download | postgresql-473194c09cb294151231374d2628ce624da346ba.tar.gz postgresql-473194c09cb294151231374d2628ce624da346ba.zip | |
Fix memory leak during regular expression execution.
For a regex containing backrefs, pg_regexec() might fail to free all the
sub-DFAs that were created during execution, resulting in a permanent
(session lifespan) memory leak. Problem was introduced by me in commit
587359479acbbdc95c8e37da40707e37097423f5. Per report from Sandro Santilli;
diagnosis by Greg Stark.
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/regex/regexec.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/backend/regex/regexec.c b/src/backend/regex/regexec.c index 0edb83c1099..2e976627f52 100644 --- a/src/backend/regex/regexec.c +++ b/src/backend/regex/regexec.c @@ -259,6 +259,7 @@ pg_regexec(regex_t *re, /* clean up */ if (v->pmatch != pmatch && v->pmatch != mat) FREE(v->pmatch); + n = (size_t) v->g->ntree; for (i = 0; i < n; i++) { if (v->subdfas[i] != NULL) |