https://gitlab.synchro.net/main/sbbs/-/commit/76e66e3390ab1570e1ebb86e
Modified Files:
src/ssh/TODO.md src/ssh/test/test_auth.c test_conn.c test_transport.c Log Message:
Add DEBUG/GLOBAL_REQUEST/banner/get_methods/OPEN_CONFIRMATION edge case tests

Transport tests:
- debug/msg_len_exceeds_payload: DEBUG with msg_len > actual data
(covers msg_len clamp to 0 on line 754)
- global_request/name_exceeds: GLOBAL_REQUEST with name_len > payload
(covers early break on line 781)

Auth tests:
- banner_truncated: three BANNER variants sent from server before
auth response — no msg_len header, msg_len > payload, valid msg
with truncated lang (covers lines 18, 22, 33-34)
- get_methods_none_accepted: server accepts "none" auth, client
get_methods receives SUCCESS with empty methods (covers lines 567-570)

Conn tests:
- truncated_open_confirmation: OPEN_CONFIRMATION < 17 bytes (line 641)
- open_conf_unknown_channel: OPEN_CONFIRMATION for nonexistent channel (line 646)
- channel_success_no_request: CHANNEL_SUCCESS/FAILURE when no request
pending (exercises the break path at line 612)

TODO.md: add bugs 5 (void* banner_cb), 6 (dead x11 type_len==2 check),
7 (KEX pubkey error check guards — already fixed).

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

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