BBS:      TELESC.NET.BR
Assunto:  src/ssh/TODO.md ssh-auth.c ssh-conn.c src/ssh/test/test_auth.c test_co
De:       Deuc¨
Data:     Fri, 27 Mar 2026 20:57:48 -0700
-----------------------------------------------------------
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) 
n
---
  mSynchronetn  hgVertrauen n hHome of Synchronet n gh[vert/cvs/bbs].synchro.net

-----------------------------------------------------------
[Voltar]