PAG numbers are unsigned (and other warnings)
authorSimon Wilkinson <sxw@inf.ed.ac.uk>
Tue, 21 Jul 2009 19:26:34 +0000 (20:26 +0100)
committerRuss Allbery <rra|account-1000002@unknown>
Fri, 21 Aug 2009 23:34:49 +0000 (16:34 -0700)
PAG numbers are represented by afs_uint32. Fix assorted call sites to
reflect this.

While we're here, prototype ktc_newpag, and fix the casts used by
the ktc_curpag() call.

Reviewed-on: http://gerrit.openafs.org/167
Reviewed-by: Russ Allbery <rra@stanford.edu>
Tested-by: Russ Allbery <rra@stanford.edu>

src/afs/afs_osi_pag.c
src/afs/afs_pioctl.c
src/auth/auth.p.h
src/auth/ktc.c

index 879cd1d..3bc4723 100644 (file)
@@ -407,7 +407,8 @@ int
 AddPag(afs_int32 aval, struct AFS_UCRED **credpp)
 #endif
 {
-    afs_int32 newpag, code;
+    afs_int32 code;
+    afs_uint32 newpag;
 
     AFS_STATCNT(AddPag);
 #if defined(AFS_OSF_ENV) || defined(AFS_DARWIN_ENV) || defined(AFS_XBSD_ENV)
index 622a4dd..2ca6d09 100644 (file)
@@ -1551,7 +1551,7 @@ DECL_PIOCTL(PSetTokens)
     i = tcell->cellNum;
     afs_PutCell(tcell, READ_LOCK);
     if (set_parent_pag) {
-       afs_int32 pag;
+       afs_uint32 pag;
 #if defined(AFS_DARWIN_ENV) || defined(AFS_XBSD_ENV)
 #if defined(AFS_DARWIN_ENV)
        struct proc *p = current_proc();        /* XXX */
index 6c00bc8..a1b40fe 100644 (file)
@@ -33,6 +33,10 @@ int ktc_ForgetToken(struct ktc_principal *);
 int ktc_ForgetAllTokens(void);
 afs_uint32 ktc_curpag(void);
 
+#ifdef AFS_KERBEROS_ENV
+int ktc_newpag(void);
+#endif
+
 #ifdef AFS_NT40_ENV
 
 /* Flags for the flag word sent along with a token */
index 3cba192..617c058 100644 (file)
@@ -928,13 +928,13 @@ ktc_curpag(void)
 {
     int code;
     struct ViceIoctl iob;
-    afs_int32 pag;
+    afs_uint32 pag;
 
     /* now setup for the pioctl */
     iob.in = NULL;
     iob.in_size = 0;
-    iob.out = &pag;
-    iob.out_size = sizeof(afs_int32);
+    iob.out = (caddr_t) &pag;
+    iob.out_size = sizeof(afs_uint32);
 
     code = PIOCTL(0, VIOC_GETPAG, &iob, 0);
     if (code < 0) {