[Libreoffice-commits] core.git: sw/qa

[OmkarAcharekar (via logerrit)]

 sw/qa/filter/ww8/data/null-pointer-dereference.doc |binary
 sw/qa/filter/ww8/ww8.cxx                           |    9 +++++++++
 2 files changed, 9 insertions(+)

New commits:
commit 1278ffc6e9d999bdfba7dbc582dfe8e7b5d6170a
Author:     OmkarAcharekar <omkarachareka...@gmail.com>
AuthorDate: Tue Oct 10 10:31:06 2023 +0530
Commit:     Ilmari Lauhakangas <ilmari.lauhakan...@libreoffice.org>
CommitDate: Tue Oct 10 13:11:32 2023 +0200

    tdf#107786: sw_ww8: Add test for null pointer dereference
    
    Change-Id: I54bd01ce3b0007abe9adb58c0b17195e38e8ceaf
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/157742
    Tested-by: Jenkins
    Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>
    Tested-by: Ilmari Lauhakangas <ilmari.lauhakan...@libreoffice.org>
    Reviewed-by: Ilmari Lauhakangas <ilmari.lauhakan...@libreoffice.org>

diff --git a/sw/qa/filter/ww8/data/null-pointer-dereference.doc 
b/sw/qa/filter/ww8/data/null-pointer-dereference.doc
new file mode 100755
index 000000000000..c2bc0a974477
Binary files /dev/null and b/sw/qa/filter/ww8/data/null-pointer-dereference.doc 
differ
diff --git a/sw/qa/filter/ww8/ww8.cxx b/sw/qa/filter/ww8/ww8.cxx
index 03c922a4c3f4..6a858193e776 100644
--- a/sw/qa/filter/ww8/ww8.cxx
+++ b/sw/qa/filter/ww8/ww8.cxx
@@ -525,6 +525,15 @@ CPPUNIT_TEST_FIXTURE(Test, testSplitFlyInInlineTableDOC)
         CPPUNIT_ASSERT(!pTab->GetFollow());
     }
 }
+
+CPPUNIT_TEST_FIXTURE(Test, testNullPointerDereference)
+{
+    // Given a document with multiple pages:
+    // When loading that document:
+    // Without the accompanying fix in place, this test would have crashed due 
to null pointer access
+    createSwDoc("null-pointer-dereference.doc");
+    CPPUNIT_ASSERT_EQUAL(6, getPages());
+}
 }
 
 CPPUNIT_PLUGIN_IMPLEMENT();