From: Yunhee Seo Date: Tue, 20 Feb 2024 02:43:13 +0000 (+0900) Subject: sound: Remove set_theme_id() from id initialization X-Git-Tag: accepted/tizen/unified/20240327.141644^0 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f4f83ffdb8624f754e2b2f0de29ae21fe08afe29;p=platform%2Fcore%2Fsystem%2Flibsvi.git sound: Remove set_theme_id() from id initialization When the sound current_theme_id is initialized, there is no need to call sound_set_theme_id(). Because sound_set_theme_id() calls vconf_set_int(), this requires privilege "http://tizen.org/privilege/systemsettings.admin" when it call for VCONFKEY_SETAPPL_ACCESSIBILITY_SOUND_FEEDBACK_THEME. That makes all apps have that privilege during sound init process. Thus, this function usage is removed. Current theme id policy is as follows. 1. If there is a set vconf value, current theme id follows that value. 2. It there is no set vconf value, current theme id will be default theme id by sound conf parsing. Change-Id: Ie526151a993006793902bb502f915d8b94c996b3 Signed-off-by: Yunhee Seo --- diff --git a/src/sound.c b/src/sound.c index 02bfbb7..5a67eef 100644 --- a/src/sound.c +++ b/src/sound.c @@ -163,7 +163,7 @@ static void sound_theme_id_init(void) if (ret < 0 || theme_id == SOUND_THEME_ID_UNINITIALIZED) sound_thememan_get_default_sound_theme_id(&theme_id); - sound_set_theme_id(theme_id); + current_theme_id = theme_id; } static void sound_init(void)