[squid-dev] [PATCH] Fixed Write.cc:41 "!ccb->active()" assertion
christos at chtsanti.net
Thu Nov 17 10:47:31 UTC 2016
I am also attaching the t4 patch for squid-3.5.
This is include all fixes.
On 11/16/2016 04:43 PM, Christos Tsantilas wrote:
> If no objection I will apply this patch to trunk.
> On 11/16/2016 02:35 PM, Amos Jeffries wrote:
>> On 16/11/2016 12:58 a.m., Christos Tsantilas wrote:
>>> Hi all,
>>> I applied the patch as r14945 with an r14946 fix.
>>> Unfortunately while I was testing the Squid-3.5 variant of the patch I
>>> found a bug:
>>> When the Http::One::Server::writeControlMsgAndCall fails to write the
>>> control message, schedules a Comm::Write callback using just a
>>> ScheduleCallHere command.
>>> The callback called without the CommIoCbParams details and squid is
>>> There are two ways to fix this issue.
>>> 1) complete the missing CommIoCbParams details inside
>>> 2) Make the ConnStateData::writeControlMsgAndCall to return false on
>>> failures and allow caller handle the failure.
>>> This patch implements the (2) as I believe it is the better option.
>>> My apologies for the buggy patch.
>> Now that the method wroteControlMsgOK() is needing to be called in
>> not-OK situations I think it should have a different name.
>> Perhapse doneWithControlMsg() ?
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 16749 bytes
Desc: not available
More information about the squid-dev