summaryrefslogtreecommitdiffstats
path: root/drivers/scsi
diff options
context:
space:
mode:
authorHugh Dickins <hugh@veritas.com>2005-12-02 15:58:09 +0000
committerJames Bottomley <jejb@titanic.(none)>2005-12-02 10:24:22 -0600
commit4d5cda069b3f0a3bbc18576bc15903ed665d0295 (patch)
treee89d75e2db026d7f15a13464a32197d7ecfc8272 /drivers/scsi
parent032c09d76cdb448484859cc84dac4bba8f5f8f14 (diff)
downloadop-kernel-dev-4d5cda069b3f0a3bbc18576bc15903ed665d0295.zip
op-kernel-dev-4d5cda069b3f0a3bbc18576bc15903ed665d0295.tar.gz
[SCSI] sg: fix a bug in st_map_user_pages failure path
sg's st_map_user_pages is modelled on an earlier version of st's sgl_map_user_pages, and has the same bug: if get_user_pages got some but not all of the pages, then those got were released, but the positive res code returned implied that they were still to be freed. Signed-off-by: Hugh Dickins <hugh@veritas.com> Signed-off-by: Douglas Gilbert <dougg@torque.net> Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/scsi')
-rw-r--r--drivers/scsi/sg.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/scsi/sg.c b/drivers/scsi/sg.c
index 6de7ff9..b55c2a8 100644
--- a/drivers/scsi/sg.c
+++ b/drivers/scsi/sg.c
@@ -1860,9 +1860,11 @@ st_map_user_pages(struct scatterlist *sgl, const unsigned int max_pages,
unlock_page(pages[j]); */
res = 0;
out_unmap:
- if (res > 0)
+ if (res > 0) {
for (j=0; j < res; j++)
page_cache_release(pages[j]);
+ res = 0;
+ }
kfree(pages);
return res;
}
OpenPOWER on IntegriCloud