semantic search

[Johnson-zs]

• fix(dfm-io): drain GLib deferred callbacks after g_file_enumerate_children
• fix(dfm-io): close FTS tree in destructor to prevent memory leak
• perf(dfm-io): cache statx results in DFileInfoPrivate::attributesBySelf
• fix: prevent BOM character loss in path concatenation

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Johnson-zs

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

[sourcery-ai]

Sorry @Johnson-zs, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

[deepin-ci-robot]

deepin pr auto review

★ 总体评分：100分

■ 【总体评价】

代码精准修复了内存泄漏、空指针崩溃、路径遍历越权及重复系统调用四大缺陷，整体质量极高
逻辑严密、注释详实且性能优化效果显著，无任何扣分项

■ 【详细分析】

• 1.语法逻辑（完全正确）✓

DEnumeratorPrivate::~DEnumeratorPrivate() 中正确释放了 fts 句柄并置空指针，防止悬空指针；createEnumerator() 中通过 g_main_context_get_thread_default() 获取上下文并在非空时调用 g_main_context_iteration，逻辑严谨；buildUrl() 增加了空指针拦截，ensureStatxCached() 中使用 statxCached 布尔值作为守卫条件，避免了重复进入系统调用，头文件 <sys/stat.h> 的移动保证了编译依赖的正确性

• 2.代码质量（优秀）✓

消除了 attributesBySelf() 中 6 处完全重复的 statx 调用代码块，将其优雅地抽取为 ensureStatxCached() 方法，符合 DRY 原则；新增的 [[nodiscard]] 属性强制调用方处理返回值，规范了接口契约；针对 GLib 底层 GWeakRef 泄漏问题的注释极其详尽，准确指明了 gdbusconnection.c 中的触发链路，极大提升了代码可维护性

• 3.代码性能（高效）✓

将 6 个时间属性获取逻辑中各自独立发起的 statx 系统调用，重构为基于 mutable 成员变量的单次调用加缓存复用模式，将系统调用次数从最坏情况下的 6 次直降为 1 次，大幅减少了内核态与用户态的上下文切换开销；g_main_context_iteration 仅在存在默认上下文时同步消费空闲源，不引入额外的轮询或阻塞开销

• 4.代码安全（存在0个安全漏洞）✓

漏洞对比统计：新增漏洞 0 个，减少漏洞 0 个，持平 0 个
成功修复了原有的空指针解引用崩溃风险与路径遍历越权漏洞，通过黑名单机制精准拦截了包含 /、\、.、.. 的恶意文件名，同时使用 QByteArray 替代 QString 进行底层字节数组拼接，彻底规避了 QString 构造函数剥离 BOM 头（efbbbf）导致的安全校验绕过问题，整体防御机制完备

■ 【改进建议代码示例】

// 文件：denumerator.cpp buildUrl()
// 建议增加对空字节截断攻击的防御，使安全防护更加严密
QUrl DEnumeratorPrivate::buildUrl(const QUrl &url, const char *fileName)
{
    // 防御空指针，避免std::string或QByteArray构造时崩溃
    if (!fileName) {
        return QUrl();
    }

    QByteArray fileNameBa(fileName);
    
    // 防御空字节截断攻击，防止类似 "file.txt\0.exe" 的恶意绕过
    if (fileNameBa.contains('\0')) {
        return QUrl();
    }

    // 拦截路径遍历攻击，防止恶意文件名越权
    if (fileNameBa.contains('/') || fileNameBa.contains('\\') || fileNameBa == "." || fileNameBa == "..") {
        return QUrl();
    }

    QByteArray path;
    QString urlPath = url.path();

    if (urlPath == "/" || urlPath.isEmpty()) {
        path = QByteArray("/") + fileNameBa;
    } else {
        QByteArray dirPath = urlPath.toUtf8();
        if (!dirPath.endsWith('/')) {
            dirPath.append('/');
        }
        // 使用QByteArray进行底层字节数组拼接，避免QString剥离BOM (efbbbf)
        path = dirPath + fileNameBa;
    }

    // 保留原始 URL 的 scheme 和 host，而不是假定为本地文件
    QUrl newUrl(url);
    newUrl.setPath(QString::fromUtf8(path));
    return newUrl;
}