Re: ArrayIndexOutOfBoundsException-Bug?

• To: Mathias Stümpert <atzes@xxxxxx>
• Subject: Re: ArrayIndexOutOfBoundsException-Bug?
• From: Dave Glowacki <dglo@xxxxxxxxxxxxxxxxxx>
• Date: Wed, 01 May 2002 11:43:33 -0500

=?iso-8859-1?Q?Mathias_St=FCmpert?= wrote:
> I think I found a bug in ShapeControl.setShapes:
>
> int len = Math.max(shs.length, shapes.length);
> for (int i=0; i<len; i++) {
>   shapes[i] = shs[i];
> }
>
> That causes an ArrayIndexOutOfBoundsException in my app because the for
> loops to the max of the lengths of both arrays.

Looks like a bug to me, too.  This patch should take care of
that:

diff -u -3 -p -r1.20 ShapeControl.java
--- ShapeControl.java   7 Jan 2002 18:42:55 -0000       1.20
+++ ShapeControl.java   1 May 2002 16:33:17 -0000
@@ -101,9 +101,14 @@ public class ShapeControl extends Contro
          throws VisADException, RemoteException {
     if (shapeSet == null) return;
     if (shs != null && shs.length > 0) {
-      int len = Math.max(shs.length, shapes.length);
+      final int len = Math.min(shs.length, shapes.length);
       for (int i=0; i<len; i++) {
         shapes[i] = shs[i];
+      }
+      if (shapes.length > shs.length) {
+        for (int i=shs.length; i < shapes.length; i++) {
+          shapes[i] = null;
+        }
       }
     }
     changeControl(true);

• References:
  • ArrayIndexOutOfBoundsException-Bug?
    • From: Mathias Stümpert