diff --git a/codec-http/src/main/java/io/netty/handler/codec/http/DefaultHttpMessage.java b/codec-http/src/main/java/io/netty/handler/codec/http/DefaultHttpMessage.java index 3fc0535ab0..b3b6ff71bd 100644 --- a/codec-http/src/main/java/io/netty/handler/codec/http/DefaultHttpMessage.java +++ b/codec-http/src/main/java/io/netty/handler/codec/http/DefaultHttpMessage.java @@ -66,6 +66,9 @@ public abstract class DefaultHttpMessage extends DefaultHttpObject implements Ht @Override public HttpMessage setProtocolVersion(HttpVersion version) { + if (version == null) { + throw new NullPointerException("version"); + } this.version = version; return this; } diff --git a/codec-http/src/main/java/io/netty/handler/codec/http/DefaultHttpRequest.java b/codec-http/src/main/java/io/netty/handler/codec/http/DefaultHttpRequest.java index d4ed2b7179..392b8755aa 100644 --- a/codec-http/src/main/java/io/netty/handler/codec/http/DefaultHttpRequest.java +++ b/codec-http/src/main/java/io/netty/handler/codec/http/DefaultHttpRequest.java @@ -56,12 +56,18 @@ public class DefaultHttpRequest extends DefaultHttpMessage implements HttpReques @Override public HttpRequest setMethod(HttpMethod method) { + if (method == null) { + throw new NullPointerException("method"); + } this.method = method; return this; } @Override public HttpRequest setUri(String uri) { + if (uri == null) { + throw new NullPointerException("method"); + } this.uri = uri; return this; } diff --git a/codec-http/src/main/java/io/netty/handler/codec/http/DefaultHttpResponse.java b/codec-http/src/main/java/io/netty/handler/codec/http/DefaultHttpResponse.java index a36d77287c..fae07f8d05 100644 --- a/codec-http/src/main/java/io/netty/handler/codec/http/DefaultHttpResponse.java +++ b/codec-http/src/main/java/io/netty/handler/codec/http/DefaultHttpResponse.java @@ -45,6 +45,9 @@ public class DefaultHttpResponse extends DefaultHttpMessage implements HttpRespo @Override public HttpResponse setStatus(HttpResponseStatus status) { + if (status == null) { + throw new NullPointerException("status"); + } this.status = status; return this; }