From 243669584ab726471e32fb820b29d7eea85ecb64 Mon Sep 17 00:00:00 2001 From: Jonas Konrad Date: Fri, 5 Sep 2025 11:02:21 +0200 Subject: [PATCH] Improve GHSA-fghv-69vj-qj49 --- .../2025/09/GHSA-fghv-69vj-qj49/GHSA-fghv-69vj-qj49.json | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/advisories/github-reviewed/2025/09/GHSA-fghv-69vj-qj49/GHSA-fghv-69vj-qj49.json b/advisories/github-reviewed/2025/09/GHSA-fghv-69vj-qj49/GHSA-fghv-69vj-qj49.json index 3c7d78fff9498..c27903c11b786 100644 --- a/advisories/github-reviewed/2025/09/GHSA-fghv-69vj-qj49/GHSA-fghv-69vj-qj49.json +++ b/advisories/github-reviewed/2025/09/GHSA-fghv-69vj-qj49/GHSA-fghv-69vj-qj49.json @@ -8,12 +8,7 @@ ], "summary": "Netty vulnerable to request smuggling due to incorrect parsing of chunk extensions", "details": "## Summary\nA flaw in netty's parsing of chunk extensions in HTTP/1.1 messages with chunked encoding can lead to request smuggling issues with some reverse proxies.\n\n## Details\nWhen encountering a newline character (LF) while parsing a chunk extension, netty interprets the newline as the end of the chunk-size line regardless of whether a preceding carriage return (CR) was found. This is in violation of the HTTP 1.1 standard which specifies that the chunk extension is terminated by a CRLF sequence (see the [RFC](https://datatracker.ietf.org/doc/html/rfc9112#name-chunked-transfer-coding)).\n\nThis is by itself harmless, but consider an intermediary with a similar parsing flaw: while parsing a chunk extension, the intermediary interprets an LF without a preceding CR as simply part of the chunk extension (this is also in violation of the RFC, because whitespace characters are not allowed in chunk extensions). We can use this discrepancy to construct an HTTP request that the intermediary will interpret as one request but netty will interpret as two (all lines ending with CRLF, notice the LFs in the chunk extension):\n\n```\nPOST /one HTTP/1.1\nHost: localhost:8080\nTransfer-Encoding: chunked\n\n48;\\nAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA\\n0\n\nPOST /two HTTP/1.1\nHost: localhost:8080\nTransfer-Encoding: chunked\n\n0\n\n```\n\nThe intermediary will interpret this as a single request. Once forwarded to netty, netty will interpret it as two separate requests. This is a problem, because attackers can then the intermediary, as well as perform standard request smuggling attacks against other live users (see [this Portswigger article](https://portswigger.net/web-security/request-smuggling/exploiting)).\n\n## Impact\nThis is a request smuggling issue which can be exploited for bypassing front-end access control rules as well as corrupting the responses served to other live clients.\n\nThe impact is high, but it only affects setups that use a front-end which:\n1. Interprets LF characters (without preceding CR) in chunk extensions as part of the chunk extension.\n2. Forwards chunk extensions without normalization.\n\n## Disclosure\n\n - This vulnerability was disclosed on June 18th, 2025 here: https://w4ke.info/2025/06/18/funky-chunks.html\n\n## Discussion\nDiscussion for this vulnerability can be found here:\n - https://github.com/netty/netty/issues/15522\n - https://github.com/JLLeitschuh/unCVEed/issues/1\n\n## Credit\n\n - Credit to @JeppW for uncovering this vulnerability.\n - Credit to @JLLeitschuh at [Socket](https://socket.dev/) for coordinating the vulnerability disclosure.", - "severity": [ - { - "type": "CVSS_V4", - "score": "CVSS:4.0/AV:N/AC:L/AT:P/PR:N/UI:N/VC:N/VI:H/VA:N/SC:N/SI:N/SA:N" - } - ], + "severity": [], "affected": [ { "package": { @@ -88,7 +83,7 @@ "cwe_ids": [ "CWE-444" ], - "severity": "HIGH", + "severity": "LOW", "github_reviewed": true, "github_reviewed_at": "2025-09-04T17:35:20Z", "nvd_published_at": null