[debbugs-tracker] bug#17341: closed (memory leak in dfa.c)

[GNU bug Tracking System]

Your message dated Thu, 24 Apr 2014 23:30:36 -0700
with message-id <address@hidden>
and subject line Re: bug#17341: memory leak in dfa.c
has caused the debbugs.gnu.org bug report #17341,
regarding memory leak in dfa.c
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
17341: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=17341
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> memory leak in dfa.c
>
>
>
>
> Date: 
>
> Thu, 24 Apr 2014 22:19:53 +0300
>
>
>
>
> User-agent: 
>
> Heirloom mailx 12.5 6/20/10
>
>
>
>
> Hi.
>
> Thanks to valgrind, I tracked down this memory leak in dfa.c. dfasuperset()
> when it does dfafree(sup) doesn't free(sup).  Here's the fix.
>
> Thanks,
>
> Arnold
> -----------------------
> diff --git a/src/dfa.c b/src/dfa.c
> index 8fc3d6f..9106814 100644
> --- a/src/dfa.c
> +++ b/src/dfa.c
> @@ -3533,7 +3533,10 @@ dfasuperset (struct dfa *d)
>    sup->tindex = j;
>  
>    if ((d->mb_cur_max == 1 && !have_achar) || !have_nchar)
> -    dfafree (sup);
> +    {
> +      dfafree (sup);
> +      free (sup);
> +    }
>    else
>      d->superset = sup;
>  }
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#17341: memory leak in dfa.c
>
>
>
>
> Date: 
>
> Thu, 24 Apr 2014 23:30:36 -0700
>
>
>
>
> User-agent: 
>
> Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0
>
>
>
>
> Thanks, I merged that into the master version.
>
>
> --- End Message ---