BBS:      TELESC.NET.BR
Assunto:  src/ssh/TODO.md src/ssh/test/test_auth.c test_conn.c test_transport.c
De:       Deuc¨
Data:     Tue, 24 Mar 2026 20:58:43 -0700
-----------------------------------------------------------
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) 
n
---
  mSynchronetn  hgVertrauen n hHome of Synchronet n gh[vert/cvs/bbs].synchro.net

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