xdr: Len can never be negative
authorSimon Wilkinson <sxw@your-file-system.com>
Fri, 30 Mar 2012 18:40:30 +0000 (19:40 +0100)
committerDerrick Brashear <shadow@dementix.org>
Mon, 9 Apr 2012 01:17:40 +0000 (18:17 -0700)
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 <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementix.org>

src/rx/xdr_mem.c

index 94e618b..ae849f5 100644 (file)
@@ -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;