Memory leaks

Russ Allbery rra at stanford.edu
Wed Jul 4 08:45:19 UTC 2001


Alex Kiernan <alexk at demon.net> writes:

> Woohoo, Purify's not confused, I've found it (I think)... does this
> patch look reasonable to everyone:

Um, yup.  Looks right to me.  Aie.  I'll commit this tomorrow if no one
objects.  It looks like this only applies to CURRENT because STABLE
doesn't include an ARTDATA in a CHANNEL.

> Index: chan.c
> ===================================================================
> RCS file: /upstream-repositories/inn-cvs.isc.org/inn/innd/chan.c,v
> retrieving revision 1.54
> diff -c -r1.54 chan.c
> *** chan.c	2001/05/14 04:05:33	1.54
> --- chan.c	2001/07/04 08:08:39
> ***************
> *** 343,348 ****
> --- 343,358 ----
>   		DISPOSE(cp->Data.Path.List);
>   		cp->Data.Path.List = NULL;
>   	    }
> + 	    if (cp->Data.Overview.Size != 0) {
> + 		DISPOSE(cp->Data.Overview.Data);
> + 		cp->Data.Overview.Data = NULL;
> + 		cp->Data.Overview.Size = 0;
> + 	    }
> + 	    if (cp->Data.XrefBufLength != NULL) {
> + 		DISPOSE(cp->Data.Xref);
> + 		cp->Data.Xref = NULL;
> + 		cp->Data.XrefBufLength = 0;
> + 	    }
>   	} else if (cp->Type == CTreject)
>   	    syslog(L_NOTICE, "%s %ld", name, cp->Rejected);
>   	else if (cp->Out.Left)

-- 
Russ Allbery (rra at stanford.edu)             <http://www.eyrie.org/~eagle/>


More information about the inn-workers mailing list