Re: [devel] [git update] packages/ulogd: heads/master

Re: [devel] [git update] packages/ulogd: heads/master

[Vladimir V. Kamarzin]

>>>>> On 28 Nov 2007 at 18:51 "AA" == Avramenko Andrew writes:

 AA> Update of /people/liks/packages/ulogd.git
 AA>     1.24-alt9
 AA>     Fix #13469. The problem was in the rights on the /var/lib/ulogd dir - user ulogd must be the owner of it.

 AA> Full diff since `1.24-alt6-3-g922f3d9' follows:
 AA> diff --git a/ulogd.spec b/ulogd.spec
 AA> index 45e5adc..89b72eb 100644
 AA> --- a/ulogd.spec
 AA> +++ b/ulogd.spec
 AA> @@ -128,7 +128,7 @@ rm -rf %buildroot/%_includedir/libipulog
 AA>  %doc COPYING AUTHORS README
 AA>  %doc doc/%name.txt doc/%name.a4.ps doc/%name.html
 AA>  %_man8dir/*
 AA> -%dir %_localstatedir/%name/
 AA> +%attr(0750,ulogd,ulogd) %_localstatedir/%name/

Ну вообще-то у нас так не принято делать:

Владельцы каталогов

Пакеты НЕ ДОЛЖНЫ содержать каталоги, принадлежащие псевдо-пользователям. Вместо этого следует использовать каталоги, принадлежащие root, с установленным sticky bit и доступом группы по записи. 

Обоснование: Псевдо-пользователь не должен иметь право изменять атрибуты
каталогов, а также файлы и каталоги, созданные другими пользователями;
нарушение этого правила, как правило приводит к возможности осуществления
pseudouser/root compromise.

http://docs.altlinux.ru/alt/devel/ch01s03.html#id2884290

-- 
vvk

Re: [devel] [git update] packages/ulogd: heads/master

[Avramenko Andrew]

Спасибо. Исправлю.



-----Original Message-----
From: "Vladimir V. Kamarzin" <vvk@vvk.pp.ru>
To: ALT Devel discussion list <devel@lists.altlinux.org>
Date: Thu, 29 Nov 2007 18:47:10 +0500
Subject: Re: [devel] [git update] packages/ulogd: heads/master

> >>>>> On 28 Nov 2007 at 18:51 "AA" == Avramenko Andrew writes:
> 
>  AA> Update of /people/liks/packages/ulogd.git
>  AA>     1.24-alt9
>  AA>     Fix #13469. The problem was in the rights on the /var/lib/ulogd dir - user ulogd must be the owner of it.
> 
>  AA> Full diff since `1.24-alt6-3-g922f3d9' follows:
>  AA> diff --git a/ulogd.spec b/ulogd.spec
>  AA> index 45e5adc..89b72eb 100644
>  AA> --- a/ulogd.spec
>  AA> +++ b/ulogd.spec
>  AA> @@ -128,7 +128,7 @@ rm -rf %buildroot/%_includedir/libipulog
>  AA>  %doc COPYING AUTHORS README
>  AA>  %doc doc/%name.txt doc/%name.a4.ps doc/%name.html
>  AA>  %_man8dir/*
>  AA> -%dir %_localstatedir/%name/
>  AA> +%attr(0750,ulogd,ulogd) %_localstatedir/%name/
> 
> Ну вообще-то у нас так не принято делать:
> 
> Владельцы каталогов
> 
> Пакеты НЕ ДОЛЖНЫ содержать каталоги, принадлежащие псевдо-пользователям. Вместо этого следует использовать каталоги, принадлежащие root, с установленным sticky bit и доступом группы по записи. 
> 
> Обоснование: Псевдо-пользователь не должен иметь право изменять атрибуты
> каталогов, а также файлы и каталоги, созданные другими пользователями;
> нарушение этого правила, как правило приводит к возможности осуществления
> pseudouser/root compromise.
> 
> http://docs.altlinux.ru/alt/devel/ch01s03.html#id2884290
> 
> -- 
> vvk
> 
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel