You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@myfaces.apache.org by lu...@apache.org on 2015/09/15 01:18:36 UTC

svn commit: r1703077 - /myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context/servlet/ServletExternalContextImplBase.java

Author: lu4242
Date: Mon Sep 14 23:18:35 2015
New Revision: 1703077

URL: http://svn.apache.org/r1703077
Log:
MYFACES-3982 javax.faces.context.ExternalContext#getInitParameter should throw NPE when name is null

Modified:
    myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context/servlet/ServletExternalContextImplBase.java

Modified: myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context/servlet/ServletExternalContextImplBase.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context/servlet/ServletExternalContextImplBase.java?rev=1703077&r1=1703076&r2=1703077&view=diff
==============================================================================
--- myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context/servlet/ServletExternalContextImplBase.java (original)
+++ myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/context/servlet/ServletExternalContextImplBase.java Mon Sep 14 23:18:35 2015
@@ -100,6 +100,10 @@ public abstract class ServletExternalCon
     @Override
     public String getInitParameter(final String s)
     {
+        if (s == null)
+        {
+            throw new NullPointerException("Init parameter name cannot be null");
+        }
         return _servletContext.getInitParameter(s);
     }