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=-2.6 required=5.0 tests=BAYES_00 autolearn=ham
	version=3.2.5
Message-ID: <512F3FDA.3010703@altlinux.ru>
Date: Thu, 28 Feb 2013 15:30:34 +0400
From: Alexey Gladkov <legion@altlinux.ru>
User-Agent: Mozilla/5.0 (X11; Linux x86_64;
	rv:17.0) Gecko/20130117 Thunderbird/17.0.2
MIME-Version: 1.0
To: sisyphus@lists.altlinux.org
References: <1388768.vWpN7ZqqRi@epiphanov.bkpi82.nikiet.ru>
	<512F030C.8010103@altlinux.ru>
	<1374390.7FYEg3PTU2@epiphanov.bkpi82.nikiet.ru>
	<512F3088.8050809@altlinux.ru> <20130228102904.GF22041@osdn.org.ua>
	<512F3C6E.3020702@altlinux.ru> <512F3D38.8060702@etersoft.ru>
In-Reply-To: <512F3D38.8060702@etersoft.ru>
X-Enigmail-Version: 1.5a1pre
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Subject: Re: [sisyphus] =?utf-8?b?0JLQvtC/0YDQvtGBINC/0L4gb3MtcmVsZWFzZQ==?=
X-BeenThere: sisyphus@lists.altlinux.org
X-Mailman-Version: 2.1.12
Precedence: list
Reply-To: ALT Linux Sisyphus discussions <sisyphus@lists.altlinux.org>
List-Id: ALT Linux Sisyphus discussions <sisyphus.lists.altlinux.org>
List-Unsubscribe: <https://lists.altlinux.org/mailman/options/sisyphus>,
	<mailto:sisyphus-request@lists.altlinux.org?subject=unsubscribe>
List-Archive: <http://lists.altlinux.org/pipermail/sisyphus>
List-Post: <mailto:sisyphus@lists.altlinux.org>
List-Help: <mailto:sisyphus-request@lists.altlinux.org?subject=help>
List-Subscribe: <https://lists.altlinux.org/mailman/listinfo/sisyphus>,
	<mailto:sisyphus-request@lists.altlinux.org?subject=subscribe>
X-List-Received-Date: Thu, 28 Feb 2013 11:30:47 -0000
Archived-At: <http://lore.altlinux.org/sisyphus/512F3FDA.3010703@altlinux.ru/>
List-Archive: <http://lore.altlinux.org/sisyphus/>
List-Post: <mailto:sisyphus@altlinux.ru>

28.02.2013 15:19, Pavel Vainerman wrote:
> 28.02.2013 15:15, Alexey Gladkov пишет:
>> 28.02.2013 14:29, Michael Shigorin wrote:
>>> On Thu, Feb 28, 2013 at 02:25:12PM +0400, Alexey Gladkov wrote:
>>>> Тот кто положил в этот файл такой мусор сломал всех кто хочет
>>>> использовать этот файл.
>>>
>>> Видимо, всё-таки source-ить придётся с обработкой ошибок.
>>
>> Видимо, заполнять его нужно с обработкой ошибок.

Вот выдержка:

The basic file format of os-release is a newline-separated list of
environment-like shell-compatible variable assignments. It is possible
to source the configuration from shell scripts.

>   Вроде общее правило в "безопасном программировании" - "не доверяй 
> ВХОДНЫМ данным"

У паранои должны быть пределы иначе можно перестать доверять выводу
системных утилит и содержимому конфигов.

В данном случае речь идёт о конфигурационном файле. Если
конфигурационный файл некорректный, то его нельзя прочитать и
выполнение программы может быть прекращено (зависит от важности
конфигурации).

В данном случае произошло прекращение выполнения. Я могу добавить
красочное сообщение о том, что конфигурация битая, но сути это не
изменит и то что не работает работать не начнёт.

> Логичнее всё-таки сделать обработку ошибок в том месте, где он 
> используется/парситься, т.к. в общем случая я так понимаю этот файл,
> может создавать "кто угодно" и править его в ручную в том числе..

Логичнее не писать фигню в конфиги. Даже если программы их
использующие продолжат работу, они будут действовать возможно не так
как ожидаете (выбор за ними).

-- 
Rgrds, legion