fix: Improve command execution using QProcess

[dengzhongyuan365-dev]

• Updated command execution in PageInfo and DriverManager to use QStringList for better argument handling.
• Enhanced error handling in PageMultiInfo when removing drivers by validating row index.
• Refactored Utils to append to blacklist configuration file directly instead of using echo command.
• Replaced system calls with QProcess in EnableUtils for network operations to improve reliability and error handling.

Log: Refactor command execution and improve error handling across multiple files.

[sourcery-ai]

Sorry @dengzhongyuan365-dev, 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

这份代码修改主要涉及进程启动方式的优化、文件操作方式的改进以及数组越界检查的增加。以下是对这些修改的详细审查和改进建议：

1. 进程启动方式优化 (drivermanager.cpp, utils.cpp, PageInfo.cpp)

修改内容：
将 QProcess::start(const QString &program) 改为 QProcess::start(const QString &program, const QStringList &arguments)。

审查意见：

• 优点：
  • 安全性提升：这是最重要的改进。原先通过字符串拼接构造命令（如 "sudo apt install " + packageName），如果 packageName 包含空格或特殊字符（例如 package name; rm -rf /），可能会导致命令注入漏洞。使用 QStringList 传参可以确保参数被正确转义，防止注入攻击。
  • 跨平台性：QStringList 方式在处理参数时更具一致性，减少因 shell 解析差异带来的问题。
• 改进建议：
  • 错误处理：目前的代码只调用了 waitForFinished(-1)，但没有检查 QProcess 的错误状态（如 QProcess::Crashed, QProcess::FailedToStart）。建议增加错误检查：

    p.start("sudo", QStringList() << "apt" << "install" << packageName);
    if (!p.waitForFinished(-1)) {
        qCWarning(appLog) << "Process failed to start or crashed:" << p.errorString();
        return false;
    }
    if (p.exitCode() != 0) {
        qCWarning(appLog) << "Process exited with error:" << p.readAllStandardError();
        return false;
    }

  • 异步执行：waitForFinished(-1) 会阻塞当前线程。如果这是在主线程中运行，会导致界面冻结。建议考虑使用信号槽机制（QProcess::finished）进行异步处理。

2. 文件操作改进 (utils.cpp)

修改内容：
将使用 echo ... >> file 的 shell 命令改为使用 QFile 直接写入文件。

审查意见：

• 优点：
  • 性能：直接使用 QFile 避免了启动 shell 进程的开销，效率更高。
  • 可靠性：不依赖于 shell 的存在和语法，更加稳定。
  • 安全性：避免了通过 shell 命令注入的风险。
• 改进建议：
  • 文件权限：写入 /etc/modprobe.d/*.conf 通常需要 root 权限。代码中虽然去掉了 sudo，但程序本身必须以 root 身份运行才能成功写入。建议在函数开头或文档中明确这一前置条件。
  • 原子性：当前的写入操作不是原子的。如果在写入过程中程序崩溃，可能会导致文件损坏。更好的做法是先写入临时文件，然后重命名：

    QFile tempFile(BLACKLIST_CONF + ".tmp");
    if (tempFile.open(QIODevice::WriteOnly | QIODevice::Text)) {
        // ... 写入内容 ...
        tempFile.close();
        if (tempFile.rename(BLACKLIST_CONF)) {
            return true;
        }
    }
    return false;

  • 重复检查：在添加黑名单之前，最好检查该模块是否已经在黑名单中，避免重复添加。

3. 复杂命令执行 (enableutils.cpp)

修改内容：
将 system() 调用改为 QProcess 执行。

审查意见：

• 优点：
  • 与前述类似，避免了 shell 命令注入的风险。
  • 使用 QProcess 可以更好地与 Qt 的日志系统集成。
• 改进建议：
  • 代码可读性：rfkill 命令非常复杂，包含管道和命令替换。虽然使用了 sh -c，但仍然难以维护。建议将其拆分为多个 QProcess 调用，或者使用 C++ 代码解析 iw dev 和 rfkill list 的输出来构建最终的命令。
  • 硬编码路径：/sbin/ifconfig 是硬编码的路径。虽然常见，但并非所有系统都使用这个路径。建议使用 QStandardPaths::findExecutable("ifconfig") 来查找可执行文件，或者直接使用 Qt 的 QNetworkInterface 类来管理网络接口。

4. 数组越界检查 (PageMultiInfo.cpp)

修改内容：
在访问 m_lstDevice[row] 之前增加了 row 的有效性检查。

审查意见：

• 优点：
  • 防止了数组越界访问，避免了潜在的崩溃。
• 改进建议：
  • 防御性编程：虽然增加了检查，但 m_lstDevice 本身可能包含 nullptr。代码中已经有 if (nullptr == device) 的检查，这是好的。建议使用 m_lstDevice.value(row)，它在索引无效时返回默认构造的值（对于指针来说是 nullptr），这样代码会更简洁：

    DeviceBaseInfo *device = m_lstDevice.value(row);
    if (!device) {
        qCWarning(appLog) << "Null device at row:" << row;
        return;
    }

总结

这些修改主要关注于安全性（防止命令注入）和健壮性（防止数组越界）。改进方向是正确的，但在错误处理、异步操作和代码可维护性方面还有提升空间。特别是涉及系统命令执行的部分，建议进一步抽象和封装，以提高代码的可测试性和可维护性。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: dengzhongyuan365-dev, lzwind

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

[dengzhongyuan365-dev]

/forcemerge