From: Inkyun Kil Date: Wed, 9 May 2018 04:57:23 +0000 (+0900) Subject: Add codes for defensive programming X-Git-Tag: accepted/tizen/unified/20180516.065523~1 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F68%2F178268%2F1;p=platform%2Fcore%2Fsystem%2Fbuxton2.git Add codes for defensive programming Before buxton daemon wakes up, some modules can use buxton2ctl directly by using vconftool. Then 'memory.db' file's owner may be wrong. Change-Id: Ieed25b378c1c579d1216f10b590063a910b21a49 Signed-off-by: Inkyun Kil --- diff --git a/vconf-compat/vconftool b/vconf-compat/vconftool index 6826dc0..833b46c 100755 --- a/vconf-compat/vconftool +++ b/vconf-compat/vconftool @@ -10,6 +10,9 @@ OPT_INIT="" OPT_DEBUG=0 OPT_FORCE=0 +OWNER=buxton +MEM_DBPATH=/run/buxton2/memory.db + usage() { COMM=`basename $0` cat << EOF @@ -86,6 +89,12 @@ do_get() { for k in $LIST; do get_key $k done + + #For defensive programming + if [ "${OPT_DIRECT}" == "-d" ] + then + chown ${OWNER}:${OWNER} ${MEM_DBPATH} + fi } do_unset() { @@ -121,6 +130,7 @@ do_create() { RES=`${BUXTONTOOL} ${OPT_DIRECT} create-${TYPE} \ ${LAYER} ${OPT_KEY} "${OPT_VAL}" "" "" 2>&1` [ $? -ne 0 ] && echo "Error: $RES" && exit 1 + exit 0 } @@ -143,6 +153,12 @@ do_set() { exit $? fi + #For defensive programming + if [ "${OPT_DIRECT}" == "-d" ] + then + chown ${OWNER}:${OWNER} ${MEM_DBPATH} + fi + dbg ${BUXTONTOOL} ${OPT_DIRECT} ${OPT_INIT} set-${TYPE} \ ${LAYER} ${OPT_KEY} \"${OPT_VAL}\" RES=`${BUXTONTOOL} ${OPT_DIRECT} ${OPT_INIT} set-${TYPE} \