coverity#735370 : signed/unsigned mixup

Change-Id: I98366335430ba5ede4e46e1e9ec6488739e2099a

coverity#735370 : signed/unsigned mixup
Change-Id: I98366335430ba5ede4e46e1e9ec6488739e2099a
4f957dc0 · Norbert Thiebaud · 05a9f91d · 4f957dc0
Kaydet (Commit) 4f957dc0 authored Ock 30, 2014 tarafından Norbert Thiebaud
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

vbacontrol.cxx vbahelper/source/msforms/vbacontrol.cxx +3 -2

No files found.
--- a/vbahelper/source/msforms/vbacontrol.cxx
+++ b/vbahelper/source/msforms/vbacontrol.cxx
@@ -737,9 +737,10 @@ sal_Int32 ScVbaControl::getBackColor() throw (uno::RuntimeException)

 void ScVbaControl::setBackColor( sal_Int32 nBackColor ) throw (uno::RuntimeException)
 {
-    if ( ( nBackColor >= (sal_Int32)0x80000000 ) && ( nBackColor <= (sal_Int32)0x80000018 ) )
+    if ( ( (sal_uInt32)nBackColor >= (sal_uInt32)0x80000000 ) &&
+         ( (sal_uInt32)nBackColor <= (sal_uInt32)0x80000000 + SAL_N_ELEMENTS(nSysCols) ) )
    {
-        nBackColor = nSysCols[ nBackColor - 0x80000000 ];
+        nBackColor = nSysCols[ nBackColor & 0x0FF];
    }
    m_xProps->setPropertyValue( "BackgroundColor" , uno::makeAny( XLRGBToOORGB( nBackColor ) ) );
 }