Re: [SR-Users] Memory leak in http_query() function (utils module)

2014-02-18 Thread Juha Heinanen
Guillaume Bour writes: > Hi Juha, here it is thanks, i just committed the patch. -- juha ___ SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list sr-users@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-

Re: [SR-Users] Memory leak in http_query() function (utils module)

2014-02-18 Thread Guillaume Bour
On 17/02/2014 20:34, Juha Heinanen wrote: Guillaume Bour writes: Can't you directly apply my patch on 4.0 branch ? i can if you send me all changes in one patch. -- juha Hi Juha, here it is Regards, Guillaume diff --git a/kamailio-4.0.5.1/modules/utils/functions.c b/kamailio-4.0.5.1/modul

Re: [SR-Users] Memory leak in http_query() function (utils module)

2014-02-17 Thread Juha Heinanen
Guillaume Bour writes: > Can't you directly apply my patch on 4.0 branch ? i can if you send me all changes in one patch. -- juha ___ SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list sr-users@lists.sip-router.org http://lists.si

Re: [SR-Users] Memory leak in http_query() function (utils module)

2014-02-17 Thread Guillaume Bour
On 15/02/2014 03:21, Juha Heinanen wrote: Guillaume Bour writes: I attached patches to fix those issues (applies to 4.0 branch) thanks for the patch. since it was for 4.0 i had to manually apply the changes to master and then i did cherry pick to 4.1. cherry pick to 4.0 failed. -- juha Hi

[SR-Users] Memory leak in http_query() function (utils module)

2014-02-14 Thread Juha Heinanen
Guillaume Bour writes: > I attached patches to fix those issues (applies to 4.0 branch) thanks for the patch. since it was for 4.0 i had to manually apply the changes to master and then i did cherry pick to 4.1. cherry pick to 4.0 failed. -- juha __

[SR-Users] Memory leak in http_query() function (utils module)

2014-02-14 Thread Guillaume Bour
Hi folks, We identified a memory leak in *utils* module, when calling the http_query() function. a memory piece is allocated with malloc() to store http request result string (functions.c:54), but this memory is never freed. There is also a possible crash issue, because curl_easy_getinfo() is