Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 3 Oct 2012 20:39:11 +0000 (UTC)
From:      "Pedro F. Giffuni" <pfg@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r241165 - head/lib/libc/rpc
Message-ID:  <201210032039.q93KdBOr089732@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: pfg
Date: Wed Oct  3 20:39:11 2012
New Revision: 241165
URL: http://svn.freebsd.org/changeset/base/241165

Log:
  rpc: convert all uid and gid variables of the type uid_t and gid_t.
  
  The previous change (based on Solaris) doesn't work properly either
  as the casting only has the effect of quieting the compiler.
  
  Move back to the previous solution but adjust the sizeof()
  type in xdr_array(). This should mostly work (by accident).
  
  Reported by:	bde

Modified:
  head/lib/libc/rpc/authunix_prot.c

Modified: head/lib/libc/rpc/authunix_prot.c
==============================================================================
--- head/lib/libc/rpc/authunix_prot.c	Wed Oct  3 17:20:34 2012	(r241164)
+++ head/lib/libc/rpc/authunix_prot.c	Wed Oct  3 20:39:11 2012	(r241165)
@@ -69,10 +69,10 @@ xdr_authunix_parms(xdrs, p)
 
 	if (xdr_u_long(xdrs, &(p->aup_time)) &&
 	    xdr_string(xdrs, &(p->aup_machname), MAX_MACHINE_NAME) &&
-	    xdr_int(xdrs, (int *) &(p->aup_uid)) &&
-	    xdr_int(xdrs, (int *) &(p->aup_gid)) &&
+	    xdr_u_int(xdrs, &(p->aup_uid)) &&
+	    xdr_u_int(xdrs, &(p->aup_gid)) &&
 	    xdr_array(xdrs, (char **) paup_gids,
-		    &(p->aup_len), NGRPS, sizeof(int), (xdrproc_t)xdr_int) ) {
+	    &(p->aup_len), NGRPS, sizeof(gid_t), (xdrproc_t)xdr_int) ) {
 		return (TRUE);
 	}
 	return (FALSE);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201210032039.q93KdBOr089732>