[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