After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 100682 - Possible leak in configuration handling
Possible leak in configuration handling
Status: RESOLVED DUPLICATE of bug 100008
Product: gnome-vfs
Classification: Deprecated
Component: File operations
cvs (head)
Other Linux
: Normal normal
: ---
Assigned To: gnome-vfs maintainers
gnome-vfs maintainers
Depends on:
Blocks:
 
 
Reported: 2002-12-08 18:59 UTC by Kjartan Maraas
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch which seems to fix the leak (765 bytes, patch)
2002-12-09 09:09 UTC, Christophe Fergeau 		none Details | Review

Description Kjartan Maraas 2002-12-08 18:59:42 UTC 
Found in memprof when running gweather applet:

Leaked 0x8085798 (5 bytes)
        malloc(): /gnome/head/cvs/memprof/memintercept.c:89
        g_malloc(): /gnome/head/cvs/glib/glib/gmem.c:137
        g_strndup(): /gnome/head/cvs/glib/glib/gstrfuncs.c:124
        parse_line():
/gnome/head/cvs/gnome-vfs/libgnomevfs/gnome-vfs-configuration.c:248
        parse_file():
/gnome/head/cvs/gnome-vfs/libgnomevfs/gnome-vfs-configuration.c:341
        configuration_load():
/gnome/head/cvs/gnome-vfs/libgnomevfs/gnome-vfs-configuration.c:388
        gnome_vfs_configuration_init():
/gnome/head/cvs/gnome-vfs/libgnomevfs/gnome-vfs-configuration.c:477
        gnome_vfs_init():
/gnome/head/cvs/gnome-vfs/libgnomevfs/gnome-vfs-init.c:132
        gnome_vfs_post_args_parse():
/gnome/head/cvs/libgnome/libgnome/gnome-init.c:410
        gnome_program_postinit():
/gnome/head/cvs/libgnome/libgnome/gnome-program.c:1632
        gnome_program_initv():
/gnome/head/cvs/libgnome/libgnome/gnome-program.c:1882
        gnome_program_init():
/gnome/head/cvs/libgnome/libgnome/gnome-program.c:1685
        main(): /gnome/head/cvs/gnome-applets/gweather/main.c:57
        __libc_start_main():
/usr/src/build/151052-i686/BUILD/glibc-2.2.4/csu/../sysdeps/generic/libc-start.c:129
        _start()
Comment 1 Christophe Fergeau 2002-12-09 09:09:12 UTC 
Created attachment 12866 [details] [review]
patch which seems to fix the leak
Comment 2 Christophe Fergeau 2002-12-09 09:11:22 UTC 
The CDDA method was defined in cdda-module.conf and in
default-modules.conf, but when we add a method to
method_to_module_patch, we weren't checking if there was already a
method with the same name, hence the leak. kmaraas confirmed on irc
that the leak was gone with this patch applied
Comment 3 Christophe Fergeau 2003-01-20 09:46:25 UTC 
Bleh, there was already a patch for that bug :(

*** This bug has been marked as a duplicate of 100008 ***