on xenial help sign in
Don't use appInfo after checking it if is null

Approved by: Michael Zanetti, Unity8 CI Bot

commit491624380f5aca26c36dad023e77dc550d3dbba3 authorAlbert Astals Cid <albert.astals@canonical.com> dateTue 28 Mar 2017, 21:47 (10 years ago) parentf8114aa1213691
1 file changed+10−9
plugins/Unity/Launcher/launchermodel.cpp+10−9
plugins/Unity/Launcher/launchermodel.cpp+10−9
@@ -207,16 +207,17 @@ void LauncherModel::quickListActionInvoked(const QString &appId, int actionIndex
207207 }
208208 } else if (actionId.startsWith(QStringLiteral("surface_"))){
209209 ApplicationInfoInterface *appInfo = m_appManager->findApplication(appId);
210− if (!appInfo) {
211− qWarning() << "App for" << appId << "not found in launcher. Cannot invoke quicklist action";
212− }
213− for (int i = 0; i < appInfo->surfaceList()->count(); ++i) {
214− MirSurfaceInterface *iface = appInfo->surfaceList()->get(i);
215− QString id = actionId;
216− id.remove(QRegExp("^surface_"));
217− if (id == iface->persistentId()) {
218− iface->activate();
210+ if (appInfo) {
211+ for (int i = 0; i < appInfo->surfaceList()->count(); ++i) {
212+ MirSurfaceInterface *iface = appInfo->surfaceList()->get(i);
213+ QString id = actionId;
214+ id.remove(QRegExp("^surface_"));
215+ if (id == iface->persistentId()) {
216+ iface->activate();
217+ }
219218 }
219+ } else {
220+ qWarning() << "App for" << appId << "not found in launcher. Cannot invoke quicklist action";
220221 }
221222 // Nope, we don't know this action, let the backend forward it to the application
222223 } else {