* [PATCH] ALSA: usb-midi: fix a memory leak bug
@ 2019-08-07 3:22 Wenwen Wang
2019-08-07 5:31 ` Takashi Iwai
0 siblings, 1 reply; 3+ messages in thread
From: Wenwen Wang @ 2019-08-07 3:22 UTC (permalink / raw)
To: Wenwen Wang
Cc: Clemens Ladisch, Jaroslav Kysela, Takashi Iwai,
moderated list:SOUND, open list
In __snd_usbmidi_create(), a MIDI streaming interface structure is
allocated through kzalloc() and the pointer is saved to 'umidi'. Later on,
the endpoint structures are created by invoking
snd_usbmidi_create_endpoints_midiman() or snd_usbmidi_create_endpoints(),
depending on the type of the audio quirk type. However, if the creation
fails, the allocated 'umidi' is not deallocated, leading to a memory leak
bug.
To fix the above issue, free 'umidi' before returning the error.
Signed-off-by: Wenwen Wang <wenwen@cs.uga.edu>
---
sound/usb/midi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/usb/midi.c b/sound/usb/midi.c
index b737f0e..22db37f 100644
--- a/sound/usb/midi.c
+++ b/sound/usb/midi.c
@@ -2476,7 +2476,7 @@ int __snd_usbmidi_create(struct snd_card *card,
else
err = snd_usbmidi_create_endpoints(umidi, endpoints);
if (err < 0)
- goto exit;
+ goto free_midi;
usb_autopm_get_interface_no_resume(umidi->iface);
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ALSA: usb-midi: fix a memory leak bug
2019-08-07 3:22 [PATCH] ALSA: usb-midi: fix a memory leak bug Wenwen Wang
@ 2019-08-07 5:31 ` Takashi Iwai
2019-08-07 5:37 ` Wenwen Wang
0 siblings, 1 reply; 3+ messages in thread
From: Takashi Iwai @ 2019-08-07 5:31 UTC (permalink / raw)
To: Wenwen Wang
Cc: moderated list:SOUND, Clemens Ladisch, Jaroslav Kysela,
Takashi Iwai, open list
On Wed, 07 Aug 2019 05:22:09 +0200,
Wenwen Wang wrote:
>
> In __snd_usbmidi_create(), a MIDI streaming interface structure is
> allocated through kzalloc() and the pointer is saved to 'umidi'. Later on,
> the endpoint structures are created by invoking
> snd_usbmidi_create_endpoints_midiman() or snd_usbmidi_create_endpoints(),
> depending on the type of the audio quirk type. However, if the creation
> fails, the allocated 'umidi' is not deallocated, leading to a memory leak
> bug.
>
> To fix the above issue, free 'umidi' before returning the error.
>
> Signed-off-by: Wenwen Wang <wenwen@cs.uga.edu>
It's again a false-positive report. The object is released
automatically by the destructor of its base snd_rawmidi object.
thanks,
Takashi
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ALSA: usb-midi: fix a memory leak bug
2019-08-07 5:31 ` Takashi Iwai
@ 2019-08-07 5:37 ` Wenwen Wang
0 siblings, 0 replies; 3+ messages in thread
From: Wenwen Wang @ 2019-08-07 5:37 UTC (permalink / raw)
To: Takashi Iwai
Cc: moderated list:SOUND, Clemens Ladisch, Jaroslav Kysela,
Takashi Iwai, open list, Wenwen Wang
On Wed, Aug 7, 2019 at 1:31 AM Takashi Iwai <tiwai@suse.de> wrote:
>
> On Wed, 07 Aug 2019 05:22:09 +0200,
> Wenwen Wang wrote:
> >
> > In __snd_usbmidi_create(), a MIDI streaming interface structure is
> > allocated through kzalloc() and the pointer is saved to 'umidi'. Later on,
> > the endpoint structures are created by invoking
> > snd_usbmidi_create_endpoints_midiman() or snd_usbmidi_create_endpoints(),
> > depending on the type of the audio quirk type. However, if the creation
> > fails, the allocated 'umidi' is not deallocated, leading to a memory leak
> > bug.
> >
> > To fix the above issue, free 'umidi' before returning the error.
> >
> > Signed-off-by: Wenwen Wang <wenwen@cs.uga.edu>
>
> It's again a false-positive report. The object is released
> automatically by the destructor of its base snd_rawmidi object.
Thanks for your response! Sorry for the false positives. :(
Wenwen
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-08-07 5:38 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-07 3:22 [PATCH] ALSA: usb-midi: fix a memory leak bug Wenwen Wang
2019-08-07 5:31 ` Takashi Iwai
2019-08-07 5:37 ` Wenwen Wang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).