Skip to content

Commit

Permalink
keep quarantine files on SIGABRT too
Browse files Browse the repository at this point in the history
AdminModel::cleanupResourceConsumingDocs uses SIGABRT as first
attept to kill misbehaving documents, we should give
DocumentBroker a chance to quarantine documents killed off by
SIGABRT as we do SIGSEGV/SIGBUS

Signed-off-by: Caolán McNamara <[email protected]>
Change-Id: Ic3a703572393050379b5a1444a5380bbeafcf2d3
  • Loading branch information
caolanm authored and vmiklos committed Oct 16, 2023
1 parent 5f9b8ca commit e954898
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion kit/ForKit.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -298,7 +298,9 @@ static void cleanupChildren()
const auto it = childJails.find(exitedChildPid);
if (it != childJails.end())
{
if (WIFSIGNALED(status) && (WTERMSIG(status) == SIGSEGV || WTERMSIG(status) == SIGBUS))
if (WIFSIGNALED(status) && (WTERMSIG(status) == SIGSEGV ||
WTERMSIG(status) == SIGBUS ||
WTERMSIG(status) == SIGABRT))
{
++segFaultCount;

Expand Down

0 comments on commit e954898

Please sign in to comment.