Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=808e48882316dd4a325cd1cc382516945edad77d
Commit:     808e48882316dd4a325cd1cc382516945edad77d
Parent:     1a908c735aed44c8bbed303371202e416813b271
Author:     Michael Ernst <[EMAIL PROTECTED]>
AuthorDate: Sat Jan 26 14:10:51 2008 +0100
Committer:  Martin Schwidefsky <[EMAIL PROTECTED]>
CommitDate: Sat Jan 26 14:11:05 2008 +0100

    [S390] cio: memory leak in cio processing
    
    Allocated kernel memory for locks is not freed in case of subchannel found
    to be invalid.
    
    Signed-off-by: Michael Ernst <[EMAIL PROTECTED]>
    Signed-off-by: Martin Schwidefsky <[EMAIL PROTECTED]>
---
 drivers/s390/cio/cio.c |    7 +++++--
 1 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/drivers/s390/cio/cio.c b/drivers/s390/cio/cio.c
index f16c15c..11f632f 100644
--- a/drivers/s390/cio/cio.c
+++ b/drivers/s390/cio/cio.c
@@ -577,8 +577,11 @@ cio_validate_subchannel (struct subchannel *sch, struct 
subchannel_id schid)
        }
 
        /* Initialization for io subchannels. */
-       if (!css_sch_is_valid(&sch->schib))
-               return -ENODEV;
+       if (!css_sch_is_valid(&sch->schib)) {
+               err = -ENODEV;
+               goto out;
+       }
+
        /* Devno is valid. */
        if (is_blacklisted (sch->schid.ssid, sch->schib.pmcw.dev)) {
                /*
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to