ofz#41345 Negative-size-param

Change-Id: I7398fc39c84670f455993c80ea6c9defbcf911d4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/125724
Tested-by: Jenkins
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>

1 files changed, 6 insertions, 6 deletions

diff --git a/editeng/source/uno/unotext.cxx b/editeng/source/uno/unotext.cxx
index cae30b9eaa99..fb11be5baf5a 100644
--- a/editeng/source/uno/unotext.cxx
+++ b/editeng/source/uno/unotext.cxx
@@ -1329,13 +1329,13 @@ bool SvxUnoTextRangeBase::IsCollapsed() noexcept
              maSelection.nStartPos  == maSelection.nEndPos );
 }
 
-bool SvxUnoTextRangeBase::GoLeft(sal_Int16 nCount, bool Expand) noexcept
+bool SvxUnoTextRangeBase::GoLeft(sal_Int32 nCount, bool Expand) noexcept
 {
     CheckSelection( maSelection, mpEditSource.get() );
 
     //  #75098# use end position, as in Writer (start is anchor, end is cursor)
-    sal_uInt16 nNewPos = maSelection.nEndPos;
-    sal_Int32  nNewPar = maSelection.nEndPara;
+    sal_Int32 nNewPos = maSelection.nEndPos;
+    sal_Int32 nNewPar = maSelection.nEndPara;
 
     bool bOk = true;
     SvxTextForwarder* pForwarder = nullptr;
@@ -1367,15 +1367,15 @@ bool SvxUnoTextRangeBase::GoLeft(sal_Int16 nCount, bool Expand) noexcept
     return bOk;
 }
 
-bool SvxUnoTextRangeBase::GoRight(sal_Int16 nCount, bool Expand)  noexcept
+bool SvxUnoTextRangeBase::GoRight(sal_Int32 nCount, bool Expand)  noexcept
 {
     SvxTextForwarder* pForwarder = mpEditSource ? mpEditSource->GetTextForwarder() : nullptr;
     if( pForwarder )
     {
         CheckSelection( maSelection, pForwarder );
 
-        sal_Int32 nNewPos = maSelection.nEndPos + nCount; //! Overflow???
-        sal_Int32  nNewPar = maSelection.nEndPara;
+        sal_Int32 nNewPos = maSelection.nEndPos + nCount;
+        sal_Int32 nNewPar = maSelection.nEndPara;
 
         bool bOk = true;
         sal_Int32 nParCount = pForwarder->GetParagraphCount();