summaryrefslogtreecommitdiffstats
path: root/drivers/s390/cio
diff options
context:
space:
mode:
authorCornelia Huck <cornelia.huck@de.ibm.com>2007-08-10 14:32:26 +0200
committerMartin Schwidefsky <schwidefsky@de.ibm.com>2007-08-10 14:32:36 +0200
commit5693ce6f9b9f08942e55e3825db014f8b1205772 (patch)
tree6aa4dbe6eb1720f1427af96eb33a98de4c721cf1 /drivers/s390/cio
parentac07860264bd2b18834d3fa3be47032115524cea (diff)
downloadop-kernel-dev-5693ce6f9b9f08942e55e3825db014f8b1205772.zip
op-kernel-dev-5693ce6f9b9f08942e55e3825db014f8b1205772.tar.gz
[S390] cio: avoid memory leak on error in css_alloc_subchannel().
sch->lock has been allocated in cio_validate_subchannel(), it must be freed if cio_modify() fails. Signed-off-by: Cornelia Huck <cornelia.huck@de.ibm.com> Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Diffstat (limited to 'drivers/s390/cio')
-rw-r--r--drivers/s390/cio/css.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/s390/cio/css.c b/drivers/s390/cio/css.c
index 1c27a5a..5635e65 100644
--- a/drivers/s390/cio/css.c
+++ b/drivers/s390/cio/css.c
@@ -79,6 +79,7 @@ css_alloc_subchannel(struct subchannel_id schid)
sch->schib.pmcw.intparm = (__u32)(unsigned long)sch;
ret = cio_modify(sch);
if (ret) {
+ kfree(sch->lock);
kfree(sch);
return ERR_PTR(ret);
}
OpenPOWER on IntegriCloud