Fix a double free when you have a mousebind without a button attribute
authorMikael Magnusson <mikachu@gmail.com>
Mon, 16 Aug 2010 01:08:56 +0000 (03:08 +0200)
committerMikael Magnusson <mikachu@gmail.com>
Thu, 16 Sep 2010 15:24:12 +0000 (17:24 +0200)
Introduced in f307a3feabedd9bcadeaafd0fa8e1b1a60736eb2

openbox/config.c

index 656ad3c..025a683 100644 (file)
@@ -526,8 +526,8 @@ static void parse_mouse(xmlNodePtr node, gpointer d)
                         mouse_bind(buttonstr, cx, mact, action);
                     nact = obt_xml_find_node(nact->next, "action");
                 }
                         mouse_bind(buttonstr, cx, mact, action);
                     nact = obt_xml_find_node(nact->next, "action");
                 }
-            next_nbut:
             g_free(buttonstr);
             g_free(buttonstr);
+            next_nbut:
             nbut = obt_xml_find_node(nbut->next, "mousebind");
             }
         }
             nbut = obt_xml_find_node(nbut->next, "mousebind");
             }
         }