From 7d46292af667ae485fc5d744f7aae05136e7769a Mon Sep 17 00:00:00 2001 From: "shyun.min" Date: Thu, 17 May 2018 13:27:49 +0900 Subject: [PATCH] Fix coverity issue Change-Id: Ieaca864601c45b5e0972dbe557da73109735d2f4 --- UIAutomator/dbus.cpp | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/UIAutomator/dbus.cpp b/UIAutomator/dbus.cpp index d6c4ce7..0530555 100644 --- a/UIAutomator/dbus.cpp +++ b/UIAutomator/dbus.cpp @@ -100,16 +100,19 @@ void DBusSignal::DBusSignalHandler(void *data, DBusMessage *msg) { dbus_message_iter_get_basic(&args, &second); dbus_message_iter_next(&args); _D("Type=%s, 1st=%s, 2nd=%s", type, first, second); - } - string action = type; - if (Server::getInstance().HandlerMap.find(action) != Server::getInstance().HandlerMap.end()) { - _D("[%s] Handler", action.c_str()); - //sleep(1); - Server::getInstance().HandlerMap[action](first, second); + string action = type; + if (Server::getInstance().HandlerMap.find(action) != Server::getInstance().HandlerMap.end()) { + _D("[%s] Handler", action.c_str()); + //sleep(1); + Server::getInstance().HandlerMap[action](first, second); + } else { + _D("Invalid Command"); + } } else { - _D("Invalid Command"); + _D("Invalid Message"); } + return; } @@ -132,6 +135,8 @@ void DBusSignal::DBusAppSignalHandler(void *data, DBusMessage *msg) { kill(pid, 9); _D("PopupController kills %s during UI testing", app_name); } + } else { + _D("Invalid Message"); } return; -- 2.34.1