Handle empty container name in AddContainer API

[Bug]           AddContainer API did not handle the case when container id was empty.
[Cause]         N/A
[Solution]      N/A
[Verification]  Build, install, try adding a container with empty string as name.

Change-Id: I9340e56c58070c5b4c7aa1a0e4d7c2f5ea9c3aa6

diff --git a/server/containers-manager.cpp b/server/containers-manager.cpp
index a707fe2..d0d0a36 100644 (file)
--- a/server/containers-manager.cpp
+++ b/server/containers-manager.cpp
@@ -572,6 +572,13 @@ void ContainersManager::generateNewConfig(const std::string& id,
 void ContainersManager::handleAddContainerCall(const std::string& id,
                                                dbus::MethodResultBuilder::Pointer result)
 {
+    if (id.empty()) {
+        LOGE("Failed to add container - invalid name.");
+        result->setError(api::host::ERROR_CONTAINER_CREATE_FAILED,
+                         "Failed to add container - invalid name.");
+        return;
+    }
+
     LOGI("Adding container " << id);
 
     // TODO: This solution is temporary. It utilizes direct access to config files when creating new