[squid-dev] [PATCH] Do not die silently when dying via std::terminate()

Alex Rousskov rousskov at measurement-factory.com
Fri Jun 2 00:38:09 UTC 2017


On 05/17/2017 06:20 PM, Alex Rousskov wrote:
> Hello,
> 
>     While triaging a silent crash of a production Squid, I realized that
> our death reporting code has a serious (and recently grown) hole. The
> attached patch closes it. This reporting-only patch does not fix any
> crashes. There are more technical details in the preamble.
> 
> Researching this problem gave me an idea on how we can preserve Squid
> stack when dying from unhandled runtime exceptions. This patch does not
> implement that idea but adds a corresponding XXX and TODO comments.
> 
> This patch changes the same SquidMainSafe() code as the pending atomic
> PID patch does. While the changes conflict at the patch level, they are
> fully compatible at the "logic level". The conflict is easy to resolve.

I saw no reviews and will commit these changes soon unless there are
last-minute objections.

Alex.


More information about the squid-dev mailing list