[Bug 539096] New: jack-0.116.2-3.10: memory leak

http://bugzilla.novell.com/show_bug.cgi?id=539096 Summary: jack-0.116.2-3.10: memory leak Classification: openSUSE Product: openSUSE 11.2 Version: Factory Platform: All OS/Version: openSUSE 11.1 Status: NEW Severity: Normal Priority: P5 - None Component: Other AssignedTo: bnc-team-screening@forge.provo.novell.com ReportedBy: dcb314@hotmail.com QAContact: qa@suse.de Found By: --- I just had a look at factory package jack-0.116.2-3.10 For source code file jack-audio-connection-kit-0.116.2/drivers/alsa/alsa_driver.c around line 2630, function driver_initialize, is the following code midi_driver_name = strdup (param->value.str); but I fail to find a matching call to free. This looks like a memory leak to me. Suggest add call to free near the end of the routine. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

http://bugzilla.novell.com/show_bug.cgi?id=539096 Marcus Meissner <meissner@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|bnc-team-screening@forge.pr |tiwai@novell.com |ovo.novell.com | -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

http://bugzilla.novell.com/show_bug.cgi?id=539096 User tiwai@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=539096#c1 --- Comment #1 from Takashi Iwai <tiwai@novell.com> 2009-09-22 08:07:35 MDT --- It's a wrong fix. The right fix would be just not to call strdup(). -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

http://bugzilla.novell.com/show_bug.cgi?id=539096 User dcb314@hotmail.com added comment http://bugzilla.novell.com/show_bug.cgi?id=539096#c2 --- Comment #2 from David Binderman <dcb314@hotmail.com> 2009-09-22 10:29:25 MDT --- (In reply to comment #1)
The right fix would be just not to call strdup().
Maybe. Suggest run the non-strdup version through valgrind to check. I am not sure how to make sure the processor reaches the section of code. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

http://bugzilla.novell.com/show_bug.cgi?id=539096 User tiwai@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=539096#c3 Takashi Iwai <tiwai@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #3 from Takashi Iwai <tiwai@novell.com> 2009-09-23 03:49:42 MDT --- Submitted to FACTORY now. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

http://bugzilla.novell.com/show_bug.cgi?id=539096 http://bugzilla.novell.com/show_bug.cgi?id=539096#c4 --- Comment #4 from Bernhard Wiedemann <bwiedemann@suse.com> --- This is an autogenerated message for OBS integration: This bug (539096) was mentioned in https://build.opensuse.org/request/show/20882 Factory / jack -- You are receiving this mail because: You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply@novell.com