evgeny
38c9f229bc
- u_async.c: process immediate_queue BEFORE epoll_wait in uasync_poll - stcp_link.c: defensive NULL etcp_conn/etcp_link in stcp_link_close - utun_instance.c: move stcp_server_list_destroy_all to Phase J (before ETCP), add deferred drain in both Phase J and L - stcp.h/c: STCP_HS_ENC_CLIENT 6→38, STCP_HS_ENC_SERVER 7→39, exchange ed25519_pubkey in both handshake directions - etcp_connections.c: copy peer_ed25519 from STCP to ETCP_CONN in etcp_link_enter_ready_tcp; guard link->etcp in tcp_link_close_cb - chat_sync.c: fix lk->conn NULL dereference for TCP links - test_stcp.c: update stcp_server_create/stcp_client_connect callsites - tools/chat_tcp_test: change transport udp→tcp
25 lines
1.0 KiB
Diff
25 lines
1.0 KiB
Diff
--- test_etcp_simple_traffic.c.backup
|
|
+++ test_etcp_simple_traffic.c
|
|
@@ -243,12 +243,19 @@
|
|
DEBUG_DEBUG(DEBUG_CATEGORY_ETCP, "Parsing crypto keys: priv_len=%zu, pub_len=%zu",
|
|
strlen(priv_key_hex), strlen(pub_key_hex));
|
|
|
|
+ // Validate key string lengths to prevent buffer overflow
|
|
+ if (strlen(priv_key_hex) != 64 || strlen(pub_key_hex) != 128) {
|
|
+ DEBUG_ERROR(DEBUG_CATEGORY_ETCP, "Invalid key string lengths: priv=%zu, pub=%zu",
|
|
+ strlen(priv_key_hex), strlen(pub_key_hex));
|
|
+ return NULL;
|
|
+ }
|
|
+
|
|
// Parse keys
|
|
for (int i = 0; i < 32; i++) {
|
|
if (sscanf(&priv_key_hex[i*2], "%2hhx", &inst->instance->my_keys.private_key[i]) != 1) {
|
|
DEBUG_ERROR(DEBUG_CATEGORY_ETCP, "Failed to parse private key byte %d", i);
|
|
}
|
|
}
|
|
for (int i = 0; i < 64; i++) {
|
|
if (sscanf(&pub_key_hex[i*2], "%2hhx", &inst->instance->my_keys.public_key[i]) != 1) {
|
|
DEBUG_ERROR(DEBUG_CATEGORY_ETCP, "Failed to parse public key byte %d", i);
|
|
}
|
|
}
|