[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