Author: tyrell
Date: Thu Jan 17 23:01:19 2008
New Revision: 12458

Log:

Adding a null check when retrieving cookies from session.

Modified:
   trunk/mashup/java/modules/www/signout.jsp

Modified: trunk/mashup/java/modules/www/signout.jsp
==============================================================================
--- trunk/mashup/java/modules/www/signout.jsp   (original)
+++ trunk/mashup/java/modules/www/signout.jsp   Thu Jan 17 23:01:19 2008
@@ -51,9 +51,6 @@
 <%@ page import="java.util.List" %>
 <%@ page import="java.util.Map" %>
 
-<!--Required to keep a user logged in if 'Remember Me' option is selected-->
-<%@ include file="validate_login.jsp" %>
-
 <%
     // required by header.jsp
     String thisPage = "signout.jsp";
@@ -99,18 +96,22 @@
         request.getSession().setAttribute(MashupConstants.USER_REGISTRY, 
secureRegistry);
 
         //Deleting user login cookies
-        curCookie = null;
-        for (int x = 0; x < cookies.length; x++) {
-            curCookie = cookies[x];
-            if (curCookie.getName().equalsIgnoreCase("rememberMe")) {
-                curCookie.setMaxAge(0);
-                response.addCookie(curCookie);
-            } else if (curCookie.getName().equalsIgnoreCase("username")) {
-                curCookie.setMaxAge(0);
-                response.addCookie(curCookie);
-            } else if (curCookie.getName().equalsIgnoreCase("password")) {
-                curCookie.setMaxAge(0);
-                response.addCookie(curCookie);
+        Cookie[] cookies = request.getCookies();
+
+        if (cookies != null) {
+            Cookie curCookie = null;
+            for (int x = 0; x < cookies.length; x++) {
+                curCookie = cookies[x];
+                if (curCookie.getName().equalsIgnoreCase("rememberMe")) {
+                    curCookie.setMaxAge(0);
+                    response.addCookie(curCookie);
+                } else if (curCookie.getName().equalsIgnoreCase("username")) {
+                    curCookie.setMaxAge(0);
+                    response.addCookie(curCookie);
+                } else if (curCookie.getName().equalsIgnoreCase("password")) {
+                    curCookie.setMaxAge(0);
+                    response.addCookie(curCookie);
+                }
             }
         }
 

_______________________________________________
Mashup-dev mailing list
[email protected]
http://www.wso2.org/cgi-bin/mailman/listinfo/mashup-dev

Reply via email to