From 5060c5015882b7109c54598c4ea858949beafc43 Mon Sep 17 00:00:00 2001
From: Stephan Bergmann <sbergman@redhat.com>
Date: Tue, 24 May 2022 15:26:06 +0200
Subject: Use o3tl::make_unsigned in some places

...where a signed and an unsigned value are compared, and the signed value has
just been proven to be non-negative here

Change-Id: I20600d61a5d59d739bc1bee838c0038e4611aec2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/134875
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
---
 vbahelper/source/msforms/vbacontrols.cxx | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'vbahelper')

diff --git a/vbahelper/source/msforms/vbacontrols.cxx b/vbahelper/source/msforms/vbacontrols.cxx
index 503c75311165..a1dc6b6eea8e 100644
--- a/vbahelper/source/msforms/vbacontrols.cxx
+++ b/vbahelper/source/msforms/vbacontrols.cxx
@@ -32,6 +32,7 @@
 #include "vbacontrol.hxx"
 #include <cppuhelper/exc_hlp.hxx>
 #include <cppuhelper/implbase.hxx>
+#include <o3tl/safeint.hxx>
 #include <unordered_map>
 
 using namespace com::sun::star;
@@ -135,7 +136,7 @@ public:
 
     virtual uno::Any SAL_CALL getByIndex( ::sal_Int32 Index ) override
     {
-        if ( Index < 0 || Index >= static_cast< sal_Int32 >( mControls.size() ) )
+        if ( Index < 0 || o3tl::make_unsigned(Index) >= mControls.size() )
             throw lang::IndexOutOfBoundsException();
         return uno::Any( mControls[ Index ] );
     }
-- 
cgit