release/snipe

[Johnson-zs]

• perf(dfm-io): cache statx results in DFileInfoPrivate::attributesBySelf
• fix: prevent BOM character loss in path concatenation

[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]

[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

[deepin-ci-robot]

deepin pr auto review

★ 总体评分：100分

■ 【总体评价】

代码有效修复了空指针崩溃与路径遍历漏洞，并通过缓存机制大幅优化了系统调用性能
逻辑严谨且消除了重复代码，无任何安全漏洞与性能问题

■ 【详细分析】

• 1.语法逻辑（基本正确）✓

在 DEnumeratorPrivate::buildUrl 中正确增加了空指针防御，路径拼接逻辑从 QString 切换到 QByteArray 处理得当；DFileInfoPrivate::ensureStatxCached 的缓存状态判断与返回值逻辑正确，头文件中新增的成员变量初始化合理。
潜在问题：无
建议：无需修改

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

通过提取 ensureStatxCached 函数，消除了 attributesBySelf 中 6 处重复的 statx 调用代码，大幅提升了可维护性；注释清晰解释了使用 QByteArray 避免剥离 BOM 以及缓存机制的目的。
潜在问题：无
建议：无需修改

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

原先每次获取时间属性均会触发一次 statx 系统调用，现在通过 statxCached 标志位确保同一对象的生命周期内最多只执行一次系统调用，显著降低了内核态切换开销。
潜在问题：无
建议：无需修改

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

漏洞对比统计：新增漏洞 0 个，减少漏洞 2 个，持平 0 个
本次代码修复了原有的空指针解引用崩溃和路径遍历攻击面，未引入新的安全风险。

• 安全漏洞1：无

• 建议：无需修复

■ 【改进建议代码示例】

// 代码已非常完善，无需额外修复。以下为当前优秀代码的保留展示：
bool DFileInfoPrivate::ensureStatxCached() const
{
    if (statxCached)
        return statxValid;

    statxCached = true;

    const QUrl &url = q->uri();
    if (!url.isLocalFile())
        return false;

    unsigned mask = STATX_BASIC_STATS | STATX_BTIME;
    int ret = statx(AT_FDCWD, url.path().toStdString().data(),
                    AT_SYMLINK_NOFOLLOW | AT_NO_AUTOMOUNT, mask, &statxBuf);
    statxValid = (ret == 0);
    return statxValid;
}

[Johnson-zs]

/forcemerge

[deepin-bot]

This pr force merged! (status: blocked)