Comment 2 for bug 93050

Revision history for this message
didier (did447-deactivatedaccount) wrote :

Hi,

from the comment in _gnome_vfs_get_current_unix_mounts()
                /* ignore any mnt_fsname that is repeated and begins with a '/'
                 *
                 * We do this to avoid being fooled by --bind mounts, since
                 * these have the same device as the location they bind to.
                 * Its not an ideal solution to the problem, but its likely that
                 * the most important mountpoint is first and the --bind ones after
                 * that aren't as important. So it should work.
                 *
                 * The '/' is to handle procfs, tmpfs and other no device mounts.
                 */
So I'm expecting gnomevfs-df returning only one line.
$ sudo mount -o bind / /mnt/foo
$ gnomevfs-df
/ xxx MB (604188672)
/mnt/foo xxx MB (604188672)

/mnt/foo is a duplicate and shouldn't be there.

At least the trash applet is expecting the same. (the tooltips count is wrong when /home and / are on the same partition because the mount bind on /dev/.static/dev ).

The test is always false with 08_uuid_mount.patch applied when the device path is a symlink (volume mounted by uuid) because it's comparing symlinks with realpath.

I will attached a modified 08_uuid_mount.patch.