diff options
| author | Pavel Shilovsky <piastry@etersoft.ru> | 2011-10-21 10:14:04 +0400 |
|---|---|---|
| committer | Steve French <smfrench@gmail.com> | 2011-10-21 09:21:04 -0500 |
| commit | a2d6b6cacb4fd4494b4037c01abb1332cefbb37b (patch) | |
| tree | 0357c72de853c8f7760200e617712f8ac2999fb7 /fs/cifs | |
| parent | b957ae9c53d5715a07f8bac644d8ff0a407c7e07 (diff) | |
| download | blackbird-op-linux-a2d6b6cacb4fd4494b4037c01abb1332cefbb37b.tar.gz blackbird-op-linux-a2d6b6cacb4fd4494b4037c01abb1332cefbb37b.zip | |
CIFS: Fix error handling in cifs_readv_complete
In cifs_readv_receive we don't update rdata->result to error value
after kmap'ing a page. We should kunmap the page in the no error
case only.
Signed-off-by: Pavel Shilovsky <piastry@etersoft.ru>
Reviewed-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <smfrench@gmail.com>
Diffstat (limited to 'fs/cifs')
| -rw-r--r-- | fs/cifs/cifssmb.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/cifs/cifssmb.c b/fs/cifs/cifssmb.c index aaad4ce6e6c5..4435b11c41b9 100644 --- a/fs/cifs/cifssmb.c +++ b/fs/cifs/cifssmb.c @@ -1619,9 +1619,9 @@ cifs_readv_complete(struct work_struct *work) list_for_each_entry_safe(page, tpage, &rdata->pages, lru) { list_del(&page->lru); lru_cache_add_file(page); - kunmap(page); if (rdata->result == 0) { + kunmap(page); flush_dcache_page(page); SetPageUptodate(page); } |
