[pppanghu77]

Root Cause Analysis

ZipFileProcessResultWidget::backPage() (export result page "Back" button
handler) emits only changeWidget(choosewidget) to switch pages, but does not
emit clearWidget beforehand. The clearWidget signal is connected to
MainWindowPrivate::clearWidget() (mainwindow_p_win.cpp:139-140), which
calls clear() on all selection widgets. Because the signal is never emitted,
selection state is never reset, leaving stale file selections when the user
returns to the choose page.

Key evidence:

• zipfileprocessresultwidget.cpp:136-138 — backPage() missing clearWidget
• errorwidget.cpp:99-102 — same-module ErrorWidget::backPage() emits
  clearWidget before changeWidget, establishing the correct pattern

Fix

Add emit TransferHelper::instance()->clearWidget(); before the existing
changeWidget emit in backPage(), matching the pattern in
ErrorWidget::backPage() and other back-handlers in the same module.

Change Safety Assessment

Code Safety

• Risk Level: Low
• Single-file, single-function, one-line addition; no signature change, no
  public member change, no caller impact beyond the backButton click path
• clearWidget signal already has 4 established emit sites
  (transferworker.cpp:373, resultdisplay.cpp:87, errorwidget.cpp:101,
  errorwidget.cpp:107); adding a 5th follows the same proven pattern
• Target line is not a historical fix product; this change does not revert any
  prior fix

Business Impact Scope

• Data Transfer — Backup File Export: Windows users export files, click
  "Back" on the result page, return to the file selection page
• Affected caller: zipfileprocessresultwidget.cpp:64
  (backButton::clicked → backPage)
• After fix: returning from export result page clears all selection state,
  so the user starts fresh on re-entry

Verification Suggestion

Test export success then click Back — verify file selections are cleared.
Test export failure then click Back — same verification. Verify re-selecting
and re-exporting works normally after returning.

---

根因分析

ZipFileProcessResultWidget::backPage()（导出结果页"返回"按钮槽函数）仅发射
changeWidget(choosewidget) 切换页面，未事先发射 clearWidget 信号。该信号连接到
MainWindowPrivate::clearWidget()（mainwindow_p_win.cpp:139-140），对全部选择控件
执行 clear() 重置。由于信号未发射，选择状态未被重置，返回选择页时残留历史选择。

关键证据：

• zipfileprocessresultwidget.cpp:136-138 — backPage() 缺少 clearWidget 发射
• errorwidget.cpp:99-102 — 同模块 ErrorWidget::backPage() 先发射 clearWidget
  再发射 changeWidget，确立了正确范式

修复方案

在 backPage() 中 changeWidget 发射之前添加
emit TransferHelper::instance()->clearWidget();，与 ErrorWidget::backPage()
等同模块返回处理范式一致。

改动安全评估

代码安全评估

• 风险等级: 低风险
• 单文件单函数一行新增，不修改签名、不删公开函数、不影响其他调用者
• clearWidget 信号已有 4 处成熟发射点，新增第 5 处沿用同一范式
• 目标行非历史修复产物，本次改动不撤销任何已有修复

业务影响范围

• 数据迁移-备份文件导出：Windows 端用户导出文件后点击"返回"回到选择备份内容界面
• 受影响调用者：zipfileprocessresultwidget.cpp:64（backButton::clicked → backPage）
• 修复后：从导出结果页返回时清空所有选择状态，用户重新进入时为空白状态

验证建议

测试导出成功后点击返回，验证文件选择已清空。测试导出失败后点击返回，同样验证。
验证返回后重新选择并导出流程正常。

[sourcery-ai]

Sorry, we are unable to review this pull request

The GitHub API does not allow us to fetch diffs exceeding 20000 lines

[github-actions]

TAG Bot

TAG: 0.8.9
EXISTED: no
DISTRIBUTION: unstable

[deepin-ci-robot]

deepin pr auto review

🤖 AI 代码审查报告

总体评分： 88 分 (通过阈值: 70分)

