[squid-dev] [PATCH] Fixed Write.cc:41 "!ccb->active()" assertion

Christos Tsantilas 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
>>> crashes.
>>> There are two ways to fix this issue.
>>>   1) complete the missing CommIoCbParams details inside
>>> writeControlMsgAndCall
>>>   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...
Name: SQUID-238-assertion_failed_Write_cc_Not_ccb_active-squid3.5-t4.patch
Type: text/x-patch
Size: 16749 bytes
Desc: not available
URL: <http://lists.squid-cache.org/pipermail/squid-dev/attachments/20161117/30b14dde/attachment-0001.bin>

More information about the squid-dev mailing list