mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

cid#1607535 silence Overflowed constant

Browse Source 
and

cid#1608504 Overflowed constant
cid#1607518 Overflowed constant

Change-Id: Ife45a2f414ea703f627b7083d746bc11f6d4f359
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/173832
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
This commit is contained in:
Caolán McNamara
2024-09-22 20:36:24 +01:00
parent f528d18f99
commit c46c109204
3 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
dbaccess/source/ui/querydesign/SelectionBrowseBox.cxx
View File

@@ -1618,7 +1618,7 @@ OTableFieldDescRef OSelectionBrowseBox::FindFirstFreeCol(sal_uInt16& _rColumnPos
for (auto const& field : getFields()) for (auto const& field : getFields())
{ {
++_rColumnPosition; _rColumnPosition = static_cast<sal_uInt16>(_rColumnPosition + 1);
OTableFieldDescRef pEntry = field; OTableFieldDescRef pEntry = field;
if ( pEntry.is() && pEntry->IsEmpty() ) if ( pEntry.is() && pEntry->IsEmpty() )
return pEntry; return pEntry;

2
sw/source/core/doc/DocumentRedlineManager.cxx
View File

@@ -1404,7 +1404,7 @@ DocumentRedlineManager::AppendRedline(SwRangeRedline* pNewRedl, bool const bCall
bool bCompress = false; bool bCompress = false;
SwRedlineTable::size_type n = 0; SwRedlineTable::size_type n = 0;
// look up the first Redline for the starting position // look up the first Redline for the starting position
if( !GetRedline( *pStt, &n ) && n ) if( !GetRedline( *pStt, &n ) && n > 0 )
--n; --n;
const SwRedlineTable::size_type nStartPos = n; const SwRedlineTable::size_type nStartPos = n;
bool bDec = false; bool bDec = false;

10
sw/source/core/doc/ftnidx.cxx
View File

@@ -135,8 +135,12 @@ void SwFootnoteIdxs::UpdateFootnote( const SwNode& rStt )
{ {
// Step forward until the Index is not the same anymore // Step forward until the Index is not the same anymore
const SwNode* pCmpNd = &rStt; const SwNode* pCmpNd = &rStt;
while( nPos && pCmpNd == &((*this)[ --nPos ]->GetTextNode()) ) while (nPos > 0)
; {
--nPos;
if (pCmpNd != &((*this)[nPos]->GetTextNode()))
break;
}
++nPos; ++nPos;
} }
@@ -146,7 +150,7 @@ void SwFootnoteIdxs::UpdateFootnote( const SwNode& rStt )
if( rOutlNds.empty() ) if( rOutlNds.empty() )
{ {
nFootnoteNo = nPos+1; nFootnoteNo = nPos+1;
if (nPos) if (nPos > 0)
{ {
nFootnoteNoHidden = (*this)[nPos - 1]->GetFootnote().GetNumberRLHidden() + 1; nFootnoteNoHidden = (*this)[nPos - 1]->GetFootnote().GetNumberRLHidden() + 1;
} }