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/20190805.070446~4^2^2~7
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6242dd15eb2b8fd945a6d4fbbf421da833a40749;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 c7547a18..ab7d8aa7 100644
--- a/src/mediacontroller/mediacontroller_instance.cc
+++ b/src/mediacontroller/mediacontroller_instance.cc
@@ -321,11 +321,16 @@ void MediaControllerInstance::MediaControllerServerReplyCommand(const picojson::
   CHECK_EXIST(args, "requestId", out)
   CHECK_EXIST(args, "data", out)
 
-  server_->CommandReply(
+  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);
 }