fix(test): resolve ASAN heap-buffer-overflow in textedit unit tests

[pengfeixx]

Add stubs for insertMultiTextEx, deleteMultiTextEx and slotCanUndoChanged to prevent unsafe Window pointer access via EditWrapper::window() when no real Window parent exists.

修复单元测试中因 EditWrapper 缺少 Window 父对象导致的堆越界崩溃，
通过 stub 隔离 undo 栈信号链中的 Window 访问路径。

同时修复源码中 slotCanUndoChanged/slotCanRedoChanged 的空指针 解引用风险，使用 dynamic_cast 替代直接调用，增加 m_wrapper 空检查。
修复 Tabbar::eventFilter 事件过滤范围过宽的问题。
修复 Qt6 API 兼容性（QMouseEvent/QHoverEvent 构造函数变更）。
修复 resizeEvent lambda 中的悬空指针风险（使用 QPointer 保护）。
修复 Window::checkBlockShutdown 空标签文本的越界访问。

Log: 修复单元测试ASAN堆越界崩溃及多处空指针/类型安全问题
Influence: 修复后单元测试不再因类型混淆和堆越界崩溃，提升测试稳定性；源码修复消除了 slotCanUndoChanged 等路径中的空指针解引用和悬空指针风险。

[sourcery-ai]

Sorry @pengfeixx, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

[github-actions]

Note

[静态代码检查]

• 敏感词检查失败, 检测到1个文件存在敏感词

详情

{
    "CMakeLists.txt": [
        {
            "line": "    HOMEPAGE_URL \"https://github.com/linuxdeepin/deepin-editor\"",
            "line_number": 8,
            "rule": "S35",
            "reason": "Url link | 9c6a12da6f"
        }
    ]
}

[github-actions]

Note

[静态代码检查]

• 敏感词检查失败, 检测到1个文件存在敏感词

详情

{
    "CMakeLists.txt": [
        {
            "line": "    HOMEPAGE_URL \"https://github.com/linuxdeepin/deepin-editor\"",
            "line_number": 8,
            "rule": "S35",
            "reason": "Url link | 9c6a12da6f"
        }
    ]
}

[deepin-ci-robot]

deepin pr auto review

你好！我是CodeGeeX。我已仔细审查了你提供的Git Diff，本次提交主要涉及Qt6兼容性修复、空指针与生命周期安全增强、逻辑优化以及单元测试更新。

以下是我对本次代码变更在语法逻辑、代码质量、代码性能和代码安全四个方面的详细审查意见及改进建议：

---

一、 语法与逻辑

1. window.cpp 中的逻辑运算符误用（严重）

• 问题：在 slotCanRedoChanged 和 slotCanUndoChanged 中，使用了按位或运算符 | 来判断文件是否修改：

  bool isModified = this->m_wrapper->isTemFile() | (m_pUndoStack->canUndo() || m_pUndoStack->index() != m_lastSaveIndex);

  isTemFile() 返回的是 bool，这里应该使用逻辑或 ||。虽然 bool 转为 int 后 | 和 || 的结果在多数情况下一致，但在 C++ 语义上，按位或用于位掩码操作，用于布尔逻辑是典型的代码坏味道，且容易引发难以排查的逻辑错误。
• 改进：将 | 改为 ||。

  bool isModified = this->m_wrapper->isTemFile() || (m_pUndoStack->canUndo() || m_pUndoStack->index() != m_lastSaveIndex);

2. window.cpp 中的越界风险（严重）

• 问题：在 checkBlockShutdown 中，虽然增加了 isEmpty() 的判断，但后续直接调用了 at(0)：

  if (tabText.isNull() || tabText.isEmpty()) { return false; }
  if (tabText.at(0) == '*') { ... }

  如果 tabText 为空字符串，tabText.isEmpty() 判断为真并返回，不会执行到 at(0)，逻辑上是安全的。但是，代码的防御性不足，如果未来有人调整了逻辑顺序，极易引发越界崩溃。
• 改进：使用 startsWith 语义更清晰且绝对安全：

  if (tabText.isNull() || tabText.isEmpty()) { return false; }
  if (tabText.startsWith('*')) { ... }

3. tabbar.cpp 中的事件过滤逻辑截断（潜在Bug）

• 问题：在 Tabbar::eventFilter 中，为了减少日志打印，增加了提前返回的逻辑：

  if (event->type() != QEvent::MouseButtonPress && ... && event->type() != QEvent::DragMove) {
      return false;
  }

  这段代码在 #if (QT_VERSION >= QT_VERSION_CHECK(6, 0, 0)) 内部。这意味着在 Qt6 环境下，所有非鼠标/拖拽的事件（如 HoverMove、Paint、Resize 等）都会被直接 return false 截断，不再执行原有后续的过滤逻辑。请确认原有的 eventFilter 尾部逻辑是否仅针对拖拽事件，如果还有其他事件的处理逻辑，此处会导致 Qt6 下功能丢失。
