[PATCH] plug a nasty leak and 3 others like it
Jeff Darcy
jdarcy at redhat.com
Tue Sep 28 17:22:14 UTC 2010
On 09/28/2010 09:17 AM, Jim Meyering wrote:
>
>
>>From 2214d5b9c6adb68baabcaffabc73dd02239096b1 Mon Sep 17 00:00:00 2001
> From: Jim Meyering <meyering at redhat.com>
> Date: Tue, 28 Sep 2010 15:12:33 +0200
> Subject: [PATCH] plug a nasty leak and 3 others like it
>
> * rest.c (proxy_update_prov): Don't leak the state buffer
> allocated in access_handler. I confirmed that this does plug
> an actual leak.
> (proxy_query, proxy_bucket_post, proxy_object_post):
> These are in the same vein, but were fixed by inspection,
> since tests don't yet exercise these.
ACK
> ---
> rest.c | 4 ++++
> 1 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/rest.c b/rest.c
> index 475adc1..a2fffc3 100644
> --- a/rest.c
> +++ b/rest.c
> @@ -825,6 +825,7 @@ proxy_query (void *cctx, struct MHD_Connection *conn, const char *url,
> MHD_add_response_header(resp,"Content-Type","text/xml");
> MHD_queue_response(conn,MHD_HTTP_OK,resp);
> MHD_destroy_response(resp);
> + free_ms(ms);
> }
>
> return MHD_YES;
> @@ -1191,6 +1192,7 @@ proxy_bucket_post (void *cctx, struct MHD_Connection *conn, const char *url,
> }
> MHD_queue_response(conn,rc,resp);
> MHD_destroy_response(resp);
> + free_ms(ms);
> }
>
> return MHD_YES;
> @@ -1308,6 +1310,7 @@ proxy_object_post (void *cctx, struct MHD_Connection *conn, const char *url,
> }
> MHD_queue_response(conn,rc,resp);
> MHD_destroy_response(resp);
> + free_ms(ms);
> }
>
> return MHD_YES;
> @@ -1471,6 +1474,7 @@ proxy_update_prov (void *cctx, struct MHD_Connection *conn, const char *url,
> }
> MHD_queue_response(conn,rc,resp);
> MHD_destroy_response(resp);
> + free_ms(ms);
> }
>
> return MHD_YES;
> --
> 1.7.3.293.gca9a76
More information about the iwhd-devel
mailing list