Changeset 104554 in vbox

Timestamp:

May 8, 2024 12:59:09 PM (7 months ago)

Author:

vboxsync

Message:

Runtime/common/zip/pkzipvfs.cpp: Remove redundant condition in if statement, return real status code instead of always VINF_SUCCESS, bugref:3409

File:

• trunk/src/VBox/Runtime/common/zip/pkzipvfs.cpp (modified) (2 diffs)

trunk/src/VBox/Runtime/common/zip/pkzipvfs.cpp

@@ -533 +533 @@
         *pcbCompressed  = rtZipPkzipReaderCompressed(pThis);
     }
-    return VINF_SUCCESS;
+    return rc;
 }
 
@@ -875 +875 @@
     if (!cbToRead)
         return rc;
-    if (   pThis->fPassZipType
-        && cbToRead > 0)
-    {
+    if (pThis->fPassZipType)
+    {
+        Assert(cbToRead);
+
         uint8_t *pu8Buf = (uint8_t*)pvBuf;
         pu8Buf[0] = pThis->enmZipType;