KarpelesLab / slirp / 23944797218

71%

Build Type

push

github

Committed by MagicalTux

Commit Message

Replace tcpConn/tcpConn6 with vtcp.Conn + NAT bridge

Phase 3.2: Outbound TCP NAT proxy connections now use vtcp.Conn as the
protocol engine, bridged to a real net.Conn via io.Copy goroutines.

New type: tcpNATConn wraps vtcp.Conn (facing virtual client) +
net.Conn (facing real server). Two goroutines copy data bidirectionally.
When either side closes, the other is torn down.

Changes:
- Stack.tcp/tcp6 maps now hold *tcpNATConn
- SYN handling: dials remote outside any lock, creates vtcp.Conn via
  AcceptSYN, starts bridge goroutines
- Subsequent packets: ParseSegment + HandleSegment on the vtcp.Conn
- RST for non-SYN stray packets uses vtcp.Segment.Marshal()
- Maintenance cleanup checks vtcp.Conn.State()
- Stack.Close() calls tcpNATConn.close() which aborts both sides
- Tests updated to create tcpNATConn instances

The old tcpConn/tcpConn6 types remain as dead code — they will be
removed after vclient migration completes.

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

Coverage Stats

93 of 138 new or added lines in 3 files covered. (67.39%)

13 existing lines in 2 files now uncovered.

3369 of 4872 relevant lines covered (69.15%)

8.83 hits per line