[mediacontroller] Fix error handling in ReplyCommand. 10/208810/1
authorMichal Michalski <m.michalski2@partner.samsung.com>
Tue, 16 Apr 2019 10:26:16 +0000 (12:26 +0200)
committerPiotr Kosko/Native/Web API (PLT) /SRPOL/Professional/삼성전자 <p.kosko@samsung.com>
Fri, 28 Jun 2019 13:18:53 +0000 (15:18 +0200)
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

index c7547a18b1ac3ac0b2195f0c0d904ce565cce075..ab7d8aa7e2ac25341661f32fe4ecff3b6355941e 100644 (file)
@@ -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);
 }