moparisthebest/poi
1
0
Fork 0
Code Issues Pull Requests Releases 1 Wiki Activity

add additional sanity checks

Browse Source 
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1142880 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Sergey Vladimirov 2011-07-05 03:02:12 +00:00
parent efd149c834
commit 9b69954212
2 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/scratchpad/src/org/apache/poi/hwpf/model/BytePropertyNode.java
View File

@ -38,6 +38,11 @@ public abstract class BytePropertyNode extends PropertyNode {
translator.getCharIndex(fcEnd, translator.getCharIndex(fcStart)), translator.getCharIndex(fcEnd, translator.getCharIndex(fcStart)),
buf buf
); );
if ( fcStart > fcEnd )
throw new IllegalArgumentException( "fcStart (" + fcStart
+ ") > fcEnd (" + fcEnd + ")" );
this.startBytes = fcStart; this.startBytes = fcStart;
this.endBytes = fcEnd; this.endBytes = fcEnd;
} }

4
src/scratchpad/src/org/apache/poi/hwpf/model/PropertyNode.java
View File

@ -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"); _logger.log(POILogger.WARN, "A property claimed to start before zero, at " + _cpStart + "! Resetting it to zero, and hoping for the best");
_cpStart = 0; _cpStart = 0;
} }
if ( fcStart > fcEnd )
throw new IllegalArgumentException( "fcStart (" + fcStart
+ ") > fcEnd (" + fcEnd + ")" );
} }
/** /**