[sisyphus] Вопрос по os-release

Alexey Gladkov legion на altlinux.ru
Чт Фев 28 15:30:34 MSK 2013


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



Подробная информация о списке рассылки Sisyphus