[Bug 543417] New: e2fsprogs-1.41.9-2.2: missing call to fclose
http://bugzilla.novell.com/show_bug.cgi?id=543417 Summary: e2fsprogs-1.41.9-2.2: missing call to fclose Classification: openSUSE Product: openSUSE 11.2 Version: Factory Platform: All OS/Version: openSUSE 11.1 Status: NEW Severity: Normal Priority: P5 - None Component: Basesystem 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 e2fsprogs-1.41.9-2.2 source code file e2fsprogs-1.41.9/misc/ismounted.c function check_mntent_file local variable f I notice the following code if ((f = fopen(mtab_file, "r")) == NULL) return errno; if ((f = setmntent (mtab_file, "r")) == NULL) return errno; I notice that if the first if succeeds, then there is a resource leak. Suggest add call to fclose between the two ifs. if ((f = fopen(mtab_file, "r")) == NULL) return errno; fclose( f); if ((f = setmntent (mtab_file, "r")) == NULL) return errno; -- 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=543417 Marcus Meissner <meissner@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|bnc-team-screening@forge.pr |hvogel@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=543417 User crrodriguez@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=543417#c1 Cristian Rodríguez <crrodriguez@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED CC| |crrodriguez@novell.com Resolution| |DUPLICATE --- Comment #1 from Cristian Rodríguez <crrodriguez@novell.com> 2009-10-05 12:37:11 MDT --- Duplicate of 503008, which is an internal bug, this issue has been fixed. *** This bug has been marked as a duplicate of bug 503008 *** http://bugzilla.novell.com/show_bug.cgi?id=503008 -- 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