From: Marc Dionne <marc.c.dionne@gmail.com>
Date: Wed, 5 Dec 2012 23:23:34 +0000 (-0500)
Subject: cache bypass: fix afs_bypass_copy_page memcpy oops
X-Git-Tag: upstream/1.8.0_pre1^2~1731
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=5b2cc290f8e991b152411168cabd1c9e7f31deb4;p=packages%2Fo%2Fopenafs.git

cache bypass: fix afs_bypass_copy_page memcpy oops

The two memory copy cases were inverted when the code was
refactored, causing an oops when cache bypass is enabled.

Change-Id: I30a9a51b7280883fb9b7bbeebbabe464c139b714
Reviewed-on: http://gerrit.openafs.org/8617
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>
Tested-by: Stephan Wiesand <stephan.wiesand@desy.de>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
---

diff --git a/src/afs/afs_bypasscache.c b/src/afs/afs_bypasscache.c
index 666a83fcb..698839840 100644
--- a/src/afs/afs_bypasscache.c
+++ b/src/afs/afs_bypasscache.c
@@ -320,9 +320,9 @@ afs_bypass_copy_page(bypass_page_t pp, int pageoff, struct iovec *rxiov,
     int dolen;
 
     if (partial)
-	dolen = rxiov[iovno].iov_len - iovoff;
-    else
 	dolen = auio->uio_iov[curiov].iov_len - pageoff;
+    else
+	dolen = rxiov[iovno].iov_len - iovoff;
 
 #if !defined(UKERNEL)
 # if defined(KMAP_ATOMIC_TAKES_NO_KM_TYPE)