diff options
| author | Aleksandr Nogikh <nogikh@google.com> | 2024-01-05 12:27:42 +0100 |
|---|---|---|
| committer | Aleksandr Nogikh <nogikh@google.com> | 2024-01-09 11:47:02 +0000 |
| commit | 13221c1b2c80cd186bbf226f956cf79b0aa840c3 (patch) | |
| tree | 89d35fb3bd14cc7b34bf860f341fb75774a9c005 /pkg | |
| parent | b96fc6cf3218a736300f6286edfc62d9fa2028f7 (diff) | |
pkg/ipc: kill syz-executor earlier
Even though syz-executor is supposed to kill its forked children after 5
seconds, in practice it fails to do so once in a while.
As a result, we end up waiting 50+ seconds before pkg/ipc kills the
parent syz-executor process. Due to the gate ticketing system, the whole
fuzzing stay stalled in the meanwhile.
Reduce the timeout to 25 seconds.
Diffstat (limited to 'pkg')
| -rw-r--r-- | pkg/ipc/ipc.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/pkg/ipc/ipc.go b/pkg/ipc/ipc.go index 542e75714..0f7746e73 100644 --- a/pkg/ipc/ipc.go +++ b/pkg/ipc/ipc.go @@ -561,7 +561,7 @@ func makeCommand(pid int, bin []string, config *Config, inFile, outFile *os.File // Executor has an internal timeout and protects against most hangs when fork server is enabled, // so we use quite large timeout. Executor can be slow due to global locks in namespaces // and other things, so let's better wait than report false misleading crashes. - timeout *= 10 + timeout *= 5 } c := &command{ |
