svn commit: r441779 - /db/derby/code/trunk/java/drda/org/apache/derby/impl/drda/NetworkServerControlImpl.java

[ka...@apache.org]

Author: kahatlen
Date: Sat Sep  9 04:26:03 2006
New Revision: 441779

URL: http://svn.apache.org/viewvc?view=rev&rev=441779
Log:
DERBY-1675: Network Server should not send to client that it supports
EUSRIDPWD when running against Sun JVM

Updating comments. Patch contributed by Sunitha Kambhampati.

Modified:
    db/derby/code/trunk/java/drda/org/apache/derby/impl/drda/NetworkServerControlImpl.java

Modified: db/derby/code/trunk/java/drda/org/apache/derby/impl/drda/NetworkServerControlImpl.java
URL: http://svn.apache.org/viewvc/db/derby/code/trunk/java/drda/org/apache/derby/impl/drda/NetworkServerControlImpl.java?view=diff&rev=441779&r1=441778&r2=441779
==============================================================================
--- db/derby/code/trunk/java/drda/org/apache/derby/impl/drda/NetworkServerControlImpl.java (original)
+++ db/derby/code/trunk/java/drda/org/apache/derby/impl/drda/NetworkServerControlImpl.java Sat Sep  9 04:26:03 2006
@@ -297,11 +297,19 @@
 	// databases it has booted.
 	private boolean shutdownDatabasesOnShutdown = false;
 	
-    // Sun JCE does not have support for EUSRIDPWD, whereas
-    // most versions of IBM JCE have support for this.  Hence
-    // find out if the server can support EUSRIDPWD.
+    /**
+     * Can EUSRIDPWD security mechanism be used with 
+     * the current JVM
+     */
     private static boolean SUPPORTS_EUSRIDPWD = false;
 
+    /*
+     * DRDA Specification for the EUSRIDPWD security mechanism
+     * requires DH algorithm support with a 32-byte prime to be
+     * used. Not all JCE implementations have support for this.
+     * Hence here we need to find out if EUSRIDPWD can be used
+     * with the current JVM.
+     */ 
     static
     {
         try
@@ -2603,16 +2611,16 @@
     }
    
     /**
-     * EUSRIDPWD support depends on the availability of the
-     * algorithm in the JCE implementation in the classpath 
-     * of the server. At runtime, information about this 
-     * capability is figured out.  
-     * @return whether EUSRIDPWD is supported or not
-     */
+     * This method returns whether EUSRIDPWD security mechanism
+     * is supported or not. See class static block for more
+     * info.
+     * @return true if EUSRIDPWD is supported, false otherwise
+     */ 
     boolean supportsEUSRIDPWD()
     {
         return SUPPORTS_EUSRIDPWD;
     }
+    
 	/**
 	 * Get integer property values
 	 *