diff options
author | Benny Halevy <bhalevy@panasas.com> | 2010-08-29 12:13:15 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2010-08-29 12:13:15 -0400 |
commit | 2e29ebb8119e6037133921fac09cc5f9d625b511 (patch) | |
tree | 87797fdf21852b6db83a976e14a52500f06ba17f /net/sunrpc/xdr.c | |
parent | 0fe62a35903e11fb41b492bd5b0e8e4c751d5c94 (diff) | |
download | blackbird-op-linux-2e29ebb8119e6037133921fac09cc5f9d625b511.tar.gz blackbird-op-linux-2e29ebb8119e6037133921fac09cc5f9d625b511.zip |
sunrpc: don't use the copy variable in nested block
to clean up the code "copy" will be set prior to the block
hence it mustn't be used there.
Signed-off-by: Benny Halevy <bhalevy@panasas.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'net/sunrpc/xdr.c')
-rw-r--r-- | net/sunrpc/xdr.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/net/sunrpc/xdr.c b/net/sunrpc/xdr.c index 41be21f7f7b9..42a7ebf2a322 100644 --- a/net/sunrpc/xdr.c +++ b/net/sunrpc/xdr.c @@ -404,8 +404,7 @@ xdr_shrink_pagelen(struct xdr_buf *buf, size_t len) if (tail->iov_len != 0) { if (tail->iov_len > len) { char *p = (char *)tail->iov_base + len; - copy = tail->iov_len - len; - memmove(p, tail->iov_base, copy); + memmove(p, tail->iov_base, tail->iov_len - len); } /* Copy from the inlined pages into the tail */ copy = len; |