From 623185170b89c12205e9e093576b9c71118d3f59 Mon Sep 17 00:00:00 2001
From: Vaxry <vaxry@vaxry.net>
Date: Wed, 25 Feb 2026 23:15:37 +0000
Subject: [PATCH] desktop/popup: avoid crash on null popup child in rechecking

ref #13352
---
 src/desktop/view/Popup.cpp | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/desktop/view/Popup.cpp b/src/desktop/view/Popup.cpp
index 8832e2b3..f6f681d5 100644
--- a/src/desktop/view/Popup.cpp
+++ b/src/desktop/view/Popup.cpp
@@ -405,8 +405,12 @@ void CPopup::recheckChildrenRecursive() {
     for (auto const& c : cpy) {
         if (!c || !c->visible())
             continue;
-        c->onCommit(true);
-        c->recheckChildrenRecursive();
+
+        // keep ref, onCommit can call onDestroy
+        auto x = c.lock();
+
+        x->onCommit(true);
+        x->recheckChildrenRecursive();
     }
 }