[Bug 542345] New: sax2-8.1-582.4: memory leak
http://bugzilla.novell.com/show_bug.cgi?id=542345 Summary: sax2-8.1-582.4: 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 sax2-8.1-582.4 For source code file sax/sysp/lib/hw/keyboard.c around line 30, function KeyboardGetData is the following code hd_data = (hd_data_t*)calloc(1, sizeof *hd_data); but I fail to notice a matching call to free. This looks like a memory leak to me. Suggest add call to free. -- 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=542345 Marcus Meissner <meissner@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|bnc-team-screening@forge.pr |ms@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=542345 User ms@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=542345#c1 Marcus Schaefer <ms@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|ms@novell.com |sax2-maintainer-bugs@forge. | |provo.novell.com --- Comment #1 from Marcus Schaefer <ms@novell.com> 2009-09-28 03:21:55 MDT --- assigned to maintainer -- 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=542345 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |sndirsch@novell.com Component|Other |SaX2 AssignedTo|sax2-maintainer-bugs@forge. |bnc-team-xorg-bugs@forge.pr |provo.novell.com |ovo.novell.com QAContact|qa@suse.de |sax2-maintainer-bugs@forge. | |provo.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=542345 User sndirsch@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=542345#c2 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Info Provider| |dcb314@hotmail.com --- Comment #2 from Stefan Dirsch <sndirsch@novell.com> 2009-09-28 03:55:48 MDT --- Thanks. How did you find that? -- 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=542345 User dcb314@hotmail.com added comment http://bugzilla.novell.com/show_bug.cgi?id=542345#c3 David Binderman <dcb314@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW Info Provider|dcb314@hotmail.com | --- Comment #3 from David Binderman <dcb314@hotmail.com> 2009-09-28 04:12:11 MDT --- (In reply to comment #2)
Thanks. How did you find that?
Pattern matching, basically. Places where there is a call to malloc or calloc and no matching call to free in the same function. -- 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=542345 User sndirsch@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=542345#c4 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Info Provider| |dcb314@hotmail.com --- Comment #4 from Stefan Dirsch <sndirsch@novell.com> 2009-09-28 04:48:55 MDT --- So you grep for *alloc and verify if there is an appropriate free if variable is no longer in use after returning from the function? So you don't use any tool for that? Is this correct? -- 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=542345 User sndirsch@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=542345#c5 Stefan Dirsch <sndirsch@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |RESOLVED Info Provider|dcb314@hotmail.com | Resolution| |FIXED --- Comment #5 from Stefan Dirsch <sndirsch@novell.com> 2009-10-01 09:44:07 MDT --- fixed. -- 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.
participants (1)
-
bugzilla_noreply＠novell.com