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

["Dmitry V. Levin" <ldv@altlinux.org>]

[-- Attachment #1: Type: text/plain, Size: 1094 bytes --]

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

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]