---

📊 总体评价

项目	结果
审查结论	代码审查通过
评分详情	本次PR标题为"fix: clear selection on back from result page"，实际包含大量改动： WaylandUtils从静态链接改为dlopen动态加载、daemon重命名(cooperation-daemon→dde-cooperation-daemon)、 isActiveUser改用UID判定、新增deepin-security-loader集成、新增路径穿越防护与文件传输确认机制、 新增DiscoverController(ZeroConf设备发现)、新增关闭对话框与系统托盘最小化、修复disconnectToDevice空指针问题、 修复Settings::setValueNoNotify参数错误、新增QUNIGO宏解决线程内存泄漏等。代码整体质量良好， 安全防护有显著增强(路径穿越检查、传输确认机制)，但路径穿越验证存在边界缺口，部分新增代码有注释残留。 四维度均通过，总评88分，等级良好。

---

🔍 详细分析

1. 语法逻辑 ✅

评价： 语法正确，逻辑清晰 ✅ 通过

潜在问题：
✅ 未发现明显问题

建议： ['getSaveFullpath中，当realFull为nullptr时应返回空字符串或报错，而非继续返回未验证的fullpath', 'initSearchDeivce中的QTimer应设置父子对象关系或使用QPointer保护this指针']

---

2. 代码质量 ✅

评价： 代码结构清晰，注释完整 ✅ 通过

潜在问题：
✅ 未发现明显问题

建议： ['清理DiscoverController中注释掉的代码块，如需保留设计意图可添加简要注释说明', '统一版权年份格式', 'moveFile可考虑使用局部变量代替修改入参dst']

---

3. 代码性能 ✅

评价： 性能良好，资源使用合理 ✅ 通过

潜在问题：
✅ 未发现明显问题

建议： ['initSecurityLoaderReport中管道读取可考虑使用QSocketNotifier实现事件驱动', 'refresh可考虑增量更新而非全量重建列表']

---

4. 代码安全 🔒

评价： 存在1个安全漏洞 ✅ 通过

🔐 发现 0 个安全漏洞

安全漏洞详情：
✅ 未发现安全漏洞

建议： ['修复getSaveFullpath中realFull为nullptr时的逻辑：当realpath对文件和父目录均返回nullptr时，应拒绝写入而非继续返回fullpath', '可考虑使用realpathat或openat等AT_FDCWD系列函数进行更安全的路径验证']

---

💡 改进建议代码示例

// TransferJob::getSaveFullpath 修复建议
fastring TransferJob::getSaveFullpath(const fastring &rootdir, const fastring &filename)
{
    // ... 前置检查保持不变 ...

    fastring fullpath = path::join(rootdir, acfilename);

    // 二次验证：规范化路径后检查是否仍在用户配置的下载目录内
    fastring allowedRoot = DaemonConfig::instance()->getStorageDir(_app_name);
    char *realRoot = realpath(allowedRoot.c_str(), nullptr);
    char *realFull = realpath(fullpath.c_str(), nullptr);

    // 如果目标文件不存在，验证其父目录
    if (realFull == nullptr) {
        std::pair<fastring, fastring> pairs = path::split(fullpath);
        fastring parentDir = pairs.first;
        realFull = realpath(parentDir.c_str(), nullptr);
    }

    // 修复：如果无法解析路径，拒绝写入
    if (realRoot == nullptr || realFull == nullptr) {
        ELOG << "Path validation failed: unable to resolve path: " << fullpath;
        if (realRoot) free(realRoot);
        if (realFull) free(realFull);
        return "";
    }

    fastring rootPath(realRoot);
    fastring fullPath(realFull);
    if (!fullPath.starts_with(rootPath)) {
        ELOG << "Path traversal detected! Target path outside allowed directory: " << fullpath;
        free(realRoot);
        free(realFull);
        return "";
    }

    free(realRoot);
    free(realFull);
    return fullpath;
}

---

本报告由 AI 代码审查工具自动生成