You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2013/02/20 14:10:51 UTC

svn commit: r1448147 - /directory/apacheds/trunk/protocol-kerberos/src/main/java/org/apache/directory/server/kerberos/kdc/authentication/AuthenticationService.java

Author: elecharny
Date: Wed Feb 20 13:10:51 2013
New Revision: 1448147

URL: http://svn.apache.org/r1448147
Log:
Slight modification (declare a variable to avoid accessing the value twice)

Modified:
    directory/apacheds/trunk/protocol-kerberos/src/main/java/org/apache/directory/server/kerberos/kdc/authentication/AuthenticationService.java

Modified: directory/apacheds/trunk/protocol-kerberos/src/main/java/org/apache/directory/server/kerberos/kdc/authentication/AuthenticationService.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/protocol-kerberos/src/main/java/org/apache/directory/server/kerberos/kdc/authentication/AuthenticationService.java?rev=1448147&r1=1448146&r2=1448147&view=diff
==============================================================================
--- directory/apacheds/trunk/protocol-kerberos/src/main/java/org/apache/directory/server/kerberos/kdc/authentication/AuthenticationService.java (original)
+++ directory/apacheds/trunk/protocol-kerberos/src/main/java/org/apache/directory/server/kerberos/kdc/authentication/AuthenticationService.java Wed Feb 20 13:10:51 2013
@@ -58,6 +58,7 @@ import org.apache.directory.shared.kerbe
 import org.apache.directory.shared.kerberos.components.EncryptedData;
 import org.apache.directory.shared.kerberos.components.EncryptionKey;
 import org.apache.directory.shared.kerberos.components.KdcReq;
+import org.apache.directory.shared.kerberos.components.KdcReqBody;
 import org.apache.directory.shared.kerberos.components.LastReq;
 import org.apache.directory.shared.kerberos.components.LastReqEntry;
 import org.apache.directory.shared.kerberos.components.MethodData;
@@ -130,6 +131,12 @@ public class AuthenticationService
     }
 
 
+    /**
+     * 
+     * @param authContext
+     * @throws KerberosException
+     * @throws InvalidTicketException
+     */
     private static void selectEncryptionType( AuthenticationContext authContext ) throws KerberosException,
         InvalidTicketException
     {
@@ -161,8 +168,10 @@ public class AuthenticationService
         InvalidTicketException
     {
         LOG_KRB.debug( "--> Getting the client Entry" );
+        KdcReqBody kdcReqBody = authContext.getRequest().getKdcReqBody();
         KerberosPrincipal principal = KerberosUtils.getKerberosPrincipal(
-            authContext.getRequest().getKdcReqBody().getCName(), authContext.getRequest().getKdcReqBody().getRealm() );
+            kdcReqBody.getCName(),
+            kdcReqBody.getRealm() );
         PrincipalStore store = authContext.getStore();
 
         try