Lerking/gtk
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

recent-manager: Add a limit to the list's size

Browse Source 
This fixes a DOS where any app can cause all running gtk apps
to use arbitrary amounts of memory.

Originally reported against mate-panel, where running a big slideshow
in eye-of-mate caused increasing RAM usage in mate-panel.

v2: Hardcode the value
Signed-off-by: Lauri Kasanen <curaga@operamail.com>

https://bugzilla.gnome.org/show_bug.cgi?id=773587
This commit is contained in:
Lauri Kasanen
2016-10-27 16:51:29 +03:00
committed by Matthias Clasen
parent 479372c068
commit a3b2d6a65b
1 changed files with 34 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

34
gtk/gtkrecentmanager.c
View File

@@ -47,6 +47,9 @@
/* return all items by default */
#define DEFAULT_LIMIT -1
/* limit the size of the list */
#define MAX_LIST_SIZE 1000
/* keep in sync with xdgmime */
#define GTK_RECENT_DEFAULT_MIME "application/octet-stream"
@@ -137,6 +140,8 @@ static void gtk_recent_manager_set_filename (GtkRecentManager *manag
const gchar *filename);
static void gtk_recent_manager_clamp_to_age (GtkRecentManager *manager,
gint age);
static void gtk_recent_manager_clamp_to_size (GtkRecentManager *manager,
const gint size);
static void build_recent_items_list (GtkRecentManager *manager);
@@ -414,6 +419,7 @@ gtk_recent_manager_real_changed (GtkRecentManager *manager)
{
GtkSettings *settings = gtk_settings_get_default ();
gint age = 30;
gint max_size = MAX_LIST_SIZE;
g_object_get (G_OBJECT (settings), "gtk-recent-files-max-age", &age, NULL);
if (age > 0)
@@ -423,6 +429,9 @@ gtk_recent_manager_real_changed (GtkRecentManager *manager)
g_bookmark_file_free (priv->recent_items);
priv->recent_items = g_bookmark_file_new ();
}
if (max_size > 0)
gtk_recent_manager_clamp_to_size (manager, max_size);
}
write_error = NULL;
@@ -1585,6 +1594,31 @@ gtk_recent_manager_clamp_to_age (GtkRecentManager *manager,
g_strfreev (uris);
}
static void
gtk_recent_manager_clamp_to_size (GtkRecentManager *manager,
const gint size)
{
GtkRecentManagerPrivate *priv = manager->priv;
gchar **uris;
gsize n_uris, i;
if (G_UNLIKELY (!priv->recent_items) || G_UNLIKELY (size < 0))
return;
uris = g_bookmark_file_get_uris (priv->recent_items, &n_uris);
if (n_uris < size)
return;
for (i = 0; i < n_uris - size; i++)
{
const gchar *uri = uris[i];
g_bookmark_file_remove_item (priv->recent_items, uri, NULL);
}
g_strfreev (uris);
}
/*****************
* GtkRecentInfo *
*****************/