Re: PATCH: fix memory leak

Alexander Malysh <[email protected]>
Newsgroups gmane.comp.mobile.kannel.devel
Organization Centrium GmbH
Message-ID <[email protected]>
Hi,

seems a right catch for me. commited to cvs...
Thanks a lot!

On Monday 12 January 2004 18:36, Nuno Ferreira wrote:
> Hi,
> attached is a patch that fixes a memory leak in the http module.
> If you're using kannel under cygwin and are seeing handle leaks this
> might be the cause because this was also leaking a mutex for every
> request.
>
> RCS file: /home/cvs/gateway/gwlib/http.c,v
> retrieving revision 1.213
> diff -u -w -b -r1.213 http.c
> --- gwlib/http.c	2 Dec 2003 09:20:51 -0000	1.213
> +++ gwlib/http.c	12 Jan 2004 17:01:31 -0000
> @@ -1011,6 +1011,7 @@
>                      http_status_class(trans->status) ==
> HTTP_STATUS_PROVISIONAL) { /* This was a provisional reply; get the real
> one now. */ trans->state = reading_status;
> +            entity_destroy(trans->response);
>          } else if (ret == 0) {
>              trans->state = transaction_done;
>
> --
> Nuno

-- 
Best regards / Mit besten Grüßen aus Düsseldorf

Dipl.-Ing.
Alexander Malysh
___________________________________________

Centrium GmbH
Vogelsanger Weg 80
40470 Düsseldorf

Fon: +49 (0211) 74 84 51 80
Fax: +49 (0211) 277 49 109

email: a.malysh at centrium.de
web: www.centrium.de
msn: olek2002 at hotmail.com
icq: 98063111
___________________________________________

Please avoid sending me Word, Excel or PowerPoint attachments.
See http://www.fsf.org/philosophy/no-word-attachments.html
signature.asc (application/pgp-signature, 189 B)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)

iD8DBQBAAvmtnX3e5W+uJ0ERAtYVAJ9gX+/V02ycYV/jHaiV6dTL9xYmRwCg5ZBC
/deOHtAiwNm1lFcVGi8wNJ4=
=VSBY
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.