Changeset 85526 in vbox for trunk/src/VBox/HostDrivers/Support

Timestamp:

Jul 29, 2020 3:50:36 PM (4 years ago)

Author:

vboxsync

Message:

SUP: Forgot to initialize SUPLDRSEG::fUnused to zero. bugref:9801

File:

• trunk/src/VBox/HostDrivers/Support/SUPLibLdr.cpp (modified) (3 diffs)

trunk/src/VBox/HostDrivers/Support/SUPLibLdr.cpp

@@ -451 +451 @@
                 {
                     AssertReturn(pArgs->iSegs < pArgs->cSegsAlloc, VERR_INTERNAL_ERROR_5);
-                    pArgs->paSegs[pArgs->iSegs].off   = pArgs->uStartRva;
-                    pArgs->paSegs[pArgs->iSegs].cb    = pArgs->uEndRva - pArgs->uStartRva;
-                    pArgs->paSegs[pArgs->iSegs].fProt = pArgs->fProt;
+                    pArgs->paSegs[pArgs->iSegs].off     = pArgs->uStartRva;
+                    pArgs->paSegs[pArgs->iSegs].cb      = pArgs->uEndRva - pArgs->uStartRva;
+                    pArgs->paSegs[pArgs->iSegs].fProt   = pArgs->fProt;
+                    pArgs->paSegs[pArgs->iSegs].fUnused = 0;
                 }
                 pArgs->iSegs++;
@@ -481 +482 @@
         {
             AssertReturn(pArgs->iSegs < pArgs->cSegsAlloc, VERR_INTERNAL_ERROR_5);
-            pArgs->paSegs[pArgs->iSegs].off   = pArgs->uStartRva;
-            pArgs->paSegs[pArgs->iSegs].cb    = pArgs->uEndRva - pArgs->uStartRva;
-            pArgs->paSegs[pArgs->iSegs].fProt = pArgs->fProt;
+            pArgs->paSegs[pArgs->iSegs].off     = pArgs->uStartRva;
+            pArgs->paSegs[pArgs->iSegs].cb      = pArgs->uEndRva - pArgs->uStartRva;
+            pArgs->paSegs[pArgs->iSegs].fProt   = pArgs->fProt;
+            pArgs->paSegs[pArgs->iSegs].fUnused = 0;
         }
         pArgs->iSegs++;
@@ -598 +600 @@
     if (SegArgs.uEndRva > SegArgs.uStartRva)
     {
-        SegArgs.paSegs[SegArgs.iSegs].off   = SegArgs.uStartRva;
-        SegArgs.paSegs[SegArgs.iSegs].cb    = SegArgs.uEndRva - SegArgs.uStartRva;
-        SegArgs.paSegs[SegArgs.iSegs].fProt = SegArgs.fProt;
+        SegArgs.paSegs[SegArgs.iSegs].off     = SegArgs.uStartRva;
+        SegArgs.paSegs[SegArgs.iSegs].cb      = SegArgs.uEndRva - SegArgs.uStartRva;
+        SegArgs.paSegs[SegArgs.iSegs].fProt   = SegArgs.fProt;
+        SegArgs.paSegs[SegArgs.iSegs].fUnused = 0;
         SegArgs.iSegs++;
     }