commit 866476dd6a901a9657fd3bb7eef561abeed5a79a
author Noel Grandin <noel.grandin@collabora.co.uk> Thu Feb 27 10:11:39 2020 +0200
committer Noel Grandin <noel.grandin@collabora.co.uk> Thu Feb 27 12:58:28 2020 +0100
tree 5c4fa29371a07ff30a41bda2d0b1ba45b4562e8f
parent 417a3d17271a3e27e7c0aa3030d2b382ecd03a42

tdf#130959 crash when opening file with non-existing hyperlinks

regression from
    commit 0ef5c47547bec6319b853326603f3b807407fe78
    sc: rowcol: tdf#50916 convert core/tool

Change-Id: I8cb8fa4c6abf7450386a284a26be0a6d3ab9a623
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/89617
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
(cherry picked from commit 8f90d3bf33fde7759046a13282381bbd710357f1)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/89578
Reviewed-by: Michael Stahl <michael.stahl@cib.de>

sc/source/filter/oox/worksheetbuffer.cxx

diff --git a/sc/source/filter/oox/worksheetbuffer.cxx b/sc/source/filter/oox/worksheetbuffer.cxx
index 5987260..12c2c61 100644
--- a/sc/source/filter/oox/worksheetbuffer.cxx
+++ b/sc/source/filter/oox/worksheetbuffer.cxx
@@ -125,7 +125,7 @@ void WorksheetBuffer::convertSheetNameRef( OUString& sSheetNameRef ) const
            if (nSepPos < sSheetNameRef.getLength() - 1)
            {
                ScRange aRange;
                if ((aRange.ParseAny( sSheetNameRef.copy( nSepPos + 1 ), nullptr,
                if ((aRange.ParseAny( sSheetNameRef.copy( nSepPos + 1 ), &getScDocument(),
                                formula::FormulaGrammar::CONV_XL_R1C1) & ScRefFlags::VALID) == ScRefFlags::ZERO)
                    sSheetNameRef = sSheetNameRef.replaceAt( nSepPos, 1, OUString( '.' ) );
            }