fix: sync with qt5platform-plugins

[18202781743]

linuxdeepin/qt5platform-plugins#305

[deepin-ci-robot]

deepin pr auto review

这段代码是一个虚表(vtable)钩子实现的一部分，主要检查对象是否已经被钩子处理过。我来对这段代码进行审查：

语法逻辑

1. 代码逻辑基本正确，但有一个潜在问题：objToGhostVfptr.value(obj)和后面的_obj类型不匹配。objToGhostVfptr的key是quintptr**(即_obj类型)，但代码中直接使用了obj作为key。

代码质量

1. 日志输出使用了qCDebug，这是一个好的实践，但日志信息可以更详细一些，比如添加ghost vtable的地址信息。
2. 注释清晰，解释了代码的目的和逻辑流程。
3. 错误处理较为完善，考虑到了地址重用的情况。

代码性能

1. 当前实现使用了哈希表查找(objToGhostVfptr.contains和objToGhostVfptr.value)，时间复杂度为O(1)，性能较好。
2. 但是连续进行了两次查找操作(value和contains)，可以优化为一次查找。

代码安全

1. 代码正确处理了地址重用的情况，避免了潜在的内存访问错误。
2. 类型转换(quintptr**)(obj)没有进行有效性检查，如果传入的obj是nullptr，会导致解引用空指针。

改进建议

bool DVtableHook::hasVtable(const void *obj)
{
    // 参数有效性检查
    if (!obj) {
        qCDebug(vtableHook) << "hasVtable: null object pointer";
        return false;
    }
    
    quintptr **_obj = (quintptr**)(obj);
    
    // 一次性获取ghost vtable，避免重复查找
    auto it = objToGhostVfptr.find(_obj);
    if (it == objToGhostVfptr.end()) {
        return false;
    }

    // 检查当前对象的 vtable 指针是否指向我们记录的 ghost vtable
    quintptr *ghost_vtable = it.value();
    if (*_obj != adjustToEntry(ghost_vtable)) {
        // vtable 不匹配，说明地址被重用了
        qCDebug(vtableHook) << "hasVtable: vtable mismatch! Address reused by different object."
                           << "obj:" << QString("0x%1").arg((quintptr)obj, 0, 16)
                           << "current vtable:" << QString("0x%1").arg((quintptr)*_obj, 0, 16)
                           << "expected vtable:" << QString("0x%1").arg((quintptr)ghost_vtable, 0, 16);
        return false;
    }

    return true;
}

改进点：

1. 添加了nullptr检查，提高安全性
2. 使用迭代器代替直接查找，减少哈希表查找次数
3. 增加了日志信息的详细程度，包含当前vtable和期望vtable的地址
4. 保持了原有的错误处理逻辑，但使代码更健壮
5. 代码结构更清晰，逻辑流程更易理解

这些改进不会显著影响性能，但能提高代码的安全性和可维护性。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, ComixHe

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