commit fb130a4982884bf88b6cdf91d9450016e272ddc4
author Tomaž Vajngerl <tomaz.vajngerl@collabora.co.uk> Fri Jul 14 17:08:48 2023 +0900
committer Tomaž Vajngerl <quikee@gmail.com> Fri Jul 14 12:51:47 2023 +0200
tree 9837a881083c5f61db0a90df34242d9de38401be
parent a3d8f4079dc31a0496f86dc2f80eee9ad60415f5

check if pTable is null after FetchTable

Change-Id: Ia1d208a329c46812e4a9e1d1412e2b69befa3d6e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/154435
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>

sc/source/core/data/documen3.cxx

diff --git a/sc/source/core/data/documen3.cxx b/sc/source/core/data/documen3.cxx
index 8fb0e82..27051e7 100644
--- a/sc/source/core/data/documen3.cxx
+++ b/sc/source/core/data/documen3.cxx
@@ -438,7 +438,7 @@ bool ScDocument::IsScenario( SCTAB nTab ) const
void ScDocument::SetScenarioData( SCTAB nTab, const OUString& rComment,
                                        const Color& rColor, ScScenarioFlags nFlags )
{
    if (ScTable* pTable = FetchTable(nTab); pTable->IsScenario())
    if (ScTable* pTable = FetchTable(nTab); pTable && pTable->IsScenario())
    {
        pTable->SetScenarioComment( rComment );
        pTable->SetScenarioColor( rColor );
@@ -469,7 +469,7 @@ bool ScDocument::IsDefaultTabBgColor( SCTAB nTab ) const
void ScDocument::GetScenarioData( SCTAB nTab, OUString& rComment,
                                        Color& rColor, ScScenarioFlags& rFlags ) const
{
    if (const ScTable* pTable = FetchTable(nTab); pTable->IsScenario())
    if (const ScTable* pTable = FetchTable(nTab); pTable && pTable->IsScenario())
    {
        pTable->GetScenarioComment( rComment );
        rColor = pTable->GetScenarioColor();
@@ -479,7 +479,7 @@ void ScDocument::GetScenarioData( SCTAB nTab, OUString& rComment,

void ScDocument::GetScenarioFlags( SCTAB nTab, ScScenarioFlags& rFlags ) const
{
    if (const ScTable* pTable = FetchTable(nTab); pTable->IsScenario())
    if (const ScTable* pTable = FetchTable(nTab); pTable && pTable->IsScenario())
        rFlags = pTable->GetScenarioFlags();
}

@@ -722,7 +722,7 @@ bool ScDocument::HasCalcNotification( SCTAB nTab ) const
void ScDocument::SetCalcNotification( SCTAB nTab )
{
    // set only if not set before
    if (ScTable* pTable = FetchTable(nTab) ; !pTable->GetCalcNotification())
    if (ScTable* pTable = FetchTable(nTab) ; pTable && !pTable->GetCalcNotification())
        pTable->SetCalcNotification(true);
}