Author: jaz
Date: Mon Sep 24 23:29:34 2007
New Revision: 579088

URL: http://svn.apache.org/viewvc?rev=579088&view=rev
Log:
fixes possible NPE (from Anil Patel)

Modified:
    ofbiz/trunk/framework/common/src/org/ofbiz/common/login/LoginServices.java

Modified: 
ofbiz/trunk/framework/common/src/org/ofbiz/common/login/LoginServices.java
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/src/org/ofbiz/common/login/LoginServices.java?rev=579088&r1=579087&r2=579088&view=diff
==============================================================================
--- ofbiz/trunk/framework/common/src/org/ofbiz/common/login/LoginServices.java 
(original)
+++ ofbiz/trunk/framework/common/src/org/ofbiz/common/login/LoginServices.java 
Mon Sep 24 23:29:34 2007
@@ -789,9 +789,10 @@
             //No valid value is found so don't bother to save any password 
history
             passwordChangeHistoryLimit = 0;
         }
-        Debug.logInfo(" checkNewPassword 
passwordChangeHistoryLimitpasswordChangeHistoryLimitpasswordChangeHistoryLimit" 
+ passwordChangeHistoryLimit, module);
-        if(passwordChangeHistoryLimit > 0 ){
-            Debug.logInfo(" checkNewPassword Checking of user is tyring to use 
old password " + passwordChangeHistoryLimit, module);
+        Debug.logInfo(" password.change.history.limit is set to " + 
passwordChangeHistoryLimit, module);
+        Debug.logInfo(" userLogin is set to " + userLogin, module);
+        if(passwordChangeHistoryLimit > 0 && userLogin != null ){
+            Debug.logInfo(" checkNewPassword Checking if user is tyring to use 
old password " + passwordChangeHistoryLimit, module);
             GenericDelegator delegator = userLogin.getDelegator();
             String newPasswordHash = newPassword;
             if (useEncryption) {


Reply via email to