Fix some memleaks.

missing RrFontClose for osd_font in two places.
missing a g_free(area);

diff --git a/openbox/config.c b/openbox/config.c
index 56e625d4c3d9c712cc25bc447c6e59e0d8e1c8ca..50f6aef6d4fd8ec16ab45fffdb390561b31c3283 100644 (file)
--- a/openbox/config.c
+++ b/openbox/config.c
@@ -982,6 +982,7 @@ void config_shutdown(void)
     RrFontClose(config_font_inactivewindow);
     RrFontClose(config_font_menuitem);
     RrFontClose(config_font_menutitle);
+    RrFontClose(config_font_osd);
 
     for (it = config_desktops_names; it; it = g_slist_next(it))
         g_free(it->data);

diff --git a/openbox/popup.c b/openbox/popup.c
index 283348e7cbd3b2ef058c2c6262261e5a38a0a47f..15d1bf52bed291cb342b3ffee40263003e407468 100644 (file)
--- a/openbox/popup.c
+++ b/openbox/popup.c
@@ -257,6 +257,8 @@ void popup_delay_show(ObPopup *self, gulong usec, gchar *text)
     x=MAX(MIN(x, area->x+area->width-w),area->x);
     y=MAX(MIN(y, area->y+area->height-h),area->y);
 
+    g_free(area);
+
     if (m == screen_num_monitors) {
         RECT_SET(mon, x, y, w, h);
         m = screen_find_monitor(&mon);
@@ -266,6 +268,8 @@ void popup_delay_show(ObPopup *self, gulong usec, gchar *text)
 
         x=MAX(MIN(x, area->x+area->width-w),area->x);
         y=MAX(MIN(y, area->y+area->height-h),area->y);
+
+        g_free(area);
     }
 
     /* set the windows/appearances up */
@@ -301,8 +305,6 @@ void popup_delay_show(ObPopup *self, gulong usec, gchar *text)
             popup_show_timeout(self);
         }
     }
-
-    g_free(area);
 }
 
 void popup_hide(ObPopup *self)

diff --git a/render/theme.c b/render/theme.c
index 3d357e385703b4c53a5c5df45c4b3e5f95dec3ba..a57269dd2cbbea98ab130642fb11017b916b4063 100644 (file)
--- a/render/theme.c
+++ b/render/theme.c
@@ -1455,6 +1455,7 @@ void RrThemeFree(RrTheme *theme)
         RrFontClose(theme->win_font_unfocused);
         RrFontClose(theme->menu_title_font);
         RrFontClose(theme->menu_font);
+        RrFontClose(theme->osd_font);
 
         RrAppearanceFree(theme->a_disabled_focused_max);
         RrAppearanceFree(theme->a_disabled_unfocused_max);