From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <legion@altlinux.ru>
X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on sa.int.altlinux.org
X-Spam-Level: 
X-Spam-Status: No, score=-1.5 required=5.0 tests=BAYES_00,DNS_FROM_OPENWHOIS
	autolearn=no version=3.2.5
Message-ID: <4C7B9660.2080302@altlinux.ru>
Date: Mon, 30 Aug 2010 15:30:40 +0400
From: Alexey Gladkov <legion@altlinux.ru>
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US;
	rv:1.9.2.9pre) Gecko/20100815 Lightning/1.0b2pre
	Thunderbird/3.1.3pre
MIME-Version: 1.0
To: devel@lists.altlinux.org
References: <4C7B6B6E.2090206@altlinux.ru>	<20100829215746.GA21696@ssh.git.altlinux.org>	<20100829221137.GB7732@altlinux.org>	<1283149714.3294.12.camel@shrek.croc.ru>	<1283149894.3294.14.camel@shrek.croc.ru>	<20100830080754.GB6048@altlinux.org>	<20100830081837.GD6048@altlinux.org>	<1283156824.3294.22.camel@shrek.croc.ru>	<20100830095129.GA10000@altlinux.org>	<4C7B8346.5090402@altlinux.ru>	<20100830102051.GC24249@altlinux.org>	<4C7B883C.9030305@altlinux.ru>
	<20100830151602.4bf2c7f5@sem.localdomain>
In-Reply-To: <20100830151602.4bf2c7f5@sem.localdomain>
X-Enigmail-Version: 1.1.2
Content-Type: text/plain; charset=KOI8-R
Content-Transfer-Encoding: 8bit
Subject: Re: [devel] I: xorg-server, dbus: SELinux support breaks software
X-BeenThere: devel@lists.altlinux.org
X-Mailman-Version: 2.1.12
Precedence: list
Reply-To: ALT Linux Team development discussions <devel@lists.altlinux.org>
List-Id: ALT Linux Team development discussions <devel.lists.altlinux.org>
List-Unsubscribe: <https://lists.altlinux.org/mailman/options/devel>,
	<mailto:devel-request@lists.altlinux.org?subject=unsubscribe>
List-Archive: <http://lists.altlinux.org/pipermail/devel>
List-Post: <mailto:devel@lists.altlinux.org>
List-Help: <mailto:devel-request@lists.altlinux.org?subject=help>
List-Subscribe: <https://lists.altlinux.org/mailman/listinfo/devel>,
	<mailto:devel-request@lists.altlinux.org?subject=subscribe>
X-List-Received-Date: Mon, 30 Aug 2010 11:30:59 -0000
Archived-At: <http://lore.altlinux.org/devel/4C7B9660.2080302@altlinux.ru/>
List-Archive: <http://lore.altlinux.org/devel/>
List-Post: <mailto:devel@altlinux.org>

30.08.2010 15:16, Mikhail Efremov wrote:
>> Здесь мы возвращаем -1. Зачем тут -1, почему не goto out; ?
> 
> Потому что на самом деле неизвестно, включен ли SELinux. И решать что

Без смонтированого /proc selinux работать будет ?

> делать должен вызывающий код, эта функция не должна принимать таких
> решений.

Вот это сильно спорно. Вызывающий код имеет ещё меньше представлений о
том, что делать дальше. Когда is_selinux_enabled() вернул -1, то это
не значит что SELinux выключен (вы сами про это говорите), но и не
значит, что включён.

Сейчас делается упор, что

if (!is_selinux_enabled())

неправильная обработка и нужно расценивать -1 как 0 ... но это тоже
неправильно, так как решение отдано на откуп приложения. Если -1 нужно
расценивать как 0, то так и нужно возвращать.

-- 
Rgrds, legion