add additional sanity checks

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1142880 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/src/scratchpad/src/org/apache/poi/hwpf/model/BytePropertyNode.java b/src/scratchpad/src/org/apache/poi/hwpf/model/BytePropertyNode.java
index 4ceb2d1d8dbe0b2c50c33df5074115357bae0b60..59ac18e7b1d5a3c804425a5d3d7196a360a7718d 100644 (file)
--- a/src/scratchpad/src/org/apache/poi/hwpf/model/BytePropertyNode.java
+++ b/src/scratchpad/src/org/apache/poi/hwpf/model/BytePropertyNode.java
@@ -38,6 +38,11 @@ public abstract class BytePropertyNode extends PropertyNode {
                                translator.getCharIndex(fcEnd, translator.getCharIndex(fcStart)),
                                buf
                );
+
+        if ( fcStart > fcEnd )
+            throw new IllegalArgumentException( "fcStart (" + fcStart
+                    + ") > fcEnd (" + fcEnd + ")" );
+
                 this.startBytes = fcStart;
                 this.endBytes = fcEnd;
        }

diff --git a/src/scratchpad/src/org/apache/poi/hwpf/model/PropertyNode.java b/src/scratchpad/src/org/apache/poi/hwpf/model/PropertyNode.java
index e3338fd610a610067cf332decf077cb9801901ed..6c5a23972a43135d7c75667908eeee20e447bc45 100644 (file)
--- a/src/scratchpad/src/org/apache/poi/hwpf/model/PropertyNode.java
+++ b/src/scratchpad/src/org/apache/poi/hwpf/model/PropertyNode.java
@@ -56,6 +56,10 @@ public abstract class PropertyNode implements Comparable, Cloneable
          _logger.log(POILogger.WARN, "A property claimed to start before zero, at " + _cpStart + "! Resetting it to zero, and hoping for the best");
          _cpStart = 0;
       }
+
+      if ( fcStart > fcEnd )
+          throw new IllegalArgumentException( "fcStart (" + fcStart
+                  + ") > fcEnd (" + fcEnd + ")" );
   }
 
   /**