fixed: CID 1248213 Unsigned compared against 0

An unsigned value can never be negative, so this test will always evaluate the same way.
In XFILE::​CZipFile::​Read(void *, unsigned long): An unsigned value can never be less than 0 (CWE-570)

1 files changed, 3 insertions, 3 deletions

diff --git a/xbmc/filesystem/ZipFile.cpp b/xbmc/filesystem/ZipFile.cpp
index 7cc9c1d517..614b20fa5e 100644
--- a/xbmc/filesystem/ZipFile.cpp
+++ b/xbmc/filesystem/ZipFile.cpp
@@ -347,10 +347,10 @@ ssize_t CZipFile::Read(void* lpBuf, size_t uiBufSize)
   {
     if (uiBufSize+m_iFilePos > mZipItem.csize)
       uiBufSize = mZipItem.csize-m_iFilePos;
-    if (uiBufSize < 0)
-    {
+
+    if (uiBufSize == 0)
       return 0; // we are past eof, this shouldn't happen but test anyway
-    }
+
     ssize_t iResult = mFile.Read(lpBuf,uiBufSize);
     if (iResult < 0)
       return -1;