Fixed #434: Add null check before accessing integer size

ion/src/main/java/com/fasterxml/jackson/dataformat/ion/IonParser.java

@@ -393,7 +393,17 @@ public NumberType getNumberType() throws IOException
                 //Ion decimals can be arbitrary precision, need to read as big decimal
                 return NumberType.BIG_DECIMAL;
             case INT:
-                IntegerSize size = _reader.getIntegerSize();
+                IntegerSize size = null;
+                // Temporary measure until this bug fixing is merged and published
+                // https://github.com/amazon-ion/ion-java/issues/685
+                try {
+                    size = _reader.getIntegerSize();
+                } catch (NullPointerException e) {
+                    // Possible exception
+                }
+                if (size == null) {
+                    _reportError("Current token (%s) not integer", _currToken);
+                }
                 switch (size) {
                 case INT:
                     return NumberType.INT;

ion/src/test/java/com/fasterxml/jackson/dataformat/ion/fuzz/Fuzz434_65268_65274_NPETest.java

@@ -0,0 +1,58 @@
+package com.fasterxml.jackson.dataformat.ion.fuzz;
+
+import java.io.*;
+
+import org.hamcrest.Matchers;
+import org.junit.Test;
+
+import com.fasterxml.jackson.core.JsonParser;
+import com.fasterxml.jackson.core.JsonToken;
+import com.fasterxml.jackson.core.exc.StreamReadException;
+import com.fasterxml.jackson.databind.ObjectMapper;
+import com.fasterxml.jackson.dataformat.ion.*;
+
+import static org.hamcrest.MatcherAssert.assertThat;
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.fail;
+
+// [dataformats-binary#434
+public class Fuzz434_65268_65274_NPETest
+{
+    @Test
+    public void testFuzz65268() throws Exception {
+        final IonFactory factory =
+            IonFactory.builderForTextualWriters().build();
+        try (InputStream in = getClass().getResourceAsStream("/data/fuzz-65268.ion")) {
+           try (JsonParser p = factory.createParser(in)) {
+               p.nextToken();
+               p.getText();
+               p.nextTextValue();
+               p.getNumberType();
+           }
+           fail("Should not pass (invalid content)");
+       } catch (StreamReadException e) {
+           assertThat(e.getMessage(), Matchers.containsString("not integer"));
+       }
+    }
+
+    @Test
+    public void testFuzz65274() throws Exception {
+        final ObjectMapper MAPPER =
+            IonObjectMapper.builder(
+                    IonFactory.builderForBinaryWriters()
+                        .enable(IonParser.Feature.USE_NATIVE_TYPE_ID)
+                        .build())
+                 .build();
+
+        try (InputStream in = getClass().getResourceAsStream("/data/fuzz-65274.ion")) {
+           byte[] invalid = new byte[in.available()];
+           new DataInputStream(in).readFully(invalid);
+           try (JsonParser p = MAPPER.getFactory().createParser(new ByteArrayInputStream(invalid))) {
+               MAPPER.readTree(p);
+           }
+           fail("Should not pass (invalid content)");
+       } catch (StreamReadException e) {
+           assertThat(e.getMessage(), Matchers.containsString("not integer"));
+       }
+    }
+}

ion/src/test/resources/data/fuzz-65268.ion

@@ -0,0 +1,3 @@
+n
+
+null.int����~���ttttt(�t�740���������n����66}tt