summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorLinus Torvalds <torvalds@linux-foundation.org>2012-09-24 16:13:49 -0700
committerLinus Torvalds <torvalds@linux-foundation.org>2012-09-24 16:13:49 -0700
commitbee2d97b2c4583c2471aa65ab9c189a0011f62b3 (patch)
tree74bcd6a17f3db312268b4ad25aa05a1912ff174f /net
parent979570e02981d4a8fc20b3cc8fd651856c98ee9d (diff)
parent5ce765a540f34d1e2005e1210f49f67fdf11e997 (diff)
downloadop-kernel-dev-bee2d97b2c4583c2471aa65ab9c189a0011f62b3.zip
op-kernel-dev-bee2d97b2c4583c2471aa65ab9c189a0011f62b3.tar.gz
Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/sage/ceph-client
Pull two ceph fixes from Sage Weil: "The first fixes a leak in the rbd setup error path, and the second fixes a more serious problem with mismatched kmap/kunmap that surfaced after the recent refactoring work." * 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/sage/ceph-client: libceph: only kunmap kmapped pages rbd: drop dev reference on error in rbd_open()
Diffstat (limited to 'net')
-rw-r--r--net/ceph/messenger.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/net/ceph/messenger.c b/net/ceph/messenger.c
index 24c5eea..159aa8b 100644
--- a/net/ceph/messenger.c
+++ b/net/ceph/messenger.c
@@ -1073,16 +1073,13 @@ static int write_partial_msg_pages(struct ceph_connection *con)
BUG_ON(kaddr == NULL);
base = kaddr + con->out_msg_pos.page_pos + bio_offset;
crc = crc32c(crc, base, len);
+ kunmap(page);
msg->footer.data_crc = cpu_to_le32(crc);
con->out_msg_pos.did_page_crc = true;
}
ret = ceph_tcp_sendpage(con->sock, page,
con->out_msg_pos.page_pos + bio_offset,
len, 1);
-
- if (do_datacrc)
- kunmap(page);
-
if (ret <= 0)
goto out;
OpenPOWER on IntegriCloud