diff options
| author | Cornelia Huck <cornelia.huck@de.ibm.com> | 2007-08-10 12:32:26 (GMT) |
|---|---|---|
| committer | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2007-08-10 12:32:36 (GMT) |
| commit | 5693ce6f9b9f08942e55e3825db014f8b1205772 (patch) | |
| tree | 6aa4dbe6eb1720f1427af96eb33a98de4c721cf1 /drivers/s390 | |
| parent | ac07860264bd2b18834d3fa3be47032115524cea (diff) | |
| download | linux-5693ce6f9b9f08942e55e3825db014f8b1205772.tar.xz | |
[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')
| -rw-r--r-- | drivers/s390/cio/css.c | 1 |
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); } |