Add a condition to check if the provider name is blank or not.

Change-Id: Ibbb12704fc92c3c399135232234699bcf896e94e

diff --git a/src/api/maps_service.cpp b/src/api/maps_service.cpp
index e61924f..0eb64f9 100644 (file)
--- a/src/api/maps_service.cpp
+++ b/src/api/maps_service.cpp
@@ -151,7 +151,7 @@ EXPORT_API int maps_service_create(const char *maps_provider, maps_service_h *ma
 {
        if (!maps_condition_check_maps_feature())
                return MAPS_ERROR_NOT_SUPPORTED;
-       if (!maps || !maps_provider)
+       if (!maps || !maps_provider || strlen(maps_provider) == 0)
                return MAPS_ERROR_INVALID_PARAMETER;
        if (!maps_condition_check_privilege()) {
                //LCOV_EXCL_START
@@ -214,11 +214,12 @@ EXPORT_API int maps_service_create(const char *maps_provider, maps_service_h *ma
        return error;
 }
 
+//LCOV_EXCL_START
 EXPORT_API int maps_service_create_without_user_consent(const char *maps_provider, maps_service_h *maps)
 {
        if (!maps_condition_check_maps_feature())
                return MAPS_ERROR_NOT_SUPPORTED;
-       if (!maps || !maps_provider)
+       if (!maps || !maps_provider || strlen(maps_provider) == 0)
                return MAPS_ERROR_INVALID_PARAMETER;
        if (!maps_condition_check_privilege()) {
                MAPS_LOGD("ERROR: privilege is not included");
@@ -232,7 +233,7 @@ EXPORT_API int maps_service_create_without_user_consent(const char *maps_provide
        error = maps_service_create(maps_provider_without_uc, maps);
        return error;
 }
-
+//LCOV_EXCL_STOP
 
 EXPORT_API int maps_service_destroy(maps_service_h maps)
 {