From 6242dd15eb2b8fd945a6d4fbbf421da833a40749 Mon Sep 17 00:00:00 2001
From: Michal Michalski <m.michalski2@partner.samsung.com>
Date: Tue, 16 Apr 2019 12:26:16 +0200
Subject: [PATCH] [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>
---
 src/mediacontroller/mediacontroller_instance.cc | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

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);
 }
 
-- 
2.34.1