Check NOT_SUPPORTED feature ahead than INVALID_PARAMETER

Change-Id: Iad5adae6345fca3b5cd2f913f92a671426069d87
Signed-off-by: jiwon44.park <jiwon44.park@samsung.com>

diff --git a/src/phn.c b/src/phn.c
index a250f35f8b846c0ed9ab60eb670826e54e23aa17..bd3b4dce734e61f6c20dd75638f80fbb34c765aa 100644 (file)
--- a/src/phn.c
+++ b/src/phn.c
@@ -120,10 +120,6 @@ API int phone_number_get_normalized_number(const char *number, char **normalized
 {
        int ret;
 
-       RETVM_IF(NULL == number || '\0' == *number, PHONE_NUMBER_ERROR_INVALID_PARAMETER,
-                       "Invalid parameter (number is NULL)");
-       RETVM_IF(NULL == normalized_number, PHONE_NUMBER_ERROR_INVALID_PARAMETER, "Invalid parameter (normalized_number is NULL)");
-
        ret = phn_get_normalized_number(number, normalized_number);
        if (PHONE_NUMBER_ERROR_NONE != ret) {
                ERR("phn_get_normalized_number() Fail(%d)", ret);

diff --git a/src/phn_phonenumber_wrapper.cpp b/src/phn_phonenumber_wrapper.cpp
index 9e5b845f2755523e37549c58854d0737a0d649b4..7e1b56bc7df11b0f6503384c63c316d24ce732e9 100644 (file)
--- a/src/phn_phonenumber_wrapper.cpp
+++ b/src/phn_phonenumber_wrapper.cpp
@@ -522,6 +522,10 @@ int phn_get_normalized_number(const char *number, char **out_e164)
                return PHONE_NUMBER_ERROR_NOT_SUPPORTED;
        }
 
+       RETVM_IF(NULL == number || '\0' == *number, PHONE_NUMBER_ERROR_INVALID_PARAMETER,
+                       "Invalid parameter (number is NULL)");
+       RETVM_IF(NULL == out_e164, PHONE_NUMBER_ERROR_INVALID_PARAMETER, "Invalid parameter (normalized_number is NULL)");
+
        ret = _phn_get_cc(false, &cc);
        if (PHONE_NUMBER_ERROR_NONE != ret) {
                ERR("_phn_get_cc() Fail(%d)", ret);