Changeset 104553 in vbox for trunk/src/VBox/Runtime

Timestamp:

May 8, 2024 12:58:08 PM (9 months ago)

Author:

vboxsync

Message:

Runtime/common/zip/xarvfs.cpp: Unnecessary NULL pointer checked (already checked in the if right above), and remove redundant if condition, bugref:3409

File:

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

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

@@ -1521 +1521 @@
     if (pCurFile)
         pThis->XarReader.pCurFile = pCurFile = rtZipXarGetNextFileElement(pCurFile, &pThis->XarReader.cCurDepth);
-    else if (!pThis->fEndOfStream)
+    else
     {
         pThis->XarReader.cCurDepth  = 0;
@@ -1974 +1974 @@
                     xml::ElementNode const *pTocElem  = NULL;
                     if (pRootElem && pRootElem->nameEquals("xar"))
-                        pTocElem = pRootElem ? pRootElem->findChildElement("toc") : NULL;
+                        pTocElem = pRootElem->findChildElement("toc");
                     if (pTocElem)
                     {