Skip to content

process: Adds logs to the timeout timers that terminate processes#5367

Open
g-ortuno wants to merge 3 commits into
masterfrom
new-process-logs
Open

process: Adds logs to the timeout timers that terminate processes#5367
g-ortuno wants to merge 3 commits into
masterfrom
new-process-logs

Conversation

@g-ortuno

Copy link
Copy Markdown
Collaborator

In https://crbug.com/529865766, we are seeing that corpus files are being deleted after the fuzzing round finishes and during corpus merging. We suspect the FuzzTest wrapper is being sent a SIGTERM signal which doesn't properly propagate to the underlying binary, which continue to fuzz.

This PR changes new_process.py to log when we send SIGTERM and SIGKILL after a timeout.

@g-ortuno
g-ortuno marked this pull request as ready for review July 15, 2026 21:03
@g-ortuno
g-ortuno requested a review from a team as a code owner July 15, 2026 21:03
@g-ortuno
g-ortuno requested a review from dylanjew July 15, 2026 21:03
@g-ortuno

Copy link
Copy Markdown
Collaborator Author

dylanj: PTAL!

Comment thread src/clusterfuzz/_internal/system/new_process.py
@g-ortuno
g-ortuno requested a review from ViniciustCosta July 17, 2026 20:42
@g-ortuno

Copy link
Copy Markdown
Collaborator Author

@ViniciustCosta PTAL! I tested on dev and I can see the logs and there are no issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants