Bug 47261 - SlideShow.removeSlide makes PPT corrupted

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1648458 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Andreas Beeker 2014-12-30 01:05:41 +00:00
parent 74ddc1440e
commit 0456faaa44
4 changed files with 20 additions and 0 deletions

View File

@ -175,6 +175,10 @@ public final class Document extends PositionDependentRecordContainer
// The new SlideListWithText should go in // The new SlideListWithText should go in
// just before the EndDocumentRecord // just before the EndDocumentRecord
Record endDoc = _children[_children.length - 1]; Record endDoc = _children[_children.length - 1];
if(endDoc.getRecordType() == RecordTypes.RoundTripCustomTableStyles12Atom.typeID) {
// last record can optionally be a RoundTripCustomTableStyles12Atom
endDoc = _children[_children.length - 2];
}
if(endDoc.getRecordType() != RecordTypes.EndDocument.typeID) { if(endDoc.getRecordType() != RecordTypes.EndDocument.typeID) {
throw new IllegalStateException("The last child record of a Document should be EndDocument, but it was " + endDoc); throw new IllegalStateException("The last child record of a Document should be EndDocument, but it was " + endDoc);
} }

View File

@ -67,6 +67,7 @@ public final class RecordTypes {
public static final Type NamedShow = new Type(1041,null); public static final Type NamedShow = new Type(1041,null);
public static final Type NamedShowSlides = new Type(1042,null); public static final Type NamedShowSlides = new Type(1042,null);
public static final Type SheetProperties = new Type(1044,null); public static final Type SheetProperties = new Type(1044,null);
public static final Type RoundTripCustomTableStyles12Atom = new Type(1064,null);
public static final Type List = new Type(2000,null); public static final Type List = new Type(2000,null);
public static final Type FontCollection = new Type(2005,FontCollection.class); public static final Type FontCollection = new Type(2005,FontCollection.class);
public static final Type BookmarkCollection = new Type(2019,null); public static final Type BookmarkCollection = new Type(2019,null);

View File

@ -21,6 +21,7 @@ import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotNull; import static org.junit.Assert.assertNotNull;
import static org.junit.Assert.assertTrue; import static org.junit.Assert.assertTrue;
import java.io.ByteArrayOutputStream;
import java.io.File; import java.io.File;
import java.io.FileInputStream; import java.io.FileInputStream;
import java.io.IOException; import java.io.IOException;
@ -595,4 +596,18 @@ public final class TestBugs {
inputStream.close(); inputStream.close();
} }
} }
@Test
public void bug47261() throws Exception {
InputStream inputStream = new FileInputStream(_slTests.getFile("bug47261.ppt"));
try {
SlideShow slideShow = new SlideShow(inputStream);
slideShow.removeSlide(0);
slideShow.createSlide();
ByteArrayOutputStream bos = new ByteArrayOutputStream();
slideShow.write(bos);
} finally {
inputStream.close();
}
}
} }

Binary file not shown.