Re: b9173 - fd cleanups

From: Alex Rousskov <rousskov_at_measurement-factory.com>
Date: Mon, 08 Sep 2008 09:24:31 -0600

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.

HTH,

Alex.
Received on Mon Sep 08 2008 - 15:25:09 MDT

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