From 0dd4584157a32dddeae93cec43467889d850d011 Mon Sep 17 00:00:00 2001
From: "Jasper St. Pierre" <jstpierre@mecheye.net>
Date: Tue, 18 Oct 2011 21:52:46 -0400
Subject: [PATCH] st-texture-cache: Rearrange code to prevent some work and a
 memory leak

For some reason, the texture cache decides to make a request and then look up
an icon in the icon theme. If it's valid, it just returns, fine, but if it
doesn't add the icon, it tries to undo the request, leaking an
AsyncTextureLoadData that isn't freed in the process.

https://bugzilla.gnome.org/show_bug.cgi?id=660968
---
 src/st/st-texture-cache.c | 38 +++++++++++++++++---------------------
 1 file changed, 17 insertions(+), 21 deletions(-)

diff --git a/src/st/st-texture-cache.c b/src/st/st-texture-cache.c
index 398e90a5b..dbdbe496e 100644
--- a/src/st/st-texture-cache.c
+++ b/src/st/st-texture-cache.c
@@ -1053,6 +1053,18 @@ load_gicon_with_colors (StTextureCache    *cache,
   GtkIconInfo *info;
   StTextureCachePolicy policy;
 
+  /* Do theme lookups in the main thread to avoid thread-unsafety */
+  theme = cache->priv->icon_theme;
+
+  info = gtk_icon_theme_lookup_by_gicon (theme, icon, size, GTK_ICON_LOOKUP_USE_BUILTIN);
+  if (info == NULL)
+    {
+      /* gah, the icon doesn't exist. Return a blank texture that will never load */
+      texture = CLUTTER_ACTOR (create_default_texture (cache));
+      clutter_actor_set_size (texture, size, size);
+      return texture;
+    }
+
   gicon_string = g_icon_to_string (icon);
   /* A return value of NULL indicates that the icon can not be serialized,
    * so don't have a unique identifier for it as a cache key, and thus can't
@@ -1079,18 +1091,15 @@ load_gicon_with_colors (StTextureCache    *cache,
 
   if (create_texture_and_ensure_request (cache, key, size, policy, &request, &texture))
     {
+      /* If there's an outstanding request, we've just added ourselves to it */
       g_free (key);
-      return texture;
     }
-
-  /* Do theme lookups in the main thread to avoid thread-unsafety */
-  theme = cache->priv->icon_theme;
-
-  info = gtk_icon_theme_lookup_by_gicon (theme, icon, size, GTK_ICON_LOOKUP_USE_BUILTIN);
-  if (info != NULL)
+  else
     {
-      /* Transfer ownership of key */
+      /* Else, make a new request */
+
       request->cache = cache;
+      /* Transfer ownership of key */
       request->key = key;
       request->policy = policy;
       request->icon = g_object_ref (icon);
@@ -1100,19 +1109,6 @@ load_gicon_with_colors (StTextureCache    *cache,
 
       load_texture_async (cache, request);
     }
-  else
-    {
-      /* Blah; we failed to find the icon, but we've added our texture to the outstanding
-       * requests.  In that case, just undo what create_texture_and_ensure_request() did.
-       */
-       g_slist_foreach (request->textures, (GFunc) g_object_unref, NULL);
-       g_slist_free (request->textures);
-       g_free (request);
-       g_hash_table_remove (cache->priv->outstanding_requests, key);
-       g_free (key);
-       g_object_unref (texture);
-       texture = NULL;
-    }
 
   return CLUTTER_ACTOR (texture);
 }