diff options
| author | Jerry Jelinek <jerry.jelinek@joyent.com> | 2017-05-30 12:09:28 +0000 |
|---|---|---|
| committer | Jerry Jelinek <jerry.jelinek@joyent.com> | 2017-05-30 12:09:28 +0000 |
| commit | 539be8d67e75fbd3dbdebfb93ba8ec220779f0dc (patch) | |
| tree | 216317390a6e9baedf41b054985ce3860227bc17 /usr/src/lib | |
| parent | eb851a777c457362e68a9c4505129bd23d595b7a (diff) | |
| parent | 8186848379535a7535e73f719d8d947abf86522d (diff) | |
| download | illumos-joyent-539be8d67e75fbd3dbdebfb93ba8ec220779f0dc.tar.gz | |
[illumos-gate merge]
commit 8186848379535a7535e73f719d8d947abf86522d
8271 loader: Replacing iterating over rootpath by strsep
commit 1a1df1dbef1b5c57672433301a296dfe4311e6db
8247 uts: Remove archaic register keyword from zmod
commit 45681b8b0e59cad83c1547d78e25d4b7f218d635
8238 xdr_callmsg() should clear residual bytes
commit d1aea6f139360e9e7f1504facb24f8521047b15c
7768 Avoid vgatext dependency on agpmaster
commit d3d32e94615bd7b44139c48d97167020d74fa5b0
8262 sadp is neither built nor used
commit b521c41bc9226e79d6dea569da64cfe5def2ea19
8279 socketpair(AF_UNIX, SOCK_DGRAM,...) broken after 7590
commit d0d16abc1e2ce70cfd8468614a702b20e1fc7e78
8194 kmfcfg: case value not in enumerated type
commit 84e6955df14dc7f06618ed336ac0a78ba258feb6
8171 loader: distinguish NFS versus TFTP boot by rootpath
commit d993cb83958d44b76c7c56393113c24811dde5f7
8202 doors man pages contain extra whitespace
5180 door_server_create(3c): Incomplete return type
commit 6cf138763efaab6b5998bc1ed6849459e376684d
8170 update CLDR data to v31
commit dea58231324dba08972c488dcdedbab5c528ac8a
8263 pkgchk has unused -Q flag
Diffstat (limited to 'usr/src/lib')
| -rw-r--r-- | usr/src/lib/libnsl/rpc/rpc_callmsg.c | 37 |
1 files changed, 19 insertions, 18 deletions
diff --git a/usr/src/lib/libnsl/rpc/rpc_callmsg.c b/usr/src/lib/libnsl/rpc/rpc_callmsg.c index 20dcf02e49..3a0e8859f8 100644 --- a/usr/src/lib/libnsl/rpc/rpc_callmsg.c +++ b/usr/src/lib/libnsl/rpc/rpc_callmsg.c @@ -33,8 +33,6 @@ * California. */ -#pragma ident "%Z%%M% %I% %E% SMI" - #include "mt.h" #include <stdlib.h> #include <sys/param.h> @@ -54,14 +52,17 @@ xdr_callmsg(XDR *xdrs, struct rpc_msg *cmsg) struct opaque_auth *oa; if (xdrs->x_op == XDR_ENCODE) { + uint_t credrndup; + uint_t verfrndup; + if (cmsg->rm_call.cb_cred.oa_length > MAX_AUTH_BYTES) return (FALSE); if (cmsg->rm_call.cb_verf.oa_length > MAX_AUTH_BYTES) return (FALSE); - buf = XDR_INLINE(xdrs, 8 * BYTES_PER_XDR_UNIT - + RNDUP(cmsg->rm_call.cb_cred.oa_length) - + 2 * BYTES_PER_XDR_UNIT - + RNDUP(cmsg->rm_call.cb_verf.oa_length)); + credrndup = RNDUP(cmsg->rm_call.cb_cred.oa_length); + verfrndup = RNDUP(cmsg->rm_call.cb_verf.oa_length); + buf = XDR_INLINE(xdrs, 8 * BYTES_PER_XDR_UNIT + + credrndup + 2 * BYTES_PER_XDR_UNIT + verfrndup); if (buf != NULL) { IXDR_PUT_INT32(buf, cmsg->rm_xid); IXDR_PUT_ENUM(buf, cmsg->rm_direction); @@ -78,18 +79,20 @@ xdr_callmsg(XDR *xdrs, struct rpc_msg *cmsg) IXDR_PUT_INT32(buf, oa->oa_length); if (oa->oa_length) { (void) memcpy(buf, oa->oa_base, oa->oa_length); - buf += RNDUP(oa->oa_length) / sizeof (int32_t); + buf += credrndup / BYTES_PER_XDR_UNIT; + if ((credrndup -= oa->oa_length) > 0) + (void) memset((char *)buf - credrndup, + 0, credrndup); } oa = &cmsg->rm_call.cb_verf; IXDR_PUT_ENUM(buf, oa->oa_flavor); IXDR_PUT_INT32(buf, oa->oa_length); if (oa->oa_length) { (void) memcpy(buf, oa->oa_base, oa->oa_length); - /* - * no real need.... - * buf += RNDUP(oa->oa_length) / sizeof - * (int32_t); - */ + buf += verfrndup / BYTES_PER_XDR_UNIT; + if ((verfrndup -= oa->oa_length) > 0) + (void) memset((char *)buf - verfrndup, + 0, verfrndup); } return (TRUE); } @@ -116,9 +119,8 @@ xdr_callmsg(XDR *xdrs, struct rpc_msg *cmsg) if (oa->oa_base == NULL) { oa->oa_base = malloc(oa->oa_length); if (oa->oa_base == NULL) { - syslog(LOG_ERR, - "xdr_callmsg : " - "out of memory."); + syslog(LOG_ERR, "xdr_callmsg : " + "out of memory."); return (FALSE); } } @@ -153,9 +155,8 @@ xdr_callmsg(XDR *xdrs, struct rpc_msg *cmsg) if (oa->oa_base == NULL) { oa->oa_base = malloc(oa->oa_length); if (oa->oa_base == NULL) { - syslog(LOG_ERR, - "xdr_callmsg : " - "out of memory."); + syslog(LOG_ERR, "xdr_callmsg : " + "out of memory."); return (FALSE); } } |