[squid-users] Re: freeAllContexts trashes hostname?

From: Alex Rousskov <rousskov_at_measurement-factory.com>
Date: Fri, 09 Jan 2009 08:21:54 -0700

On Fri, 2009-01-09 at 01:01 -0800, David Molnar wrote:

> I noticed that the host name was being passed around as a pointer in the
> DynamicSslCert code, and I had a hunch that perhaps the buffer was being
> free'd at some point. I found a call to freeAllContexts() in
> ConnStateData::switchToHttps in client_side.cc.

My bug. I did not realize freeAllContexts() frees the request. The fix,
similar to what you have described, has been committed to the
DynamicSslCert branch (r9342).

Your debugging and analysis made the fix trivial. Do you mind if I add
you to the Squid CONTRIBUTORS file?

Thank you,

Alex.
Received on Fri Jan 09 2009 - 15:22:04 MST

This archive was generated by hypermail 2.2.0 : Fri Jan 09 2009 - 12:00:02 MST