diff options
| author | Mikael Magnusson <mikachu@gmail.com> | 2010-10-10 00:51:36 +0200 |
|---|---|---|
| committer | Dana Jansens <danakj@orodu.net> | 2011-01-24 14:19:22 -0500 |
| commit | 74f7f35b59ce395c17fa823660a480dd94c6077c (patch) | |
| tree | a92c733f2a6dd64877855e3dbde04b7c8343f9cc /openbox/actions/if.c | |
| parent | 95535e86237e6c89935eb7563f37f13474c466f2 (diff) | |
Fix small leak in If action option parsing
Diffstat (limited to 'openbox/actions/if.c')
| -rw-r--r-- | openbox/actions/if.c | 17 |
1 files changed, 10 insertions, 7 deletions
diff --git a/openbox/actions/if.c b/openbox/actions/if.c index 28010d38..0e055a98 100644 --- a/openbox/actions/if.c +++ b/openbox/actions/if.c @@ -98,13 +98,16 @@ static gpointer setup_func(xmlNodePtr node) o->decor_on = TRUE; } if ((n = obt_xml_find_node(node, "desktop"))) { - gchar *s = obt_xml_node_string(n); - if (!g_ascii_strcasecmp(s, "current")) - o->desktop_current = TRUE; - if (!g_ascii_strcasecmp(s, "other")) - o->desktop_other = TRUE; - else - o->desktop_number = atoi(s); + gchar *s; + if ((s = obt_xml_node_string(n))) { + if (!g_ascii_strcasecmp(s, "current")) + o->desktop_current = TRUE; + if (!g_ascii_strcasecmp(s, "other")) + o->desktop_other = TRUE; + else + o->desktop_number = atoi(s); + g_free(s); + } } if ((n = obt_xml_find_node(node, "omnipresent"))) { if (obt_xml_node_bool(n)) |