blumia/dde-application-manager
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: AM crashed when Launch failed

Browse Source 
Signed-off-by: ComixHe <heyuming@deepin.org>
This commit is contained in:
ComixHe 2023-09-05 14:33:21 +08:00 committed by Comix
parent 897db2c85a
commit ebe218b70e
1 changed files with 4 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/dbus/applicationservice.cpp
View File

@ -173,10 +173,8 @@ QDBusObjectPath ApplicationService::Launch(const QString &action, const QStringL
process.start(m_launcher, commands);
process.waitForFinished();
if (auto code = process.exitCode(); code != 0) {
auto msg = QString{"Launch Application Failed"};
qWarning() << msg;
sendErrorReply(QDBusError::Failed, msg);
return {};
qWarning() << "Launch Application Failed";
return QDBusError::Failed;
}
return objectPath;
}
@ -203,10 +201,8 @@ QDBusObjectPath ApplicationService::Launch(const QString &action, const QStringL
process.waitForFinished();
auto exitCode = process.exitCode();
if (exitCode != 0) {
auto msg = QString{"Launch Application Failed"};
qWarning() << msg;
sendErrorReply(QDBusError::Failed, msg);
return {};
qWarning() << "Launch Application Failed";
return QDBusError::Failed;
}
return objectPath;
},