sync: from linuxdeepin/dtkcore

[deepin-ci-robot]

Synchronize source files from linuxdeepin/dtkcore.

Source-pull-request: linuxdeepin/dtkcore#490

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: deepin-ci-robot

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

代码审查意见：

1. CMakeLists.txt 文件修改：

   • 在 CMakeLists.txt 文件中新增了 find_package(DBus1 REQUIRED) 和 target_link_libraries(${LIB_NAME} PRIVATE dbus-1)，这表明项目现在依赖于 DBus1 库。需要确认 DBus1 库的版本和兼容性，以及是否已经正确安装。

2. dsgapplication.cpp 文件修改：

   • 移除了对 QDBusUnixFileDescriptor、QDBusReply、QDBusConnectionInterface 和 DDBusInterface 的包含，转而使用 libdbus-1 库进行 D-Bus 操作。这是一个合理的改动，因为它允许更底层的控制，并且可能提高了性能。
   • 新增了两个静态函数 checkDBusServiceActivatable 和 callDBusIdentifyMethod，这些函数使用 libdbus-1 库进行 D-Bus 通信。这些函数的实现看起来是正确的，但是需要确保它们在所有情况下都能正确处理错误，并且有适当的错误处理和日志记录。
   • 在 isServiceActivatable 函数中，移除了原来的基于 Qt 的 D-Bus 操作，转而使用 checkDBusServiceActivatable 函数。这是一个改进，因为它简化了代码并使用了更底层的库。
   • 在 getId 函数中，移除了原来的基于 Qt 的 D-Bus 操作，转而使用 callDBusIdentifyMethod 函数。这同样是一个改进，因为它简化了代码并使用了更底层的库。

3. 代码质量：

   • 新增的 checkDBusServiceActivatable 和 callDBusIdentifyMethod 函数应该有适当的错误处理和日志记录，以确保在出现问题时能够及时发现和解决问题。
   • 代码中的日志记录应该更加详细，包括函数的输入参数和返回值，以便于调试和问题追踪。

4. 代码性能：

   • 使用 libdbus-1 库进行 D-Bus 操作可能比使用 Qt 的 D-Bus 操作更高效，因为它避免了 Qt 的抽象层开销。需要确保这种改动能带来预期的性能提升。

5. 代码安全性：

   • 确保所有与 D-Bus 通信相关的函数都正确处理了错误情况，以防止潜在的拒绝服务攻击或其他安全问题。
   • 验证从 D-Bus 接收到的数据，确保它们符合预期的格式和内容，以防止注入攻击。

6. 代码维护性：

   • 新增的函数 checkDBusServiceActivatable 和 callDBusIdentifyMethod 应该有适当的文档注释，说明它们的用途、参数和返回值。
   • 如果这些函数在多个地方使用，考虑将它们提取到单独的模块或类中，以提高代码的可重用性和可维护性。

总体来说，这些修改看起来是合理的，并且可能提高了代码的性能和可维护性。但是，需要确保所有新增的函数都有适当的错误处理和日志记录，并且已经通过了充分的测试。