fix: Improve command execution using QProcess

- 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.

Author: dengzhongyuan365-dev

deepin-devicemanager-server/deepin-devicecontrol/src/drivercontrol/drivermanager.cpp

@@ -639,8 +639,7 @@ QList<DriverManager::TDriverInfo> DriverManager::parsePrinterInfo(const QByteArr
 bool DriverManager::printerHasInstalled(const QString &packageName)
 {
     QProcess p;
-    QString cmd = "sudo dpkg -s " + packageName;
-    p.start(cmd);
+    p.start("sudo", QStringList() << "dpkg" << "-s" << packageName);
     p.waitForFinished(-1);
 
     QByteArray r = p.readAll();
@@ -655,7 +654,7 @@ bool DriverManager::printerHasInstalled(const QString &packageName)
 bool DriverManager::installPrinter(const QString &packageName)
 {
     QProcess p;
-    p.start("sudo apt install " + packageName);
+    p.start("sudo", QStringList() << "apt" << "install" << packageName);
     p.waitForFinished(-1);
 
     return printerHasInstalled(packageName);
@@ -669,7 +668,7 @@ bool DriverManager::installPrinter(const QString &packageName)
 bool DriverManager::unInstallPrinter(const QString &packageName)
 {
     QProcess p;
-    p.start("sudo dpkg -r " + packageName);
+    p.start("sudo", QStringList() << "dpkg" << "-r" << packageName);
     p.waitForFinished(-1);
 
     return !printerHasInstalled(packageName);

deepin-devicemanager-server/deepin-devicecontrol/src/drivercontrol/utils.cpp

@@ -127,18 +127,19 @@ bool Utils::getVersion(QString &major, QString &minor)
 bool Utils::addModBlackList(const QString &moduleName)
 {
     qCDebug(appLog) << "Adding module to blacklist:" << moduleName;
-    QProcess process;
-    process.start(QString("echo blacklist %1 >> %2").arg(moduleName).arg(BLACKLIST_CONF));
-    if (!process.waitForFinished())
-        return  false;
-    return  true;
+    QFile file(BLACKLIST_CONF);
+    if (!file.open(QIODevice::Append | QIODevice::Text))
+        return false;
+    file.write(QString("blacklist %1\n").arg(moduleName).toUtf8());
+    file.close();
+    return true;
 }
 
 bool Utils::unInstallPackage(const QString &packageName)
 {
     qCDebug(appLog) << "Uninstalling package:" << packageName;
     QProcess process;
-    process.start(QString("apt remove %1").arg(packageName));
+    process.start("apt", QStringList() << "remove" << packageName);
     if (!process.waitForFinished())
         return  false;
     return  true;

deepin-devicemanager-server/deepin-devicecontrol/src/enablecontrol/enableutils.cpp

@@ -159,14 +159,19 @@ bool EnableUtils::ioctlOperateNetworkLogicalName(const QString &logicalName, boo
         QString cmd = QString("rfkill %1 $(rfkill list | grep -A 2 \"phy$(iw dev %2 info 2>/dev/null | awk '/wiphy/{print $2}')\" | awk 'NR==1{print $1}' | tr -d ':')")
                 .arg(enable ? "unblock" : "block")
                 .arg(logicalName);
-        int ret = system(cmd.toStdString().c_str());
-        if (ret != 0) {
-            qCritical() << "Failed to block/unblock wifi: " << " error code: " << ret ;
+        QProcess p1;
+        p1.start("sh", QStringList() << "-c" << cmd);
+        p1.waitForFinished(-1);
+        if (p1.exitCode() != 0) {
+            qCritical() << "Failed to block/unblock wifi: " << " error code: " << p1.exitCode() ;
         }
-        cmd = QString("/sbin/ifconfig %1 %2").arg(logicalName).arg(enable ? "up" : "down");
-        ret = system(cmd.toStdString().c_str());
-        if (ret != 0) {
-            qCritical() << "Failed to up/down network: " << logicalName << enable << " error code: " << ret ;
+        
+        // 使用QProcess执行ifconfig
+        QProcess ifconfigProcess;
+        ifconfigProcess.start("/sbin/ifconfig", QStringList() << logicalName << (enable ? "up" : "down"));
+        ifconfigProcess.waitForFinished(-1);
+        if (ifconfigProcess.exitCode() != 0) {
+            qCritical() << "Failed to up/down network: " << logicalName << enable << " error code: " << ifconfigProcess.exitCode();
             return false;
         }
     } else {

deepin-devicemanager/src/Page/PageInfo.cpp

@@ -73,8 +73,7 @@ bool PageInfo::packageHasInstalled(const QString &packageName)
 {
     qCDebug(appLog) << "Checking if package is installed:" << packageName;
     QProcess p;
-    QString cmd = "dpkg -s " + packageName;
-    p.start(cmd);
+    p.start("dpkg", QStringList() << "-s" << packageName);
     p.waitForFinished(-1);
 
     QByteArray r = p.readAll();

deepin-devicemanager/src/Page/PageMultiInfo.cpp

@@ -242,6 +242,10 @@ void PageMultiInfo::slotActionUpdateDriver(int row)
 void PageMultiInfo::slotActionRemoveDriver(int row)
 {
     qCDebug(appLog) << "Removing driver for device at row:" << row;
+    if (row < 0 || row >= m_lstDevice.size()) {
+        qCWarning(appLog) << "Invalid row index:" << row;
+        return;
+    }
     DeviceBaseInfo *device = m_lstDevice[row];
     if (nullptr == device) {
         qCWarning(appLog) << "Null device at row:" << row;