From 4ee538c6cd60b3fd7daeaa4e394d2b6a519e7642 Mon Sep 17 00:00:00 2001
From: Johan Malm <jgm323@gmail.com>
Date: Thu, 13 Jun 2024 21:00:44 +0100
Subject: [PATCH] menu: ignore <item> without parent <menu>

...to avoid assert() in item_create() because current_menu is NULL.

Reproduce crash with...

    <?xml version="1.0" encoding="utf-8"?>
    <openbox_menu>
      <item label="foo"/>
    </openbox_menu>
---
 src/menu/menu.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/menu/menu.c b/src/menu/menu.c
index a30fcba2..9342ab28 100644
--- a/src/menu/menu.c
+++ b/src/menu/menu.c
@@ -605,6 +605,11 @@ xml_tree_walk(xmlNode *node, struct server *server)
 			continue;
 		}
 		if (!strcasecmp((char *)n->name, "item")) {
+			if (!current_menu) {
+				wlr_log(WLR_ERROR,
+					"ignoring <item> without parent <menu>");
+				continue;
+			}
 			in_item = true;
 			traverse(n, server);
 			in_item = false;