[Libreoffice-commits] core.git: sc/source

[Caolán McNamara (via logerrit)]

 sc/source/ui/unoobj/chart2uno.cxx |    9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

New commits:
commit d0f12a16145b6831a6cecc017a1d9879efb576cc
Author:     Caolán McNamara <caol...@redhat.com>
AuthorDate: Sun Dec 11 21:26:01 2022 +0000
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Sun Dec 11 23:11:38 2022 +0000

    crashtesting: use of uninitialized data in forum-mso-en4-93354.xls
    
    Change-Id: I678a22289827e0a9c305ac49f7f046ae2a91184c
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/143974
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caol...@redhat.com>

diff --git a/sc/source/ui/unoobj/chart2uno.cxx 
b/sc/source/ui/unoobj/chart2uno.cxx
index 18378bdfedcd..e36fba94d71e 100644
--- a/sc/source/ui/unoobj/chart2uno.cxx
+++ b/sc/source/ui/unoobj/chart2uno.cxx
@@ -532,7 +532,14 @@ void Chart2Positioner::glueState()
     }
 
     ScComplexRefData aData;
-    ScRefTokenHelper::getDoubleRefDataFromToken(aData, mrRefTokens.front());
+    if (!ScRefTokenHelper::getDoubleRefDataFromToken(aData, 
mrRefTokens.front()))
+    {
+        SAL_WARN("sc", "Chart2Positioner::glueState getDoubleRefDataFromToken 
failed");
+        invalidateGlue();
+        mnStartCol = 0;
+        mnStartRow = 0;
+        return;
+    }
     mnStartCol = aData.Ref1.Col();
     mnStartRow = aData.Ref1.Row();