39 lines
1.2 KiB
Diff
39 lines
1.2 KiB
Diff
From a7c18d44b46e02dadfe3da225a06e5091f5f328e Mon Sep 17 00:00:00 2001
|
|
From: Norman Maurer <norman_maurer@apple.com>
|
|
Date: Wed, 11 Dec 2019 15:49:07 +0100
|
|
Subject: [PATCH] Detect missing colon when parsing http headers with no value
|
|
(#9871)
|
|
|
|
Motivation:
|
|
|
|
Technical speaking its valid to have http headers with no values so we should support it. That said we need to detect if these are "generated" because of an "invalid" fold.
|
|
|
|
Modifications:
|
|
|
|
- Detect if a colon is missing when parsing headers.
|
|
- Add unit test
|
|
|
|
Result:
|
|
|
|
Fixes https://github.com/netty/netty/issues/9866
|
|
---
|
|
.../handler/codec/http/HttpObjectDecoder.java | 5 +++++
|
|
.../codec/http/HttpRequestDecoderTest.java | 16 ++++++++++++++++
|
|
2 files changed, 21 insertions(+)
|
|
|
|
--- a/src/main/java/org/jboss/netty/handler/codec/http/HttpMessageDecoder.java
|
|
+++ b/src/main/java/org/jboss/netty/handler/codec/http/HttpMessageDecoder.java
|
|
@@ -731,6 +731,11 @@
|
|
}
|
|
}
|
|
|
|
+ if (nameEnd == length) {
|
|
+ // There was no colon present at all.
|
|
+ throw new IllegalArgumentException("No colon found");
|
|
+ }
|
|
+
|
|
for (colonEnd = nameEnd; colonEnd < length; colonEnd ++) {
|
|
if (sb.charAt(colonEnd) == ':') {
|
|
colonEnd ++;
|
|
|