[devel] Покритикуйте init-скрипт

[Dmitry V. Levin]

Hi,

On Wed, Oct 31, 2007 at 04:22:35PM +0600, Slava Semushin wrote:
> В аттаче init-скрипт для OSSEC. Прошу знающих людей посмотреть и
> высказать замечания/предложения.
> init-скрипта в апстрим...
[...]
> #!/bin/sh

В скрипте используются массивы.  Если очень не хочется написать
#!/bin/bash, то может стоит хотя бы оставить комментарий.
Впрочем, скрипт и так содержит достаточно alt-специфики.

[...]
> SourceIfNotEmpty /etc/ossec-init.conf

Видимо в этом файле определена загадочная переменная DIRECTORY.
Я бы добавил строчку
DIRECTORY=
непосредственно перед включением этого файла.

[...]
> if [ "$TYPE" = "local" ]; then
> 	start_order="1 2 3 4 6 7"
> 	stop_order="7 4 6 3 1 2"
> 
> elif [ "$TYPE" = "server" ]; then
> 	start_order="1 2 3 4 5 6 7"
> 	stop_order="7 4 5 6 3 1 2"
> 
> elif [ "$TYPE" = "agent" ]; then
> 	start_order="2 0 4 6"
> 	stop_order="4 6 0 2"
> 
> else
> 	exit 1
> fi

Пожалуйста, откройте для себя case.

[...]
> 	check
> 	[ $RETVAL -eq 0 ] || return

Здесь и далее проще было бы написать
check || return


-- 
ldv
----------- следующая часть -----------
Было удалено вложение не в текстовом формате...
Имя     : =?iso-8859-1?q?=CF=D4=D3=D5=D4=D3=D4=D7=D5=C5=D4?=
Тип     : application/pgp-signature
Размер  : 189 байтов
Описание: =?iso-8859-1?q?=CF=D4=D3=D5=D4=D3=D4=D7=D5=C5=D4?=
Url     : <http://lists.altlinux.org/pipermail/devel/attachments/20071101/23a65af6/attachment-0002.bin>