Avoid displaying mounts in a subdirectory not accessible to the user

2008-04-28  David Zeuthen  <davidz@redhat.com>

        * gunixmounts.c (g_unix_mount_guess_should_display): Avoid
        displaying mounts in a subdirectory not accessible to the
        user (#526320).


svn path=/trunk/; revision=6872
This commit is contained in:
David Zeuthen 2008-04-28 22:38:49 +00:00 committed by David Zeuthen
parent 11f85653a8
commit 27dba4c28b
2 changed files with 24 additions and 1 deletions

View File

@ -1,3 +1,9 @@
2008-04-28 David Zeuthen <davidz@redhat.com>
* gunixmounts.c (g_unix_mount_guess_should_display): Avoid
displaying mounts in a subdirectory not accessible to the
user (#526320).
2008-04-22 Michael Natterer <mitch@imendio.com>
* Makefile.am: fix library versioning (it was 0.0.0).

View File

@ -44,6 +44,7 @@
#include <errno.h>
#include <string.h>
#include <signal.h>
#include <gstdio.h>
#include "gunixmounts.h"
#include "gfile.h"
@ -1887,8 +1888,24 @@ g_unix_mount_guess_should_display (GUnixMountEntry *mount_entry)
mount_path = mount_entry->mount_path;
if (mount_path != NULL)
{
if (g_str_has_prefix (mount_path, "/media/"))
if (g_str_has_prefix (mount_path, "/media/")) {
char *path;
/* Avoid displaying mounts that are not accessible to the user.
*
* See http://bugzilla.gnome.org/show_bug.cgi?id=526320 for why we
* want to avoid g_access() for every mount point.
*/
path = g_path_get_dirname (mount_path);
if (g_str_has_prefix (path, "/media/"))
{
if (g_access (path, R_OK|X_OK) != 0) {
g_free (path);
return FALSE;
}
}
g_free (path);
return TRUE;
}
if (g_str_has_prefix (mount_path, g_get_home_dir ()) && mount_path[strlen (g_get_home_dir())] == G_DIR_SEPARATOR)
return TRUE;