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 | |
parent | 0fe62a35903e11fb41b492bd5b0e8e4c751d5c94 (diff) |
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')
-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) | |||
404 | if (tail->iov_len != 0) { | 404 | if (tail->iov_len != 0) { |
405 | if (tail->iov_len > len) { | 405 | if (tail->iov_len > len) { |
406 | char *p = (char *)tail->iov_base + len; | 406 | char *p = (char *)tail->iov_base + len; |
407 | copy = tail->iov_len - len; | 407 | memmove(p, tail->iov_base, tail->iov_len - len); |
408 | memmove(p, tail->iov_base, copy); | ||
409 | } | 408 | } |
410 | /* Copy from the inlined pages into the tail */ | 409 | /* Copy from the inlined pages into the tail */ |
411 | copy = len; | 410 | copy = len; |