[devel] [SCM] packages/avreg: heads/master

Slava Semushin =?iso-8859-1?q?slava=2Esemushin_=CE=C1_gmail=2Ecom?=
Пт Июл 4 09:08:34 MSD 2008


2008/7/4 Denis Klimov <zver / altlinux.org>:
> Update of /people/zver/packages/avreg.git
[...]
> +++ b/common/create_db
> @@ -100,7 +100,10 @@ feel_db() {
>                echo "Database $AVREGDB already exist"
>        fi
>  }
> -
> +if [ $USER != 'root' ]; then
> +       echo 'This script must be run by root'
> +       exit 1
> +fi
>
>  if [ -r $MYSQL_CNF ]; then
>        chmod 0600 $MYSQL_CNF

Предлагаю хотя бы стремиться писАть более-менее стабильный код,
который был бы устойчив к пробелам в именах файлах и пустым
значениям... Этот код никуда не годится, вот так лучше, на мой взгляд:

if [ "$USER" != 'root' ]; then
       echo 'This script must be run by root' >&2
       exit 1
fi

И далее везде -- заключайте переменные в двойные кавычки, особенно в
условиях if, ведь если переменная окажется пустой, то условие будет
некорректным и вы получите синтаксическую ошибку.

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru


Подробная информация о списке рассылки Devel