From: jiyong.min Date: Thu, 6 Apr 2023 00:45:10 +0000 (+0900) Subject: Remove unnecessary 'oldname' X-Git-Tag: accepted/tizen/unified/20230406.165745^0 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F69%2F290969%2F1;p=platform%2Fcore%2Fapi%2Fmedia-controller.git Remove unnecessary 'oldname' Change-Id: I5dd7ca4116a40165769f1027fd19226b0e779153 --- diff --git a/src/media_controller_util.c b/src/media_controller_util.c index 45dbaba..a6fb54c 100644 --- a/src/media_controller_util.c +++ b/src/media_controller_util.c @@ -26,25 +26,21 @@ static void __mc_util_check_valid_name(const char *name, char **valid_name) { - char old_word[MAX_NAME_LENGTH] = {0, }; char new_word[MAX_NAME_LENGTH] = {0, }; unsigned int i = 0, j = 0; mc_retm_if_failed(name); - memset(old_word, 0, MAX_NAME_LENGTH); memset(new_word, 0, MAX_NAME_LENGTH); - g_strlcpy(old_word, name, sizeof(old_word)); - /* only 0~9, a~z, A~Z, '.', '_' will be used */ - for (i = 0; (i < strlen(old_word)) && (j < sizeof(new_word)); i++) { - if ((old_word[i] >= '0' && old_word[i] <= '9') || - (old_word[i] >= 'a' && old_word[i] <= 'z') || - (old_word[i] >= 'A' && old_word[i] <= 'Z') || - (old_word[i] == '.' && i != 0) || - (old_word[i] == '_' && i != 0)) { - new_word[j++] = old_word[i]; + for (i = 0; (i < strlen(name)) && (j < sizeof(new_word)); i++) { + if ((name[i] >= '0' && name[i] <= '9') || + (name[i] >= 'a' && name[i] <= 'z') || + (name[i] >= 'A' && name[i] <= 'Z') || + (name[i] == '.' && i != 0) || + (name[i] == '_' && i != 0)) { + new_word[j++] = name[i]; } else { if (j + 9 < sizeof(new_word)) { new_word[j++] = 'A';