Re: b9173 - fd cleanups

From: Amos Jeffries <squid3_at_treenet.co.nz>
Date: Tue, 9 Sep 2008 15:27:52 +1200 (NZST)

> On Tue, 2008-09-09 at 00:37 +1200, Amos Jeffries wrote:
>> http://www.squid-cache.org/Versions/v3/HEAD/changesets/b9173.patch
>>
>> Alex, were there any bugs or potential bugs related to this fix? It
>> rings a small bell in the back of my mind.
>
> Yes, we were leaking timeoutHandler and closeHandler call objects
> (a.k.a., callbacks).
>
> No, I do not think there was any corruption going on but that blind
> memset() in fde::clear() will probably lead to that if not changed or
> replaced.
>

Okay, so no related bugzilla entries you are aware of?

Amos
Received on Tue Sep 09 2008 - 03:27:55 MDT

This archive was generated by hypermail 2.2.0 : Tue Sep 09 2008 - 12:00:04 MDT