You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@poi.apache.org by jh...@apache.org on 2006/08/28 05:54:05 UTC
svn commit: r437522 -
/jakarta/poi/trunk/src/java/org/apache/poi/hssf/util/CellReference.java
Author: jheight
Date: Sun Aug 27 20:54:04 2006
New Revision: 437522
URL: http://svn.apache.org/viewvc?rev=437522&view=rev
Log:
Bug28755: CellReference will throws IllegalArgumentException when an illegal cell reference is provided.
Modified:
jakarta/poi/trunk/src/java/org/apache/poi/hssf/util/CellReference.java
Modified: jakarta/poi/trunk/src/java/org/apache/poi/hssf/util/CellReference.java
URL: http://svn.apache.org/viewvc/jakarta/poi/trunk/src/java/org/apache/poi/hssf/util/CellReference.java?rev=437522&r1=437521&r2=437522&view=diff
==============================================================================
--- jakarta/poi/trunk/src/java/org/apache/poi/hssf/util/CellReference.java (original)
+++ jakarta/poi/trunk/src/java/org/apache/poi/hssf/util/CellReference.java Sun Aug 27 20:54:04 2006
@@ -34,13 +34,17 @@
public CellReference(String cellRef) {
String[] parts = separateRefParts(cellRef);
sheetName = parts[0];
- String ref = parts[1];
+ String ref = parts[1];
+ if ((ref == null)||("".equals(ref)))
+ throw new IllegalArgumentException("Invalid Formula cell reference: '"+cellRef+"'");
if (ref.charAt(0) == '$') {
colAbs=true;
ref=ref.substring(1);
}
col = convertColStringToNum(ref);
ref=parts[2];
+ if ((ref == null)||("".equals(ref)))
+ throw new IllegalArgumentException("Invalid Formula cell reference: '"+cellRef+"'");
if (ref.charAt(0) == '$') {
rowAbs=true;
ref=ref.substring(1);
---------------------------------------------------------------------
To unsubscribe, e-mail: poi-dev-unsubscribe@jakarta.apache.org
Mailing List: http://jakarta.apache.org/site/mail2.html#poi
The Apache Jakarta POI Project: http://jakarta.apache.org/poi/