🐛 PDF: Fix NPE for legitimate documents

This was a sad story, that could have been prevented with tests. JUST SAYIN', dear me!
docbleach · Apr 27, 2017 · 7e70a69 · 7e70a69
1 parent 0c3eb66
commit 7e70a69
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/module/module-pdf/src/main/java/xyz/docbleach/module/pdf/PdfBleach.java b/module/module-pdf/src/main/java/xyz/docbleach/module/pdf/PdfBleach.java
@@ -93,6 +93,9 @@ private void sanitize(ScratchFile scratchFile, RandomAccessRead source, OutputSt
     }
 
     private void sanitizeNamed(BleachSession session, PDDocument doc, PDDocumentNameDictionary names) {
+        if (names == null)
+            return;
+
         sanitizeRecursiveNameTree(names.getEmbeddedFiles(), fileSpec -> sanitizeEmbeddedFile(session, doc, fileSpec));
 
         sanitizeRecursiveNameTree(names.getJavaScript(), action -> sanitizeJavascript(session, doc, action));