Apache
/
poi
mirror of https://github.com/apache/poi.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix the block allocation test, but still leave the recent blocks hssf bug fixed 

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@758353 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Nick Burch 2009-03-25 17:22:06 +00:00
parent 5e947a3a6a
commit 34bd8dd94d
2 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/java/org/apache/poi/poifs/storage/BlockAllocationTableReader.java
View File

@ -186,6 +186,7 @@ public class BlockAllocationTableReader
List blocks = new ArrayList();
int currentBlock = startBlock;
boolean firstPass = true;
ListManagedBlock dataBlock = null;
// Process the chain from the start to the end
// Normally we have header, data, end
@ -193,16 +194,21 @@ public class BlockAllocationTableReader
// For those cases, stop at the header, not the end
while (currentBlock != POIFSConstants.END_OF_CHAIN) {
try {
blocks.add(blockList.remove(currentBlock));
// Grab the data at the current block offset
dataBlock = blockList.remove(currentBlock);
blocks.add(dataBlock);
// Now figure out which block we go to next
currentBlock = _entries.get(currentBlock);
firstPass = false;
} catch(IOException e) {
if(currentBlock == headerPropertiesStartBlock) {
// Special case where things are in the wrong order
System.err.println("Warning, header block comes after data blocks in POIFS block listing");
currentBlock = POIFSConstants.END_OF_CHAIN;
} else if(currentBlock == 0) {
} else if(currentBlock == 0 && firstPass) {
// Special case where the termination isn't done right
System.err.println("Warning, incorrectly terminated data blocks in POIFS block listing (should end at -2, ended at 0)");
// on an empty set
System.err.println("Warning, incorrectly terminated empty data blocks in POIFS block listing (should end at -2, ended at 0)");
currentBlock = POIFSConstants.END_OF_CHAIN;
} else {
// Ripple up
@ -212,7 +218,7 @@ public class BlockAllocationTableReader
}
return ( ListManagedBlock [] ) blocks
.toArray(new ListManagedBlock[ 0 ]);
.toArray(new ListManagedBlock[ blocks.size() ]);
}
// methods for debugging reader

2
src/testcases/org/apache/poi/poifs/storage/TestBlockAllocationTableReader.java
View File

@ -1329,7 +1329,7 @@ public class TestBlockAllocationTableReader
if (expected_length[ j ] == -1)
{
fail("document " + j + " should have failed");
fail("document " + j + " should have failed, but found a length of " + dataBlocks.length);
}
else
{