Author: ruchith
Date: Tue Dec 11 03:22:26 2007
New Revision: 10961

Log:

Getting rid of an NPE

Modified:
   
trunk/solutions/identity/modules/identity-provider/src/main/java/org/wso2/solutions/identity/resources.properties
   
trunk/solutions/identity/modules/identity-provider/src/main/java/org/wso2/solutions/identity/sts/IdentityTokenIssuer.java

Modified: 
trunk/solutions/identity/modules/identity-provider/src/main/java/org/wso2/solutions/identity/resources.properties
==============================================================================
--- 
trunk/solutions/identity/modules/identity-provider/src/main/java/org/wso2/solutions/identity/resources.properties
   (original)
+++ 
trunk/solutions/identity/modules/identity-provider/src/main/java/org/wso2/solutions/identity/resources.properties
   Tue Dec 11 03:22:26 2007
@@ -50,6 +50,8 @@
 relyingPartyExists = Relying party {0} already exists
 errorInKeystoreAccess = Error accessing keystore
 emailNotActivated = Email Verification not activted now, but all parameters 
are saved. It will be activated when DefaultRealm is selected.
+noValueForRequestedAttribute = Value of attribute "{0}" missing
+
 
 #db errors
 createDuplicateEntity = Duplcate entity exist

Modified: 
trunk/solutions/identity/modules/identity-provider/src/main/java/org/wso2/solutions/identity/sts/IdentityTokenIssuer.java
==============================================================================
--- 
trunk/solutions/identity/modules/identity-provider/src/main/java/org/wso2/solutions/identity/sts/IdentityTokenIssuer.java
   (original)
+++ 
trunk/solutions/identity/modules/identity-provider/src/main/java/org/wso2/solutions/identity/sts/IdentityTokenIssuer.java
   Tue Dec 11 03:22:26 2007
@@ -527,6 +527,11 @@
 
             int index = uri.lastIndexOf("/");
 
+            if(claim.value == null) {
+                throw new IdentityProviderException(
+                        "noValueForRequestedAttribute",
+                        new String[] { claim.uri });
+            }
             List list = Arrays.asList(new String[] { claim.value });
             String displayTag = ipData.getDisplayName(uri);
             if ((displayTag == null) && (claim.bOptional == false)) {

_______________________________________________
Identity-dev mailing list
[email protected]
http://wso2.org/cgi-bin/mailman/listinfo/identity-dev

Reply via email to