Daniel Lowe notes passing null here through to the JNI-INCHI can crash the JVM.

johnmay · egonw · commit 5471faed821d · 2021-12-05T09:27:41.000+01:00
diff --git a/storage/inchi/src/main/java/org/openscience/cdk/inchi/InChIToStructure.java b/storage/inchi/src/main/java/org/openscience/cdk/inchi/InChIToStructure.java
@@ -112,6 +112,8 @@ public class InChIToStructure {
      * @throws CDKException
      */
     protected InChIToStructure(String inchi, IChemObjectBuilder builder) throws CDKException {
+        if (inchi == null)
+            throw new NullPointerException();
         try {
             input = new JniInchiInputInchi(inchi, "");
         } catch (JniInchiException jie) {
@@ -127,6 +129,8 @@ protected InChIToStructure(String inchi, IChemObjectBuilder builder) throws CDKE
      * @throws CDKException
      */
     protected InChIToStructure(String inchi, IChemObjectBuilder builder, String options) throws CDKException {
+        if (inchi == null)
+            throw new NullPointerException();
         try {
             input = new JniInchiInputInchi(inchi, options);
         } catch (JniInchiException jie) {
@@ -142,6 +146,8 @@ protected InChIToStructure(String inchi, IChemObjectBuilder builder, String opti
      * @throws CDKException
      */
     protected InChIToStructure(String inchi, IChemObjectBuilder builder, List<String> options) throws CDKException {
+        if (inchi == null)
+            throw new NullPointerException();
         try {
             input = new JniInchiInputInchi(inchi, options);
         } catch (JniInchiException jie) {
