lkn
/
ollie
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Mark process cleanup finding resolved

This commit is contained in:
Ollie Agent 2026-08-18 07:09:46 +02:00
parent 1b934c43f4
commit d5def6f02a
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
doc/concurrency-resource-audit.md
View File

@ -292,11 +292,9 @@ Completion notifications now use the toolsrv run context with a ten-second timeo
### M15. `DismissProc` and process GC can lose cleanup ownership
**Evidence:** `cmd/toolsrv/internal/server/proc.go:454-516`.
**Status:** Resolved by the existing process lifecycle safeguards.
Dismissal removes a process from tracking, while GC only removes exited processes after output is read. Abandoned completed processes therefore retain output until an explicit read, and dismissed running processes evade cleanup.
**Remediation:** enforce a process state machine and absolute retention TTL.
`DismissProc` refuses to remove running processes, preserving shutdown visibility. Process GC removes exited processes after an absolute `EndTime` TTL as well as after an old output read, so abandoned completed processes cannot retain output indefinitely.
### M16. Remote and local socket paths are not cleaned on abnormal startup