utun2

1287 Commits

15 Branches

2 Tags

168 MiB

Tree: 14d1673f06

Commit Graph

Author	SHA1	Message	Date
Evgeny	0b5a710fe6	etcp_connect + BGP_READY callbacks + router reconnect test - etcp_connect.c: async connect API with EARLY/LATE/BGP_READY callbacks - etcp_connect.h: declaration (via etcp_api.h) - connect_deliver: fix to preserve unfired callback nodes - etcp_set_routing_exchange_state: fires bgp_ready_cbk on TABLE_COMPLETE - route_bgp: TABLE_COMPLETE handler + route_bgp_send_table_complete - etcp.c: connections_count tracking, cleanup on close - test_etcp_router_reconnect: a-b-c topology with b restart (etcp_connect) - test_etcp_connect: unit tests for async connect API - etcp_router: inflight/retrans/close improvements - Various proxy/NAT/router cleanups	3 months ago
Evgeny	53d41d5c62	fix: use-after-free in route table — add route_delete before u_free in route_bgp_update_my_nodeinfo and route_bgp_process_nodeinfo, relax routes_overlap to allow hierarchical routes Root cause: when bgp->local_node was reallocated (changed=1), old route entries remained with stale v_node_info pointing to freed memory. The freed memory got reused by a BGP peer's NODEINFO_Q, causing route_lookup to return wrong node_id (b3b2b1b0afaeadac instead of local), which then failed at etcp_route_send with 'no BGP route' and dropped packets to 10.23.2.1. Changes: - route_node.c: route_delete(old) before u_free + route_insert(new) after alloc - utun_instance.c: remove duplicate route_insert (now atomic inside function) - route_bgp.c: route_delete before freeing old peer NODEINFO_Q - route_lib.c: routes_overlap now rejects only exact prefix+network duplicates - tests/Makefile.am: add route_lib.o to test_route6_lib	3 months ago

Author

SHA1

Message

Date

Evgeny

0b5a710fe6

etcp_connect + BGP_READY callbacks + router reconnect test

- etcp_connect.c: async connect API with EARLY/LATE/BGP_READY callbacks
- etcp_connect.h: declaration (via etcp_api.h)
- connect_deliver: fix to preserve unfired callback nodes
- etcp_set_routing_exchange_state: fires bgp_ready_cbk on TABLE_COMPLETE
- route_bgp: TABLE_COMPLETE handler + route_bgp_send_table_complete
- etcp.c: connections_count tracking, cleanup on close
- test_etcp_router_reconnect: a-b-c topology with b restart (etcp_connect)
- test_etcp_connect: unit tests for async connect API
- etcp_router: inflight/retrans/close improvements
- Various proxy/NAT/router cleanups

3 months ago

Evgeny

53d41d5c62

fix: use-after-free in route table — add route_delete before u_free in route_bgp_update_my_nodeinfo and route_bgp_process_nodeinfo, relax routes_overlap to allow hierarchical routes

Root cause: when bgp->local_node was reallocated (changed=1), old route
entries remained with stale v_node_info pointing to freed memory. The freed
memory got reused by a BGP peer's NODEINFO_Q, causing route_lookup to return
wrong node_id (b3b2b1b0afaeadac instead of local), which then failed at
etcp_route_send with 'no BGP route' and dropped packets to 10.23.2.1.

Changes:
- route_node.c: route_delete(old) before u_free + route_insert(new) after alloc
- utun_instance.c: remove duplicate route_insert (now atomic inside function)
- route_bgp.c: route_delete before freeing old peer NODEINFO_Q
- route_lib.c: routes_overlap now rejects only exact prefix+network duplicates
- tests/Makefile.am: add route_lib.o to test_route6_lib

3 months ago

2 Commits (14d1673f06ef89bb2a92dcc19665253a8e766e68)