Fixed some memory leaks. Thanks to Vincent Huisman.

1 files changed, 5 insertions, 3 deletions

diff --git a/index.c b/index.c
index 96bb227..5048da4 100644
--- a/index.c
+++ b/index.c
@@ -45,14 +45,16 @@ MENU *create_menu(char **protected) {
 
 void remove_menu(MENU *mymenu) {
 	ITEM **item_list = menu_items(mymenu);
-	int i;
+	int i, cnt;
 	
 	unpost_menu(mymenu);
 	
-	for (i=0;i<item_count(mymenu);i++) {
+	// Docs say: first free menu, then free items and only then the item list, not in any other order
+	cnt = item_count(mymenu);
+	free_menu(mymenu);
+	for (i=0;i<cnt;i++) {
 		free((char *)item_name(item_list[i]));
 		free_item(item_list[i]);
 	}
 	free(item_list);
-	free_menu(mymenu);
 }