Highlights
Information Disclosure: 1 prior fix. Scrutinize any change in this area.
conn.go: most-fixed (3 issues). Treat as high-risk during review.
3 high-severity fixes in this history; regressions here are high-impact.
Recurring patterns
The bug types that recur here, drawn from past fixes, not open vulnerabilities.
Integer Overflow / Out-of-Bounds Read: 32-bit integer overflow in buffer size calculations can lead to out-of-bounds reads, potentially causing memory corruption or information leaks. The fix changed sizes to int64 and added frame header validation.
TLS Negotiation Bypass: Improper TLS negotiation sequence (not pausing the reader and performing handshake before replacing net.Conn) can allow a bypass of TLS protections, enabling man-in-the-middle attacks.
TLS Bypass: Missing ServerName when InsecureSkipVerify is false allows TLS hostname verification to be skipped, enabling MITM attacks. The fix sets ServerName from the hostname.