diff options
| author | Yehuda Sadeh <yehuda@hq.newdream.net> | 2010-09-03 21:57:11 +0200 |
|---|---|---|
| committer | Sage Weil <sage@newdream.net> | 2010-09-11 19:52:47 +0200 |
| commit | 3d4401d9d0aef5c40706350685ddea3df6708496 (patch) | |
| tree | 91f92cdc2bb440ffd1baff6bf3efdc2b83855174 /fs | |
| parent | ceph: fix null pointer deref on anon root dentry release (diff) | |
| download | linux-3d4401d9d0aef5c40706350685ddea3df6708496.tar.xz linux-3d4401d9d0aef5c40706350685ddea3df6708496.zip | |
ceph: fix pagelist kunmap tail
A wrong parameter was passed to the kunmap.
Signed-off-by: Yehuda Sadeh <yehuda@hq.newdream.net>
Signed-off-by: Sage Weil <sage@newdream.net>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/ceph/pagelist.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/fs/ceph/pagelist.c b/fs/ceph/pagelist.c index b6859f47d364..46a368b6dce5 100644 --- a/fs/ceph/pagelist.c +++ b/fs/ceph/pagelist.c @@ -5,10 +5,18 @@ #include "pagelist.h" +static void ceph_pagelist_unmap_tail(struct ceph_pagelist *pl) +{ + struct page *page = list_entry(pl->head.prev, struct page, + lru); + kunmap(page); +} + int ceph_pagelist_release(struct ceph_pagelist *pl) { if (pl->mapped_tail) - kunmap(pl->mapped_tail); + ceph_pagelist_unmap_tail(pl); + while (!list_empty(&pl->head)) { struct page *page = list_first_entry(&pl->head, struct page, lru); @@ -26,7 +34,7 @@ static int ceph_pagelist_addpage(struct ceph_pagelist *pl) pl->room += PAGE_SIZE; list_add_tail(&page->lru, &pl->head); if (pl->mapped_tail) - kunmap(pl->mapped_tail); + ceph_pagelist_unmap_tail(pl); pl->mapped_tail = kmap(page); return 0; } |