[Libreoffice-commits] core.git: ucb/source

[Stephan Bergmann (via logerrit)]

 ucb/source/ucp/ftp/ftpurl.cxx |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

New commits:
commit a8d01742e98d29a57be6d990b7331f789bc3a5ee
Author:     Stephan Bergmann <sberg...@redhat.com>
AuthorDate: Tue Apr 26 14:39:28 2022 +0200
Commit:     Stephan Bergmann <sberg...@redhat.com>
CommitDate: Tue Apr 26 17:08:19 2022 +0200

    Improve some signed vs. unsigned comparisons
    
    p2-fwd will always be non-negative
    
    Change-Id: I143d7fd3c5ec192ffaeb56bc2e9f45ac5e83cb6f
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/133439
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <sberg...@redhat.com>

diff --git a/ucb/source/ucp/ftp/ftpurl.cxx b/ucb/source/ucp/ftp/ftpurl.cxx
index 7910a293d570..13b7e839c3e9 100644
--- a/ucb/source/ucp/ftp/ftpurl.cxx
+++ b/ucb/source/ucp/ftp/ftpurl.cxx
@@ -30,6 +30,7 @@
 #include <com/sun/star/ucb/OpenMode.hpp>
 #include <string.h>
 #include <rtl/uri.hxx>
+#include <o3tl/safeint.hxx>
 
 #include "ftpurl.hxx"
 #include "ftpcontentprovider.hxx"
@@ -458,8 +459,8 @@ std::vector<FTPDirentry> FTPURL::list(
     OUString viewurl(ident(true,false));
 
     while(true) {
-        while(p2-fwd < int(len) && *p2 != '\n') ++p2;
-        if(p2-fwd == int(len)) break;
+        while(o3tl::make_unsigned(p2-fwd) < len && *p2 != '\n') ++p2;
+        if(o3tl::make_unsigned(p2-fwd) == len) break;
 
         *p2 = 0;
         switch(osKind) {