[desktop-discuss] Vermillion b57 - careful!

[Glynn Foster]

Hey,

Dennis Clarke wrote:
> what a disaster .. it kills files in my directory,  isn't that nice
> 
> whats the offending code ?

Basically the following -

        gconf_startupdir = get_slab_gconf_string (SLAB_USER_STARTUP_DIR_KEY);
        src_filename = g_build_filename (g_get_home_dir (), gconf_startupdir,
                                         ditem_basename, NULL);

        src_uri = gnome_vfs_uri_new (src_filename);

        list = g_list_append (list, src_uri);

        gnome_vfs_xfer_delete_list (
                list, GNOME_VFS_XFER_ERROR_MODE_ABORT,
                GNOME_VFS_XFER_REMOVESOURCE, NULL, NULL);

Which would be fine assuming gconf_startupdir and ditem_basename are not NULL.
In this case, when the library part of 'gnome-main-menu' (an applet that allows
you to have a different multicolumn layout) was used in control-center, the
corresponding gconf schema wasn't ported over, resulting in NULL for the various
lookups in the gconf database.


Glynn