GHSA-xw98-5q62-jx94 Traefik: tcp router clears read deadlines before tls forwarding, enabling stalled handshakes (Slowloris DOS)
ImpactThere is a potential vulnerability in Traefik managing TLS handshake on TCP routers.When Traefik processes a TLS connection on a TCP router, the read deadline used to bound protocol sniffing is cleared before the TLS handshake is completed. When a TLS handshake read error occurs, the code attempts a second handshake with different connection parameters, silently ignoring the initial error. A remote unauthenticated client can exploit this by sending an incomplete TLS record and stopping further data transmission, causing the TLS handshake to stall indefinitely and holding connections open.By opening many such stalled connections in parallel, an attacker can exhaust file descriptors and goroutines, degrading availability of all services on the affected entrypoint.Patcheshttps://github.com/traefik/traefik/releases/tag/v2.11.38https://github.com/traefik/traefik/releases/tag/v3.6.9WorkaroundsNo workaround available.For more informationIf there are any questions or comments about this advisory, please [open an issue](https://github.com/traefik/traefik/issues).Traefik's TCP router uses a connection-level read deadline to bound protocol sniffing (peeking a TLS client hello), but then clears the deadline via conn.SetDeadline(time.Time{}) before delegating the connection to TLS forwarding.A remote unauthenticated client can send an incomplete TLS record header and stop sending data. After the initial peek times out, the router clears the deadline and the subsequent TLS handshake reads can stall indefinitely, holding connections open and consuming resources.Expected vs ActualExpected: if an entrypoint-level read deadline is used to bound initial protocol sniffing, TLS handshake reads should remain bounded by a deadline (either the same deadline is preserved, or a dedicated handshake timeout is enforced).Actual: after protocol sniffing the router clears the connection deadline and delegates to TLS handling; an attacker can keep the TLS handshake stalled beyond the configured read timeout.SeverityHIGH
CWE: CWE-400 (Uncontrolled Resource Consumption)Affected Codepkg/server/router/tcp/router.go: (*Router).ServeTCP clears the deadline before TLS forwardingconn.SetDeadline(time.Time{}) removes the entrypoint-level deadline that previously bounded readsRoot CauseIn (*Router).ServeTCP, after sniffing a TLS client hello, the router removes the connection read deadline:TLS handshake reads that happen after this point are not guaranteed to have any deadline, so a client that stops sending bytes can keep the connection open indefinitely.Attacker ControlAttacker-controlled input is the raw TCP byte stream on an entrypoint that routes to a TLS forwarder. The attacker controls:Sending a partial TLS record header (enough to trigger the TLS sniffing path)Stopping further sends so the subsequent handshake read blocksImpactEach stalled connection occupies file descriptors and goroutines (and may consume additional memory depending on buffering). By opening many such connections in parallel, an attacker can cause resource exhaustion and degrade availability.ReproductionAttachments include poc.zip with a self-contained integration harness. It pins the repository commit, applies fix.patch as the control variant, and runs a regression-style test that demonstrates the stall in canonical mode and the timeout in control mode.Run canonical (vulnerable):Canonical output excerpt: PROOF_MARKERRun control (deadline preserved / no stall):Control output excerpt: NC_MARKERRecommended FixDo not clear the entrypoint-level deadline prior to completing TLS handshake, or enforce a dedicated handshake timeout for the TLS forwarder path.Fix accepted when: an incomplete TLS record cannot stall past the configured entrypoint-level read deadline (or an explicit handshake timeout), and a regression test covers the canonical/control behavior.
CVSS v3.1 score: 7.5