From f1842bcb87c84e8862ca521f491215e42ce1ee2a Mon Sep 17 00:00:00 2001 From: Simon Wilkinson Date: Fri, 30 Mar 2012 19:40:30 +0100 Subject: [PATCH] xdr: Len can never be negative The len parameter to xdr_len is unsigned, so can never be negative. Don't bother testing to see if it is negative, so that clang likes us again. Change-Id: Iad5f8b1b2d20ec55846bc1ef7b7499a30956e17f Reviewed-on: http://gerrit.openafs.org/7089 Tested-by: BuildBot Reviewed-by: Derrick Brashear --- src/rx/xdr_mem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/rx/xdr_mem.c b/src/rx/xdr_mem.c index 94e618bb5..ae849f521 100644 --- a/src/rx/xdr_mem.c +++ b/src/rx/xdr_mem.c @@ -171,7 +171,7 @@ xdrmem_inline(XDR *xdrs, u_int len) { afs_int32 *buf = 0; - if (len >= 0 && xdrs->x_handy >= len) { + if (xdrs->x_handy >= len) { xdrs->x_handy -= len; buf = (afs_int32 *) xdrs->x_private; xdrs->x_private += len; -- 2.39.5