On 29/10/2018 18:40, ma...@apache.org wrote:
Author: markt
Date: Mon Oct 29 18:40:16 2018
New Revision: 1845162

URL: http://svn.apache.org/viewvc?rev=1845162&view=rev
Log:
Fix redundant assignment warning.

Chris,

I think this is what you intended. Can you check my change here please?

Thanks,

Mark



Modified:
     
tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/EncryptInterceptor.java

Modified: 
tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/EncryptInterceptor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/EncryptInterceptor.java?rev=1845162&r1=1845161&r2=1845162&view=diff
==============================================================================
--- 
tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/EncryptInterceptor.java
 (original)
+++ 
tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/EncryptInterceptor.java
 Mon Oct 29 18:40:16 2018
@@ -179,10 +179,11 @@ public class EncryptInterceptor extends
       */
      @Override
      public void setEncryptionKey(byte[] key) {
-        if(null == key)
-            key = null;
-        else
+        if (null == key) {
+            encryptionKeyBytes = null;
+        } else {
              encryptionKeyBytes = key.clone();
+        }
      }
/**



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


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

Reply via email to