diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2011-10-18 09:25:34 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2011-10-18 13:51:35 -0700 |
commit | 0f8fd43c42f2cc249943be76a4db0d058c16b6dc (patch) | |
tree | 247334d2cb84c610b1bc4471447311afd7ddfaa9 /drivers/usb/gadget/f_midi.c | |
parent | 0889551267e7b6b4d27dc9f726f3d0cb0d976105 (diff) | |
download | op-kernel-dev-0f8fd43c42f2cc249943be76a4db0d058c16b6dc.zip op-kernel-dev-0f8fd43c42f2cc249943be76a4db0d058c16b6dc.tar.gz |
USB: gadget: midi: memory leak in f_midi_bind_config()
There is a small memory leak on the error paths.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb/gadget/f_midi.c')
-rw-r--r-- | drivers/usb/gadget/f_midi.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/usb/gadget/f_midi.c b/drivers/usb/gadget/f_midi.c index ef1bc7a..67b2229 100644 --- a/drivers/usb/gadget/f_midi.c +++ b/drivers/usb/gadget/f_midi.c @@ -951,7 +951,7 @@ int __init f_midi_bind_config(struct usb_configuration *c, struct gmidi_in_port *port = kzalloc(sizeof(*port), GFP_KERNEL); if (!port) { status = -ENOMEM; - goto fail; + goto setup_fail; } port->midi = midi; @@ -989,6 +989,8 @@ int __init f_midi_bind_config(struct usb_configuration *c, return 0; setup_fail: + for (--i; i >= 0; i--) + kfree(midi->in_port[i]); kfree(midi); fail: return status; |