diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2015-09-18 13:55:17 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2015-09-18 13:55:17 -0400 |
| commit | a39331fa573fc2bd6f93322ff190da26ddc477b5 (patch) | |
| tree | 69e65d823bcf5d7767694672d3fc46db78dacd41 /src/include | |
| parent | e8e2999470bd8148e8caf2c86a24b7a6fd4085f1 (diff) | |
| download | postgresql-a39331fa573fc2bd6f93322ff190da26ddc477b5.tar.gz postgresql-a39331fa573fc2bd6f93322ff190da26ddc477b5.zip | |
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.
Diffstat (limited to 'src/include')
| -rw-r--r-- | src/include/regex/regguts.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/include/regex/regguts.h b/src/include/regex/regguts.h index 2f3be1aa0ae..dac04033119 100644 --- a/src/include/regex/regguts.h +++ b/src/include/regex/regguts.h @@ -470,7 +470,7 @@ struct guts size_t nsub; /* copy of re_nsub */ struct subre *tree; struct cnfa search; /* for fast preliminary search */ - int ntree; /* number of subre's, less one */ + int ntree; /* number of subre's, plus one */ struct colormap cmap; int FUNCPTR(compare, (const chr *, const chr *, size_t)); struct subre *lacons; /* lookahead-constraint vector */ |