diff options
| author |   Linus Torvalds <torvalds@linux-foundation.org> | 2012-09-24 16:13:49 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-09-24 16:13:49 -0700 |
| commit | bee2d97b2c4583c2471aa65ab9c189a0011f62b3 (patch) | |
| tree | 74bcd6a17f3db312268b4ad25aa05a1912ff174f /net | |
| parent | Linux 3.6-rc7 (diff) | |
| parent | libceph: only kunmap kmapped pages (diff) | |
| download | linux-dev-bee2d97b2c4583c2471aa65ab9c189a0011f62b3.tar.xz linux-dev-bee2d97b2c4583c2471aa65ab9c189a0011f62b3.zip | |
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.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/net/ceph/messenger.c b/net/ceph/messenger.c index 24c5eea8c45b..159aa8bef9e7 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; |