Changeset 23062 in vbox for trunk/src/VBox/HostDrivers/VBoxNetFlt

Timestamp:

Sep 16, 2009 11:59:55 AM (15 years ago)

Author:

vboxsync

Message:

Solaris/VBoxNetFlt: paranoia.

File:

• trunk/src/VBox/HostDrivers/VBoxNetFlt/solaris/VBoxNetFlt-solaris.c (modified) (2 diffs)

trunk/src/VBox/HostDrivers/VBoxNetFlt/solaris/VBoxNetFlt-solaris.c

@@ -1426 +1426 @@
     vnode_t *pVNodeHeld = NULL;
     rc = lookupname(pszDev, UIO_SYSSPACE, FOLLOW, NULLVPP, &pVNodeHeld);
-    if (!rc)
+    if (   !rc
+        && pVNodeHeld)
     {
         TIUSER *pUser;
@@ -1432 +1433 @@
         if (!rc)
         {
-            *ppVNode = pUser->fp->f_vnode;
-            *ppVNodeHeld = pVNodeHeld;
-            *ppUser = pUser;
-            return VINF_SUCCESS;
-        }
+            if (   pUser
+                && pUser->fp
+                && pUser->fp->f_vnode)
+            {
+                *ppVNode = pUser->fp->f_vnode;
+                *ppVNodeHeld = pVNodeHeld;
+                *ppUser = pUser;
+                return VINF_SUCCESS;
+            }
+            else
+            {
+                LogRel((DEVICE_NAME ":vboxNetFltSolarisOpenDev failed. pUser=%p fp=%p f_vnode=%p\n", pUser, pUser ? pUser->fp : NULL,
+                                    pUser && pUser->fp ? pUser->fp->f_vnode : NULL));
+            }
+
+            if (pUser)
+                t_kclose(pUser, 0);
+        }
+        else
+            LogRel((DEVICE_NAME ":vboxNetFltSolarisOpenDev t_kopen failed. rc=%d\n", rc));
+
         VN_RELE(pVNodeHeld);
     }
+    else
+        LogRel((DEVICE_NAME ":vboxNetFltSolarisOpenDev lookupname failed. rc=%d pVNodeHeld=%p\n", rc, pVNodeHeld));
+
     return VERR_PATH_NOT_FOUND;
 }