diff options
author | Caolán McNamara <caolanm@redhat.com> | 2015-01-26 09:34:31 +0000 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2015-01-26 12:14:35 +0000 |
commit | 1b919a83e7d6723ee2c2b8c42d55059e3c3dda5f (patch) | |
tree | 9a004fc31d5768c2d378189dbca05871171c94f2 /sc | |
parent | 5b70a95affa29119bbb0c934bf99ee6755de336e (diff) |
coverity#1266459 Improper use of negative value
Change-Id: I9b924f7a6e5f216e1429259239ac598a10d1d5dd
Diffstat (limited to 'sc')
-rw-r--r-- | sc/source/core/data/document.cxx | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/sc/source/core/data/document.cxx b/sc/source/core/data/document.cxx index ccbaf00ebbe4..f1b03205fe82 100644 --- a/sc/source/core/data/document.cxx +++ b/sc/source/core/data/document.cxx @@ -192,21 +192,22 @@ bool ScDocument::HasTable( SCTAB nTab ) const bool ScDocument::GetName( SCTAB nTab, OUString& rName ) const { if (ValidTab(nTab) && nTab < static_cast<SCTAB>(maTabs.size())) + { if (maTabs[nTab]) { maTabs[nTab]->GetName( rName ); return true; } + } rName.clear(); return false; } OUString ScDocument::GetCopyTabName( SCTAB nTab ) const { - if (nTab < static_cast<SCTAB>(maTabNames.size())) + if (ValidTab(nTab) && nTab < static_cast<SCTAB>(maTabNames.size())) return maTabNames[nTab]; - else - return OUString(); + return OUString(); } bool ScDocument::SetCodeName( SCTAB nTab, const OUString& rName ) |