• 改进：如果仅仅是为了避免 qDebug() 打印刷屏，建议将 qDebug() 移入到具体的 if 分支内部，而不是通过提前 return 来截断事件流。

4. CMakeLists.txt Release 模式开启测试

• 问题：if (CMAKE_BUILD_TYPE STREQUAL "Debug" OR CMAKE_BUILD_TYPE STREQUAL "Release") 使得 Release 版本也编译 tests 目录。通常测试代码包含大量符号且不希望打包进生产环境。
• 改进：如果是为了 CI 流水线在 Release 下跑测试，建议通过独立的 CMake 选项（如 -DBUILD_TESTING=ON）来控制，而不是硬编码绑定到 CMAKE_BUILD_TYPE。

---

二、 代码质量

1. dtextedit.cpp 中重复的向下转型代码

• 问题：slotCanRedoChanged 和 slotCanUndoChanged 中出现了完全相同的转型和调用逻辑：

  Window *wnd = dynamic_cast<Window *>(m_wrapper->window());
  if (wnd) wnd->updateModifyStatus(m_sFilePath, isModified);

• 改进：违反了 DRY（Don't Repeat Yourself）原则。建议在 EditWrapper 类中封装一个 updateModifyStatus 方法，内部完成 window() 到 Window 的转型和调用，或者提取一个私有函数 TextEdit::notifyModifyStatus。

2. ut_utils.cpp 中的 Lambda 重复构建

• 问题：测试代码中通过 lambda makeXml 动态生成 XML 数据，每次循环都会执行 QString::arg 和 toLatin1()，虽然对性能影响微乎其微，但作为单元测试，数据构造可以更简洁。
• 改进：当前写法可读性已经不错，保持现状也可。若追求极简，可使用 QByteArray::literal，但当前方案无需修改。

3. 版权声明年份格式

• 问题：2019 - 2023 修改为了 2019-2026。未来的年份（2026）出现在版权声明中。
• 改进：通常开源项目的版权年份应截止到代码实际提交的年份（如 2019-2024），除非有特殊合规要求，否则不建议写未来年份。

---

三、 代码性能

1. window.cpp 中的重复 textAt 调用

• 问题：原代码中 m_tabbar->textAt(i) 被调用了两次，你通过引入 tabText 变量优化了这一点，这是一个很好的性能改进，避免了不必要的 Qt 内部数据查找。
• 肯定：这个修改非常好。

2. dtextedit.cpp 中 dynamic_cast 的性能开销

• 问题：dynamic_cast 是运行时类型识别（RTTI），在频繁触发的信号槽（如 slotCanUndoChanged）中调用会有轻微的性能损耗。
• 改进：如果架构允许，建议在对象初始化时将 Window 指针缓存，或者使用 qobject_cast（比 dynamic_cast 稍快，且跨动态库边界安全），亦或是通过信号槽解耦，让 Window 自己去响应 UndoStack 的变化，而不是让 TextEdit 去找 Window。

---

四、 代码安全

1. dtextedit.cpp 中的悬空指针防护（极佳）

• 问题：QTimer::singleShot(0, [this]() 是 Qt 中非常经典的悬空指针陷阱，如果控件在 0ms 内被销毁，this 将变成野指针。
• 肯定：引入 QPointer<TextEdit> guard(this) 并在 Lambda 内检查 guard 是否为空，是非常标准且安全的 Qt 异步编程实践。这个修改极大地提升了稳定性。

2. dtextedit.cpp 中的空指针防护（极佳）

• 问题：多处增加了 if (!m_wrapper) return; 的判断，以及在 textChanged 信号中检查 m_wrapper。
• 肯定：这修复了对象析构期间或异常状态下信号槽触发导致的空指针崩溃问题。

3. ut_window.cpp 中的空指针防护

• 问题：对 m_printDoc 和 m_pPreview 增加了 if 判空后再调用 deleteLater()。
• 肯定：虽然 deleteLater 内部对 nullptr 调用是安全的（源码里有判断），但显式判空是好习惯，避免了不必要的函数调用开销和潜在的误读。

---

总结

本次 Diff 的核心方向非常正确，主要解决了 Qt6 兼容性和多处生命周期/空指针的安全隐患。请务必优先修复 isTemFile() | ... 的逻辑运算符问题，并评估 tabbar.cpp 中事件提前返回是否会截断其他必要事件。其余建议可根据项目长期维护的需求酌情重构。

[github-actions]

Note

[静态代码检查]

• 敏感词检查失败, 检测到1个文件存在敏感词

详情

{
    "CMakeLists.txt": [
        {
            "line": "    HOMEPAGE_URL \"https://github.com/linuxdeepin/deepin-editor\"",
            "line_number": 8,
            "rule": "S35",
            "reason": "Url link | 9c6a12da6f"
        }
    ]
}

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lzwind, pengfeixx

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[pengfeixx]

/merge