// unfortunately this might break silently with newer/other Java implementations,
// but we at least have unit-tests which will indicate this when run on Windows
private static void unmap(final ByteBuffer buffer) {
- AccessController.doPrivileged(new PrivilegedAction<Void>() {
- @Override
- @SuppressForbidden("Java 9 Jigsaw whitelists access to sun.misc.Cleaner, so setAccessible works")
- public Void run() {
- try {
- final Method getCleanerMethod = buffer.getClass().getMethod("cleaner");
- getCleanerMethod.setAccessible(true);
- final Object cleaner = getCleanerMethod.invoke(buffer);
- if (cleaner != null) {
- cleaner.getClass().getMethod("clean").invoke(cleaner);
- }
- } catch (Exception e) {
- logger.log(POILogger.WARN, "Unable to unmap memory mapped ByteBuffer.", e);
- }
- return null; // Void
- }
- });
+ // not necessary for HeapByteBuffer, avoid lots of log-output on this class
+ if(buffer.getClass().getName().endsWith("HeapByteBuffer")) {
+ return;
+ }
+
+ AccessController.doPrivileged(new PrivilegedAction<Void>() {
+ @Override
+ @SuppressForbidden("Java 9 Jigsaw whitelists access to sun.misc.Cleaner, so setAccessible works")
+ public Void run() {
+ try {
+ final Method getCleanerMethod = buffer.getClass().getMethod("cleaner");
+ getCleanerMethod.setAccessible(true);
+ final Object cleaner = getCleanerMethod.invoke(buffer);
+ if (cleaner != null) {
+ cleaner.getClass().getMethod("clean").invoke(cleaner);
+ }
+ } catch (Exception e) {
+ logger.log(POILogger.WARN, "Unable to unmap memory mapped ByteBuffer.", e);
+ }
+ return null; // Void
+ }
+ });
}
}
package org.apache.poi.poifs.nio;
-import java.io.File;
-import java.io.FileNotFoundException;
-import java.io.FileOutputStream;
-import java.io.IOException;
-import java.io.InputStream;
-import java.io.OutputStream;
-import java.nio.BufferUnderflowException;
-import java.nio.ByteBuffer;
-
+import junit.framework.TestCase;
import org.apache.poi.POIDataSamples;
import org.apache.poi.util.IOUtils;
import org.apache.poi.util.TempFile;
-import junit.framework.TestCase;
+import java.io.*;
+import java.nio.BufferUnderflowException;
+import java.nio.ByteBuffer;
/**
* Tests for the datasource implementations
}
}
- private void writeDataToFile(File temp) throws FileNotFoundException, IOException {
+ private void writeDataToFile(File temp) throws IOException {
OutputStream str = new FileOutputStream(temp);
try {
InputStream in = data.openResourceAsStream("Notes.ole2");
assertEquals(0, bs.position());
assertEquals(0xd0 - 256, bs.get(0));
assertEquals(0xcf - 256, bs.get(1));
- assertEquals(0x11 - 000, bs.get(2));
+ assertEquals(0x11, bs.get(2));
assertEquals(0xe0 - 256, bs.get(3));
assertEquals(0xd0 - 256, bs.get());
assertEquals(0xcf - 256, bs.get());
- assertEquals(0x11 - 000, bs.get());
+ assertEquals(0x11, bs.get());
assertEquals(0xe0 - 256, bs.get());
// Mid way through
// Can't go off the end
try {
- bs = ds.read(4, 8192);
+ ds.read(4, 8192);
if(!writeable) {
fail("Shouldn't be able to read off the end of the file");
}
} catch (IllegalArgumentException e) {
+ // expected here
}
}
try {
bs.get();
fail("Shouldn't be able to read off the end");
- } catch(BufferUnderflowException e) {}
+ } catch(BufferUnderflowException e) {
+ // expected here
+ }
// Past the end
try {
- bs = ds.read(4, 256);
+ ds.read(4, 256);
fail("Shouldn't be able to read off the end");
- } catch(IndexOutOfBoundsException e) {}
+ } catch(IndexOutOfBoundsException e) {
+ // expected here
+ }
// Overwrite
}
}
obj[17] = chrs.toString();
- format.append("%18$s"+HexDump.EOL);
+ format.append("%18$s").append(HexDump.EOL);
String str = String.format(LocaleUtil.getUserLocale(), format.toString(), obj);
strExp.append(str);
}
- byte bytesExp[] = strExp.toString().getBytes(HexDump.UTF8);
- return bytesExp;
+ return strExp.toString().getBytes(HexDump.UTF8);
}
@Test
assertEquals("FFFF", HexDump.toHex((short)0xFFFF));
- assertEquals("00000000000004D2", HexDump.toHex(1234l));
+ assertEquals("00000000000004D2", HexDump.toHex(1234L));
assertEquals("0xFE", HexDump.byteToHex(-2));
assertEquals("0x25", HexDump.byteToHex(37));
@Test(expected=ArrayIndexOutOfBoundsException.class)
public void testDumpToStringOutOfIndex1() throws Exception {
- HexDump.dump(new byte[ 1 ], 0, -1);
+ HexDump.dump(new byte[1], 0, -1);
}
@Test(expected=ArrayIndexOutOfBoundsException.class)
public void testDumpToStringOutOfIndex2() throws Exception {
- HexDump.dump(new byte[ 1 ], 0, 2);
+ HexDump.dump(new byte[1], 0, 2);
}
+ @Test(expected=ArrayIndexOutOfBoundsException.class)
public void testDumpToStringOutOfIndex3() throws Exception {
- HexDump.dump(new byte[ 1 ], 0, 1);
+ HexDump.dump(new byte[1], 0, 1);
+ }
+
+ @Test
+ public void testDumpToStringNoDataEOL1() throws Exception {
+ HexDump.dump(new byte[0], 0, 1);
}
+ @Test
+ public void testDumpToStringNoDataEOL2() throws Exception {
+ HexDump.dump(new byte[0], 0, 0);
+ }
@Test
public void testDumpToPrintStream() throws IOException {