From: Michal Michalski <m.michalski2@partner.samsung.com>
Date: Tue, 16 Apr 2019 10:26:16 +0000 (+0200)
Subject: [mediacontroller] Fix error handling in ReplyCommand.
X-Git-Tag: submit/tizen/20190510.110853~6
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=89195fe8f56a199ac2c819b6620299b921a41dd9;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git

[mediacontroller] Fix error handling in ReplyCommand.

MediaControllerServerReplyCommand function did not
handle errors returned by MediaControllerServer::CommandReply()
call. This commit handles those errors.

[Verification]
Code compiles.

Change-Id: Ibc5acacc32ff23a6d8fa7cac096ade0e40e2b697
Signed-off-by: Michal Michalski <m.michalski2@partner.samsung.com>
---

diff --git a/src/mediacontroller/mediacontroller_instance.cc b/src/mediacontroller/mediacontroller_instance.cc
index 1dd1fb5b..1e3bb8a6 100644
--- a/src/mediacontroller/mediacontroller_instance.cc
+++ b/src/mediacontroller/mediacontroller_instance.cc
@@ -394,8 +394,15 @@ void MediaControllerInstance::MediaControllerServerReplyCommand(const picojson::
   CHECK_EXIST(args, "requestId", out)
   CHECK_EXIST(args, "data", out)
 
-  server_->CommandReply(args.get("clientName").get<std::string>(),
-                        args.get("requestId").get<std::string>(), args.get("data"));
+  auto result = server_->CommandReply(
+    args.get("clientName").get<std::string>(),
+    args.get("requestId").get<std::string>(),
+    args.get("data"));
+
+  if (!result) {
+    LogAndReportError(result, &out);
+    return;
+  }
 
   ReportSuccess(out);
 }