From 48bd903bfd23874bd11669bc95230b37906a1654 Mon Sep 17 00:00:00 2001 From: Donat Zenichev Date: Tue, 20 Jan 2026 11:39:26 +0100 Subject: [PATCH] MT#59962 xmlrpc2di: size_t shouldn't be checked against `< 0` Because it's unsigned. Change-Id: I62d6a285119183f3c8c146478e26333b41c8420c --- apps/xmlrpc2di/xmlrpc++/src/XmlRpcSocket.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/xmlrpc2di/xmlrpc++/src/XmlRpcSocket.cpp b/apps/xmlrpc2di/xmlrpc++/src/XmlRpcSocket.cpp index 9500ba3e..2dd84dc3 100644 --- a/apps/xmlrpc2di/xmlrpc++/src/XmlRpcSocket.cpp +++ b/apps/xmlrpc2di/xmlrpc++/src/XmlRpcSocket.cpp @@ -219,7 +219,7 @@ XmlRpcSocket::nbRead(int fd, std::string& s, bool *eof, SSL* ssl) bool XmlRpcSocket::nbWrite(int fd, std::string& s, size_t &bytesSoFar, SSL* ssl) { /* a guard against overflow */ - if (bytesSoFar < 0 || bytesSoFar > s.length()) { + if (bytesSoFar > s.length()) { XmlRpcUtil::log(1, "XmlRpcSocket::nbWrite: Invalid bytesSoFar='%zu' for string length='%zu'", bytesSoFar, s.length()); return false;