I also fixed it in a patch set to make the resolution pool dynamic :)
Baptiste
Le 04/10/2017 à 11:54, Илья Шипицин a écrit :
Hi,
I'm working on the DNS part to make it thread-safe. In my patch set,
among other things, I fixed this one. I will send everything to
Willy in few days. So don't bother with it.
... using ThreadSanitizer from google ?
No. We
2017-10-04 14:00 GMT+05:00 Christopher Faulet :
> Le 04/10/2017 à 07:49, Илья Шипицин a écrit :
>
>>
>>
>> 2017-10-04 9:15 GMT+05:00 Willy Tarreau >:
>>
>> Hi Ilya,
>>
>> [also CCing Baptiste]
>>
>> On Tue, Oct 03, 2017 at
Le 04/10/2017 à 07:49, Илья Шипицин a écrit :
2017-10-04 9:15 GMT+05:00 Willy Tarreau >:
Hi Ilya,
[also CCing Baptiste]
On Tue, Oct 03, 2017 at 05:25:17PM +0500, ??? wrote:
> [src/dns.c:2502]: (error) Memory leak: buffer
>
>
2017-10-04 9:15 GMT+05:00 Willy Tarreau :
> Hi Ilya,
>
> [also CCing Baptiste]
>
> On Tue, Oct 03, 2017 at 05:25:17PM +0500, ??? wrote:
> > [src/dns.c:2502]: (error) Memory leak: buffer
> >
> >
> > I do not see any "buffer" usage except conditional free.
> > should we just
Hi Ilya,
[also CCing Baptiste]
On Tue, Oct 03, 2017 at 05:25:17PM +0500, ??? wrote:
> [src/dns.c:2502]: (error) Memory leak: buffer
>
>
> I do not see any "buffer" usage except conditional free.
> should we just remove "buffer" from there ?
I think you're referring to this part :
hello!
[src/dns.c:2502]: (error) Memory leak: buffer
I do not see any "buffer" usage except conditional free.
should we just remove "buffer" from there ?
Cheers,
Ilya Shipitsin
7 matches
Mail list logo