Do not insert null node if ParseAsXDPPacket_Config fails

Doing so trips an assert in the debug build but is harmless otherwise.
This appears to be wrong since the beginning of time.

Bug: 709793
Change-Id: I179f6944a3f7b2a89efa65a16078f6f58448ba4b
Reviewed-on: https://pdfium-review.googlesource.com/4390
Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>

1 files changed, 7 insertions, 0 deletions

diff --git a/xfa/fxfa/parser/cxfa_simple_parser_embeddertest.cpp b/xfa/fxfa/parser/cxfa_simple_parser_embeddertest.cpp
index a33169646d..174febf9ee 100644
--- a/xfa/fxfa/parser/cxfa_simple_parser_embeddertest.cpp
+++ b/xfa/fxfa/parser/cxfa_simple_parser_embeddertest.cpp
@@ -13,3 +13,10 @@ TEST_F(CXFASimpleParserEmbeddertest, Bug_216) {
   EXPECT_NE(nullptr, page);
   UnloadPage(page);
 }
+
+TEST_F(CXFASimpleParserEmbeddertest, Bug_709793) {
+  EXPECT_TRUE(OpenDocument("bug_709793.pdf"));
+  FPDF_PAGE page = LoadPage(0);
+  EXPECT_NE(nullptr, page);
+  UnloadPage(page);
+}