[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
dfa.c: fix memory leak
From: |
Aharon Robbins |
Subject: |
dfa.c: fix memory leak |
Date: |
Thu, 04 Dec 2008 23:13:51 +0200 |
Hi. Here is a patch for a bug that I just (re-)fixed in gawk.
It looks like there has not been much activity in the CVS in a long
while. Sigh.
In any case, valgrind is your friend; this showed up in gawk in a UTF-8
locale.
In general, it is worth comparing the gawk dfa.c with the grep one; I
have fixed a few bugs there.
Arnold
--------------------------------------------------
2008-12-04 Arnold D. Robbins <address@hidden>
* src/dfa.c (parse_bracket_exp_mb): Don't set work_mbc->chars
to NULL to not leak the memory allocated a few lines earlier.
Index: dfa.c
===================================================================
RCS file: /sources/grep/grep/src/dfa.c,v
retrieving revision 1.39
diff -u -r1.39 dfa.c
--- dfa.c 10 Feb 2008 03:16:10 -0000 1.39
+++ dfa.c 4 Dec 2008 21:10:13 -0000
@@ -529,7 +529,6 @@
work_mbc->nchars = work_mbc->nranges = work_mbc->nch_classes = 0;
work_mbc->nequivs = work_mbc->ncoll_elems = 0;
- work_mbc->chars = NULL;
work_mbc->ch_classes = NULL;
work_mbc->range_sts = work_mbc->range_ends = NULL;
work_mbc->equivs = work_mbc->coll_elems = NULL;
- dfa.c: fix memory leak,
Aharon Robbins <=