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

cid#1607313 silence Overflowed return value

Browse Source 
and

cid#1608099 Overflowed constant

Change-Id: I10e08c1184bb6630b849c3e24a0c9b26302fd18a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/173896
Tested-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
This commit is contained in:
Caolán McNamara 2024-09-24 19:45:58 +01:00
parent 695a46783e
commit c30fa95c5d
2 changed files with 7 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
hwpfilter/source/hcode.cxx
View File

@ -522,19 +522,16 @@ static int KsSearch(hchar c)
static hchar cdkssm2ks_han(hchar kssm) static hchar cdkssm2ks_han(hchar kssm)
{ {
unsigned int index;
unsigned char lo, hi;
/* "One" */ /* "One" */
if (kssm == 0xd3c5) if (kssm == 0xd3c5)
return 0xc7d1; return 0xc7d1;
index = KsSearch(kssm); unsigned int index = KsSearch(kssm);
if (kssm != ksTbl[index]) if (kssm != ksTbl[index])
return jaso2ks(kssm); return jaso2ks(kssm);
hi = sal::static_int_cast<char>(index / (0xFE - 0xA1 + 1) + 0xB0); unsigned char hi(index / (0xFE - 0xA1 + 1) + 0xB0);
lo = sal::static_int_cast<unsigned char>(index % (0xFE - 0xA1 + 1) + 0xA1); unsigned char lo(index % (0xFE - 0xA1 + 1) + 0xA1);
return lo | (hi << 8); return hchar(lo | (hi << 8));
} }

8
sd/source/core/stlpool.cxx
View File

@ -978,12 +978,10 @@ void SdStyleSheetPool::UpdateStdNames()
if( bHelpKnown ) if( bHelpKnown )
{ {
OUString aNewName; OUString aNewName;
if (pNameId) if (pNameId && pNameId == STR_PSEUDOSHEET_OUTLINE)
{ {
if (pNameId == STR_PSEUDOSHEET_OUTLINE) assert(nHelpId >= HID_PSEUDOSHEET_OUTLINE1 && nHelpId <= HID_PSEUDOSHEET_OUTLINE9);
{ aNewName += " " + OUString::number( sal_Int32( nHelpId - HID_PSEUDOSHEET_OUTLINE ) );
aNewName += " " + OUString::number( sal_Int32( nHelpId - HID_PSEUDOSHEET_OUTLINE ) );
}
} }
if( !aNewName.isEmpty() && aNewName != aOldName ) if( !aNewName.isEmpty() && aNewName != aOldName )