https://gitlab.synchro.net/main/sbbs/-/commit/8316e6a9848d490dcb6dc7b9
Modified Files:
src/ssh/TODO.md ssh-auth.c ssh-conn.c src/ssh/test/test_auth.c test_conn.c
Log Message:
Close item 87, add 30 NULL-parameter coverage tests (ssh-conn.c 69%->76%)

Item 87 (shutdown path tolerance) already handled by dssh_thrd_check
wrapper -- terminate atomic prevents recursion, every lock/broadcast in set_terminate() checks its return and skips on failure. Moved to Closed.

Reorder NULL guard || chains in ssh-conn.c (16 functions) and ssh-auth.c
(5 functions): sess == NULL now evaluates last so all branches are
reachable with fake non-NULL sentinels (no handshake needed). Split
parse function NULL checks into separate if statements.

25 new tests in test_conn.c + 5 in test_auth.c covering every branch in
every public API NULL guard. ssh-conn.c branch coverage 69.08% -> 76.01%
(-65 missed branches), overall 79.79% -> 81.49%.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net