[mediacontroller] Fixing Coverity issues

Fixes: 1192721

Change-Id: I1bd564b935d35f0586e174869f7df10cd0cc705b

diff --git a/src/mediacontroller/mediacontroller_server.cc b/src/mediacontroller/mediacontroller_server.cc
index 87c6bb7..cd6b1ea 100644 (file)
--- a/src/mediacontroller/mediacontroller_server.cc
+++ b/src/mediacontroller/mediacontroller_server.cc
@@ -539,6 +539,11 @@ PlatformResult MediaControllerServer::CreatePlaylist(const std::string& serverNa
                                                      mc_playlist_h& playlist_handle) {
   ScopeLogger();
 
+  if (playlist_info == nullptr) {
+    return LogAndCreateResult(ErrorCode::UNKNOWN_ERR,
+                              "Unable to createPlaylist",
+                              ("playlist_info is nullptr"));
+  }
   mc_playlist_h playlist_handle_ = nullptr;
 
   std::lock_guard<std::mutex> lock(handle_mutex_);
@@ -552,11 +557,6 @@ PlatformResult MediaControllerServer::CreatePlaylist(const std::string& serverNa
 
   playlist_handle = playlist_handle_;
 
-  if (playlist_info == nullptr) {
-    return LogAndCreateResult(ErrorCode::UNKNOWN_ERR, "Unable to createPlaylist",
-                              ("playlist_info is nullptr"));
-  }
-
   *playlist_info = picojson::value(picojson::object());
   picojson::object& obj = playlist_info->get<picojson::object>();
   obj[kName] = picojson::value(name);