Changeset 75169 in vbox for trunk/src/VBox/HostDrivers/VBoxNetAdp

Timestamp:

Oct 30, 2018 6:29:31 AM (7 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

126247

Message:

Net/Win: bugref:9280 bugref:9233 Changed back the order of offset/truncate handling.

File:

• trunk/src/VBox/HostDrivers/VBoxNetAdp/win/VBoxNetAdp-win.cpp (modified) (2 diffs)

trunk/src/VBox/HostDrivers/VBoxNetAdp/win/VBoxNetAdp-win.cpp

@@ -526 +526 @@
         }
 
-        if (cbSrc > cbPacket)
-            cbSrc = cbPacket;
-
+        /* Handle the offset in the current (which is the first for us) MDL */
         if (uOffset)
         {
@@ -539 +537 @@
             else
             {
-                uOffset -= cbSrc;
-                continue;
+                /* This is an invalid MDL chain */
+                vboxNetAdpWinDestroySG(pSG);
+                return NULL;
             }
         }
+
+        /* Do not read the last MDL beyond packet's end */
+        if (cbSrc > cbPacket)
+            cbSrc = cbPacket;
 
         Assert(cSegs < pSG->cSegsAlloc);