svn commit: r935983 - /tomcat/trunk/java/org/apache/catalina/realm/JNDIRealm.java

[kkolinko]

Author: kkolinko
Date: Tue Apr 20 16:30:52 2010
New Revision: 935983

URL: http://svn.apache.org/viewvc?rev=935983&view=rev
Log:
Additional patch for https://issues.apache.org/bugzilla/show_bug.cgi?id=48589
Do not access JNDIRealm$User class fields directly, but use the getter methods,
as (theoretically) those can be overwritten in subclasses.

Modified:
    tomcat/trunk/java/org/apache/catalina/realm/JNDIRealm.java

Modified: tomcat/trunk/java/org/apache/catalina/realm/JNDIRealm.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/realm/JNDIRealm.java?rev=935983&r1=935982&r2=935983&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/realm/JNDIRealm.java (original)
+++ tomcat/trunk/java/org/apache/catalina/realm/JNDIRealm.java Tue Apr 20 
16:30:52 2010
@@ -1425,10 +1425,10 @@ public class JNDIRealm extends RealmBase
          if (containerLog.isTraceEnabled()) {
              if (validated) {
                  
containerLog.trace(sm.getString("jndiRealm.authenticateSuccess",
-                                  user.username));
+                                  user.getUserName()));
              } else {
                  
containerLog.trace(sm.getString("jndiRealm.authenticateFailure",
-                                  user.username));
+                                  user.getUserName()));
              }
          }
          return (validated);
@@ -1454,7 +1454,7 @@ public class JNDIRealm extends RealmBase
         if (info == null || credentials == null)
             return (false);
 
-        String password = info.password;
+        String password = info.getPassword();
         if (password == null)
             return (false);
 
@@ -1544,7 +1544,7 @@ public class JNDIRealm extends RealmBase
          if (credentials == null || user == null)
              return (false);
 
-         String dn = user.dn;
+         String dn = user.getDN();
          if (dn == null)
              return (false);
 
@@ -1608,8 +1608,8 @@ public class JNDIRealm extends RealmBase
         if (user == null)
             return (null);
 
-        String dn = user.dn;
-        String username = user.username;
+        String dn = user.getDN();
+        String username = user.getUserName();
 
         if (dn == null || username == null)
             return (null);
@@ -1618,7 +1618,7 @@ public class JNDIRealm extends RealmBase
             containerLog.trace("  getRoles(" + dn + ")");
 
         // Start with roles retrieved from the user entry
-        List<String> list = user.roles;
+        List<String> list = user.getRoles();
         if (list == null) {
             list = new ArrayList<String>();
         }
@@ -1933,7 +1933,7 @@ public class JNDIRealm extends RealmBase
         User user = getUser(context, username);
 
         if (user != null) {
-            return new GenericPrincipal(user.username, user.password,
+            return new GenericPrincipal(user.getUserName(), user.getPassword(),
                     getRoles(context, user));
         }
         



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org