Changeset 49410 in vbox for trunk/src/VBox/Devices/VMMDev

Timestamp:

Nov 7, 2013 3:58:54 PM (11 years ago)

Author:

vboxsync

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

90469

Message:

Devices: sanity check

File:

• trunk/src/VBox/Devices/VMMDev/VMMDevHGCM.cpp (modified) (2 diffs)

trunk/src/VBox/Devices/VMMDev/VMMDevHGCM.cpp

@@ -633 +633 @@
 
     Log(("vmmdevHGCMCall: cParms = %d\n", cParms));
+
+    /*
+     * Sane upper limit.
+     */
+    if (cParms > VMMDEV_MAX_VMMDEV_PARMS)
+    {
+        static int s_cRelWarn;
+        if (s_cRelWarn < 50)
+        {
+            s_cRelWarn++;
+            LogRel(("VMMDev: request packet with too many parameters (%d). Refusing operation.\n", cParms));
+            return VERR_NOT_SUPPORTED;
+        }
+    }
 
     /*
@@ -1145 +1159 @@
 
     Log(("vmmdevHGCMCall: cParms = %d\n", cParms));
+
+    /*
+     * Sane upper limit.
+     */
+    if (cParms > VMMDEV_MAX_VMMDEV_PARMS)
+    {
+        static int s_cRelWarn;
+        if (s_cRelWarn < 50)
+        {
+            s_cRelWarn++;
+            LogRel(("VMMDev: request packet with too many parameters (%d). Refusing operation.\n", cParms));
+            return VERR_NOT_SUPPORTED;
+        }
+    }
 
     /*