]> git.michaelhowe.org Git - packages/o/openafs.git/commitdiff
Remove extra arguments to afs_syscall_call
authorAndrew Deason <adeason@sinenomine.net>
Tue, 13 Oct 2009 16:20:51 +0000 (11:20 -0500)
committerDerrick Brashear <shadow|account-1000005@unknown>
Tue, 13 Oct 2009 16:55:08 +0000 (09:55 -0700)
Someone appears to have mistaken afs_syscall_call for
afs_syscall_pioctl, and passed rvp and CRED to afs_syscall_call on
solaris (afs_syscall_call always takes 6 arguments; afs_syscall_pioctl
takes various arguments depending on the platform). After
b1ff4a0b1115f5739c0365cc963189b1f931971f, this breaks the client build
on AFS_SUN5_ENV, but only because it added prototypes for
afs_syscall_call; it looks like it's always been wrong, but it was never
noticed.

Reviewed-on: http://gerrit.openafs.org/653
Tested-by: Andrew Deason <adeason@sinenomine.net>
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
src/afs/afs_call.c

index 5330008db265ccac0ae0acf6e48f5c0e6550c9cc..6d029c47baa2f5a44d21f9de03f0be7d21b1656f 100644 (file)
@@ -1722,11 +1722,7 @@ Afs_syscall()
        if (uap->syscall == AFSCALL_CALL) {
           code =
               afs_syscall_call(uap->parm1, uap->parm2, uap->parm3, uap->parm4,
-                               uap->parm5, uap->parm6
-#ifdef AFS_SUN5_ENV
-                               , rvp, CRED()
-#endif
-                  );
+                               uap->parm5, uap->parm6);
        } else if (uap->syscall == AFSCALL_SETPAG) {
 #ifdef AFS_SUN5_ENV
           register proc_t *procp;