feat: Improve safety checks in capture management to prevent crashes

- Added null checks and active connection validations in the TreelandCaptureManager and related classes to ensure safe destruction of objects only when the Wayland connection is active.
- Enhanced the destroy methods in various capture classes to skip destruction if the objects are not initialized, preventing potential crashes during cleanup.
- Updated logging to provide clearer insights into the state of the capture manager and context during destruction processes.

This update enhances the stability and reliability of the capture management system, particularly in Wayland environments.

Author: dengzhongyuan365-dev

src/capture.cpp

@@ -58,10 +58,15 @@ TreelandCaptureManager::~TreelandCaptureManager()
     if (m_context) {
         qCDebug(dsrApp) << "Log: TreelandCaptureManager::~TreelandCaptureManager - Deleting context object.";
         delete m_context;
+        m_context = nullptr;
+    }
+    // 只有在 Wayland 连接仍然有效时才调用 destroy()，避免崩溃
+    if (isActive()) {
+        destroy();
+        // 销毁 Wayland 扩展
+    } else {
+        qCDebug(dsrApp) << "Log: TreelandCaptureManager::~TreelandCaptureManager - Wayland connection not active, skipping destroy";
     }
-    // 删除上下文对象
-    destroy();
-    // 销毁 Wayland 扩展
     qCDebug(dsrApp) << "Exit: TreelandCaptureManager::~TreelandCaptureManager";
 }
 
@@ -115,6 +120,11 @@ void TreelandCaptureContext::treeland_capture_context_v1_source_failed(uint32_t
 
 void TreelandCaptureManager::cancelCapture() {
     qCDebug(dsrApp) << "Entry: TreelandCaptureManager::cancelCapture";
+    // 检查 Wayland 连接是否仍然有效，避免在应用程序退出时崩溃
+    if (!isActive()) {
+        qCDebug(dsrApp) << "Cancel: Wayland connection not active, skipping destroy";
+        return;
+    }
     // 安全地结束当前选择会话：销毁 session，而不是销毁整个 Wayland 扩展
     // 目的：通知合成器退出选择器，避免 destroy() manager/context 引发无效 proxy 崩溃
     if (m_context && m_context->session()) {
@@ -165,13 +175,16 @@ TreelandCaptureContext::~TreelandCaptureContext()
     if (m_frame) {
         qCDebug(dsrApp) << "Log: TreelandCaptureContext::~TreelandCaptureContext - Deleting frame object.";
         delete m_frame;
+        m_frame = nullptr;
     }
     // 删除帧对象
     if (m_session) {
         qCDebug(dsrApp) << "Log: TreelandCaptureContext::~TreelandCaptureContext - Deleting session object.";
         delete m_session;
+        m_session = nullptr;
     }
     // 删除会话对象
+    // destroy() 方法内部已经检查了 isInitialized()，所以可以安全调用
     destroy();
     // 销毁 Wayland 上下文
     qCDebug(dsrApp) << "Exit: TreelandCaptureContext::~TreelandCaptureContext";

src/qwayland-treeland-capture-unstable-v1.cpp

@@ -93,6 +93,11 @@ static inline void *wlRegistryBind(struct ::wl_registry *registry, uint32_t name
     void treeland_capture_session_v1::destroy()
     {
         qCDebug(dsrApp) << "treeland_capture_session_v1 destroy called.";
+        // 检查对象是否已初始化，避免在 Wayland 连接已关闭时调用导致崩溃
+        if (!isInitialized() || !m_treeland_capture_session_v1) {
+            qCDebug(dsrApp) << "treeland_capture_session_v1 destroy skipped: object not initialized or already destroyed";
+            return;
+        }
         ::treeland_capture_session_v1_destroy(
             m_treeland_capture_session_v1);
         m_treeland_capture_session_v1 = nullptr;
@@ -283,6 +288,11 @@ static inline void *wlRegistryBind(struct ::wl_registry *registry, uint32_t name
     void treeland_capture_frame_v1::destroy()
     {
         qCDebug(dsrApp) << "treeland_capture_frame_v1 destroy called.";
+        // 检查对象是否已初始化，避免在 Wayland 连接已关闭时调用导致崩溃
+        if (!isInitialized() || !m_treeland_capture_frame_v1) {
+            qCDebug(dsrApp) << "treeland_capture_frame_v1 destroy skipped: object not initialized or already destroyed";
+            return;
+        }
         ::treeland_capture_frame_v1_destroy(
             m_treeland_capture_frame_v1);
         m_treeland_capture_frame_v1 = nullptr;
@@ -455,6 +465,11 @@ static inline void *wlRegistryBind(struct ::wl_registry *registry, uint32_t name
     void treeland_capture_context_v1::destroy()
     {
         qCDebug(dsrApp) << "treeland_capture_context_v1 destroy called.";
+        // 检查对象是否已初始化，避免在 Wayland 连接已关闭时调用导致崩溃
+        if (!isInitialized() || !m_treeland_capture_context_v1) {
+            qCDebug(dsrApp) << "treeland_capture_context_v1 destroy skipped: object not initialized or already destroyed";
+            return;
+        }
         ::treeland_capture_context_v1_destroy(
             m_treeland_capture_context_v1);
         m_treeland_capture_context_v1 = nullptr;
@@ -600,6 +615,11 @@ static inline void *wlRegistryBind(struct ::wl_registry *registry, uint32_t name
     void treeland_capture_manager_v1::destroy()
     {
         qCDebug(dsrApp) << "treeland_capture_manager_v1 destroy called.";
+        // 检查对象是否已初始化，避免在 Wayland 连接已关闭时调用导致崩溃
+        if (!isInitialized() || !m_treeland_capture_manager_v1) {
+            qCDebug(dsrApp) << "treeland_capture_manager_v1 destroy skipped: object not initialized or already destroyed";
+            return;
+        }
         ::treeland_capture_manager_v1_destroy(
             m_treeland_capture_manager_v1);
         m_treeland_capture_manager_v1 = nullptr;