From: Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
Date: Wed, 11 Mar 2009 05:39:03 +0000 (+0000)
Subject: Minor bug fix in XDR code introduced in last minute change before landing.
X-Git-Tag: spl-0.4.3~10
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=6c33eb816268ef3fef9bb8ca9079bb46aad26605;p=spl

Minor bug fix in XDR code introduced in last minute change before landing.

1) Removed xdr_bytesrec typedef which has no consumers.  If we re-add
   it should also probably be xdr_bytesrec_t.
---

diff --git a/include/rpc/xdr.h b/include/rpc/xdr.h
index 5b81654..43919ca 100644
--- a/include/rpc/xdr.h
+++ b/include/rpc/xdr.h
@@ -68,8 +68,6 @@ struct xdr_bytesrec {
 	size_t xc_num_avail;
 };
 
-typedef struct xdr_bytesrec xdr_bytesrec;
-
 /*
  * XDR functions.
  */
diff --git a/module/spl/spl-xdr.c b/module/spl/spl-xdr.c
index feaa59e..16e42c3 100644
--- a/module/spl/spl-xdr.c
+++ b/module/spl/spl-xdr.c
@@ -186,7 +186,7 @@ xdrmem_enc_bytes(XDR *xdrs, caddr_t cp, const uint_t cnt)
 	if (xdrs->x_addr > xdrs->x_addr_end)
 		return FALSE;
 
-	if (xdrs->x_addr_end - xdrs->x_addr > size)
+	if (xdrs->x_addr_end - xdrs->x_addr < size)
 		return FALSE;
 
 	memcpy(xdrs->x_addr, cp, cnt);
@@ -215,7 +215,7 @@ xdrmem_dec_bytes(XDR *xdrs, caddr_t cp, const uint_t cnt)
 	if (xdrs->x_addr > xdrs->x_addr_end)
 		return FALSE;
 
-	if (xdrs->x_addr_end - xdrs->x_addr > size)
+	if (xdrs->x_addr_end - xdrs->x_addr < size)
 		return FALSE;
 
 	memcpy(cp, xdrs->x_addr, cnt);