Changeset 21199 in vbox for trunk/src/VBox/Additions/common/VBoxGuest

Timestamp:

Jul 3, 2009 1:23:29 PM (16 years ago)

Author:

vboxsync

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

49502

Message:

VBoxGuest-linux.c,VBoxGuestR3Lib.cpp: Fixed ioctl error check for linux. Made sure no more VBox error status codes are lost in translation (linux only, but the principle is portable to *BSD and Solaris).

File:

• trunk/src/VBox/Additions/common/VBoxGuest/VBoxGuest-linux.c (modified) (4 diffs)

trunk/src/VBox/Additions/common/VBoxGuest/VBoxGuest-linux.c

@@ -330 +330 @@
 {
     bool fTaken = VBoxGuestCommonISR(&g_DevExt);
-    /** @todo  if (vboxDev->irqAckRequest->events &
-     *             VMMDEV_EVENT_MOUSE_POSITION_CHANGED)
-     *             kill_fasync(&vboxDev->async_queue, SIGIO, POLL_IN);
-     */
     return IRQ_RETVAL(fTaken);
 }
@@ -458 +454 @@
     /*
      * Create the release log.
+     * (We do that here instead of common code because we want to log               .
+     * early failures using the LogRel macro.)
      */
     rc = RTLogCreate(&pRelLogger, 0 /* fFlags */, "all",
@@ -647 +645 @@
          * Copy ioctl data and output buffer back to user space.
          */
-        if (RT_LIKELY(!rc))
+        if (RT_SUCCESS(rc))
         {
             rc = 0;
@@ -668 +666 @@
         {
             Log(("vboxguestLinuxIOCtl: pFilp=%p uCmd=%#x ulArg=%p failed, rc=%d\n", pFilp, uCmd, (void *)ulArg, rc));
-            rc = vboxguestLinuxConvertToNegErrno(rc);
+            rc = -rc; Assert(rc > 0); /* Positive returns == negated VBox error status codes. */
         }
     }