From 1ef5af308adaccf41542bbad403d4b3105036f9b Mon Sep 17 00:00:00 2001
From: Derrick Brashear <shadow@dementia.org>
Date: Mon, 18 Jun 2007 18:12:13 +0000
Subject: [PATCH] 
 STABLE14-darwin-afs-syscall-eliminate-needless-vfs-context-hold-20070618

avoid potential deadlock when e.g. SetTokens called before initialization completes


(cherry picked from commit 125e4511a9715a5df7dda265373d97a21177ea21)
---
 src/afs/afs_call.c | 41 ++++++++++++++++++++---------------------
 1 file changed, 20 insertions(+), 21 deletions(-)

diff --git a/src/afs/afs_call.c b/src/afs/afs_call.c
index 567db2c37..98b44dbe1 100644
--- a/src/afs/afs_call.c
+++ b/src/afs/afs_call.c
@@ -498,26 +498,27 @@ afs_syscall_call(parm, parm2, parm3, parm4, parm5, parm6)
 #endif /* AFS_SGI61_ENV */
 
     AFS_STATCNT(afs_syscall_call);
+    if (!afs_suser(
 #ifdef	AFS_SUN5_ENV
-    if (!afs_suser(CRED()) && (parm != AFSOP_GETMTU)
-	&& (parm != AFSOP_GETMASK)) {
-	/* only root can run this code */
-	return (EACCES);
+		   CRED()
 #else
-    if (!afs_suser(NULL) && (parm != AFSOP_GETMTU)
-	&& (parm != AFSOP_GETMASK)) {
+		   NULL
+#endif
+		   ) && (parm != AFSOP_GETMTU) && (parm != AFSOP_GETMASK)) {
 	/* only root can run this code */
+#if defined(AFS_OSF_ENV) || defined(AFS_SUN5_ENV) || defined(KERNEL_HAVE_UERROR)
 #if defined(KERNEL_HAVE_UERROR)
-	setuerror(EACCES);
-	return (EACCES);
+        setuerror(EACCES);
+#endif
+	code = EACCES;
 #else
-#if defined(AFS_OSF_ENV)
-	return EACCES;
-#else /* AFS_OSF_ENV */
-	return EPERM;
-#endif /* AFS_OSF_ENV */
+	code = EPERM;
 #endif
+	AFS_GLOCK();
+#ifdef AFS_DARWIN80_ENV
+	put_vfs_context();
 #endif
+	goto out;
     }
     AFS_GLOCK();
 #ifdef AFS_DARWIN80_ENV
@@ -792,7 +793,7 @@ afs_syscall_call(parm, parm2, parm3, parm4, parm5, parm6)
 	    afs_osi_Sleep(&afs_initState);
 
 #ifdef AFS_DARWIN80_ENV
-    get_vfs_context();
+	get_vfs_context();
 #endif
 	/* do it by inode */
 #ifdef AFS_SGI62_ENV
@@ -800,7 +801,7 @@ afs_syscall_call(parm, parm2, parm3, parm4, parm5, parm6)
 #endif
 	code = afs_InitCacheFile(NULL, ainode);
 #ifdef AFS_DARWIN80_ENV
-    put_vfs_context();
+	put_vfs_context();
 #endif
     } else if (parm == AFSOP_ROOTVOLUME) {
 	/* wait for basic init */
@@ -829,7 +830,7 @@ afs_syscall_call(parm, parm2, parm3, parm4, parm5, parm6)
 	    tbuffer[AFS_SMALLOCSIZ - 1] = '\0';	/* null-terminate the name */
 	    /* We have the cache dir copied in.  Call the cache init routine */
 #ifdef AFS_DARWIN80_ENV
-    get_vfs_context();
+	    get_vfs_context();
 #endif
 	    if (parm == AFSOP_CACHEFILE)
 		code = afs_InitCacheFile(tbuffer, 0);
@@ -840,7 +841,7 @@ afs_syscall_call(parm, parm2, parm3, parm4, parm5, parm6)
 	    else if (parm == AFSOP_CELLINFO)
 		code = afs_InitCellInfo(tbuffer);
 #ifdef AFS_DARWIN80_ENV
-    put_vfs_context();
+	    put_vfs_context();
 #endif
 	}
 	osi_FreeSmallSpace(tbuffer);
@@ -1072,9 +1073,6 @@ afs_syscall_call(parm, parm2, parm3, parm4, parm5, parm6)
 	code = EINVAL;
 
   out:
-#ifdef AFS_DARWIN80_ENV /* to balance the put in afs3_syscall() */
-    get_vfs_context();
-#endif
     AFS_GUNLOCK();
 #ifdef AFS_LINUX20_ENV
     return -code;
@@ -1658,7 +1656,8 @@ Afs_syscall()
     }
 
 #if defined(AFS_DARWIN80_ENV)
-    put_vfs_context();
+    if (uap->syscall != AFSCALL_CALL)
+	put_vfs_context();
 #endif
 #ifdef AFS_LINUX20_ENV
     code = -code;
-- 
2.39.5