You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by ma...@apache.org on 2013/10/31 16:04:32 UTC

svn commit: r1537523 - /commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/proxy/JdkProxySource.java

Author: markt
Date: Thu Oct 31 15:04:32 2013
New Revision: 1537523

URL: http://svn.apache.org/r1537523
Log:
Fix Findbugs gremlin that appeared because class was made public

Modified:
    commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/proxy/JdkProxySource.java

Modified: commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/proxy/JdkProxySource.java
URL: http://svn.apache.org/viewvc/commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/proxy/JdkProxySource.java?rev=1537523&r1=1537522&r2=1537523&view=diff
==============================================================================
--- commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/proxy/JdkProxySource.java (original)
+++ commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/proxy/JdkProxySource.java Thu Oct 31 15:04:32 2013
@@ -41,7 +41,9 @@ public class JdkProxySource<T> implement
      */
     public JdkProxySource(ClassLoader classLoader, Class<?>[] interfaces) {
         this.classLoader = classLoader;
-        this.interfaces = interfaces;
+        // Defensive copy
+        this.interfaces = new Class<?>[interfaces.length];
+        System.arraycopy(interfaces, 0, this.interfaces, 0, interfaces.length);
     }