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

[Aron Budea (via logerrit)]

 sw/qa/extras/ooxmlexport/data/tdf140137.docx             |binary
 sw/qa/extras/ooxmlexport/ooxmlexport16.cxx               |    5 ++++
 writerfilter/source/dmapper/DomainMapperTableManager.cxx |   18 ++++++++++-----
 3 files changed, 18 insertions(+), 5 deletions(-)

New commits:
commit 67d41607ad3b97abbb939a989e491af932e985a7
Author:     Aron Budea <aron.budea at collabora.com>
AuthorDate: Sun Feb 28 22:04:24 2021 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Mon Mar 1 10:18:06 2021 +0100

    tdf#140137 Don't throw exception when w:gridCol is missing "w" attr
    
    2149e924cbc32c370128c5f87a4f55c50c99e6bd added a division-by-zero
    check, which caused Writer to throw an error on the bugdoc.
    Since the file could be loaded fine before, let's return to a
    working version, with the check included.
    
    The cause is the following in document.xml
    (originating from a non-MS generator):
      <w:tblGrid>
          <w:gridCol/>
          <w:gridCol/>
      </w:tblGrid>
    
    Word still splits such tables differently, but that difference
    was always there in Writer.
    
    Change-Id: I6d91a736f460394a76f035298a238c41da201cb3
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/111723
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sw/qa/extras/ooxmlexport/data/tdf140137.docx b/sw/qa/extras/ooxmlexport/data/tdf140137.docx
new file mode 100644
index 000000000000..d03925e75e2c
Binary files /dev/null and b/sw/qa/extras/ooxmlexport/data/tdf140137.docx differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport16.cxx b/sw/qa/extras/ooxmlexport/ooxmlexport16.cxx
index 046721c53f59..29798dcabdec 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport16.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport16.cxx
@@ -147,6 +147,11 @@ DECLARE_OOXMLEXPORT_TEST(testTdf133473_shadowSize, "tdf133473.docx")
     CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(200000), nSize1);
 }
 
+DECLARE_OOXMLEXPORT_TEST(testTdf140137, "tdf140137.docx")
+{
+    // Don't throw exception during load
+}
+
 CPPUNIT_PLUGIN_IMPLEMENT();
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/writerfilter/source/dmapper/DomainMapperTableManager.cxx b/writerfilter/source/dmapper/DomainMapperTableManager.cxx
index da78cb682e5c..a3a76dccbf82 100644
--- a/writerfilter/source/dmapper/DomainMapperTableManager.cxx
+++ b/writerfilter/source/dmapper/DomainMapperTableManager.cxx
@@ -693,17 +693,25 @@ void DomainMapperTableManager::endOfRowAction()
         size_t nWidthsBound = getCurrentGridBefore() + m_nCell.back() - 1;
         if (nWidthsBound)
         {
-            if (nFullWidthRelative == 0)
-                throw o3tl::divide_by_zero();
-
             ::std::vector< sal_uInt32 >::const_iterator aSpansIter = rCurrentSpans.begin();
             for( size_t nBorder = 0; nBorder < nWidthsBound; ++nBorder )
             {
-                double fGridWidth = 0.;
+                double nRelPos, fGridWidth = 0.;
                 for ( sal_Int32 nGridCount = *aSpansIter; nGridCount > 0; --nGridCount )
                     fGridWidth += (*pTableGrid)[nBorderGridIndex++];
 
-                double nRelPos = (fGridWidth * 10000) / nFullWidthRelative;
+                if (fGridWidth == 0.)
+                {
+                    // allow nFullWidthRelative here, with a sane 0.0 result
+                    nRelPos = 0.;
+                }
+                else
+                {
+                    if (nFullWidthRelative == 0)
+                        throw o3tl::divide_by_zero();
+
+                    nRelPos = (fGridWidth * 10000) / nFullWidthRelative;
+                }
 
                 pSeparators[nBorder].Position = rtl::math::round(nRelPos + nLastRelPos);
                 pSeparators[nBorder].IsVisible = true;