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

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

[Sergei Epiphanov]

Сегодня наткнулся на одну баго-фичу: недавнее обновление make-initrd научилось 
работать с файлом /etc/os-release. Но в моем файле почему-то вместо правильной 
информации (в виде shell-переменных) оказалась строчка

Simply Linux 6.990.0  (Flounder)

явно взятая из altlinux-release. Из-за этого обработчик make-initrd-release 
формирует ошибку и сборка initrd обваливается. Обнаружилось это только при 
перезагрузке системы.

Какая из программ сделала этот файл и почему - не могу установить. Последняя 
точка правки этого файла - 25 июня 2012 года, то есть бомба провисела почти 
год, прежде чем рванула.

$ rpm -qa|grep simply
branding-simply-linux-indexhtml-6.991.0-alt2
branding-simply-linux-alterator-6.991.0-alt2
branding-simply-linux-release-6.991.0-alt2
branding-simply-linux-xfce-settings-6.991.0-alt2
branding-simply-linux-bootloader-6.991.0-alt2
branding-simply-linux-slideshow-6.991.0-alt2
branding-simply-linux-bootsplash-6.991.0-alt2
branding-simply-linux-notes-6.991.0-alt2
docs-simply-linux-6.0.1-alt3
branding-simply-linux-graphics-6.991.0-alt2
branding-simply-linux-menu-6.991.0-alt2
$

И непонятно, вешать ли на что-то (да и на что?) баг или принять это как 
должное? Наверно, как минимум на make-initrd за то, что не обрабатывает 
исключительную ситуацию по /etc/os-release.

-- 
С уважением, Епифанов Сергей

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

[Alexey Gladkov]

28.02.2013 09:30, Sergei Epiphanov wrote:
> Сегодня наткнулся на одну баго-фичу: недавнее обновление make-initrd научилось 
> работать с файлом /etc/os-release. Но в моем файле почему-то вместо правильной 
> информации (в виде shell-переменных) оказалась строчка

Вы можете приаттачить ваш /etc/os-release ?

-- 
Rgrds, legion

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

[Vladimir Lettiev]

On Thu, Feb 28, 2013 at 11:11:08AM +0400, Alexey Gladkov wrote:
> 28.02.2013 09:30, Sergei Epiphanov wrote:
> > Сегодня наткнулся на одну баго-фичу: недавнее обновление make-initrd научилось 
> > работать с файлом /etc/os-release. Но в моем файле почему-то вместо правильной 
> > информации (в виде shell-переменных) оказалась строчка
> 
> Вы можете приаттачить ваш /etc/os-release ?

У меня, например, в первой строке не хватает кавычек. При обновлении ядра
make-initrd не смог обработать файл.

NAME=ALT Linux
VERSION="6.9.0 alpha (Cheiron)"
ID=altlinux
VERSION_ID=6.9.0
PRETTY_NAME="ALT Linux 6.9.0 Centaurus alpha (Cheiron)"
ANSI_COLOR="1;33"
CPE_NAME="cpe:/o:alt linux:centaurus:6.9.0"
HOME_URL="http://altlinux.ru/"
BUG_REPORT_URL="https://bugs.altlinux.org/"

-- 
Vladimir Lettiev aka crux ✉ theCrux@gmail.com

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

[Sergei Epiphanov]

[-- Attachment #1: Type: text/plain, Size: 706 bytes --]

On 28 февраля 2013 11:11 Alexey Gladkov wrote:
> 28.02.2013 09:30, Sergei Epiphanov wrote:
> > Сегодня наткнулся на одну баго-фичу: недавнее обновление make-initrd
> > научилось работать с файлом /etc/os-release. Но в моем файле почему-то
> > вместо правильной информации (в виде shell-переменных) оказалась строчка
> 
> Вы можете приаттачить ваш /etc/os-release ?

Я уже написал выше, что в этом файле. :) Но файл приаттачил.

-- 
С уважением, Епифанов Сергей

[-- Attachment #2: os-release --]
[-- Type: text/plain, Size: 33 bytes --]

Simply Linux 6.990.0  (Flounder)

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

[Alexey Gladkov]

28.02.2013 14:06, Sergei Epiphanov wrote:
> On 28 февраля 2013 11:11 Alexey Gladkov wrote:
>> 28.02.2013 09:30, Sergei Epiphanov wrote:
>>> Сегодня наткнулся на одну баго-фичу: недавнее обновление make-initrd
>>> научилось работать с файлом /etc/os-release. Но в моем файле почему-то
>>> вместо правильной информации (в виде shell-переменных) оказалась строчка
>>
>> Вы можете приаттачить ваш /etc/os-release ?
> 
> Я уже написал выше, что в этом файле. :) Но файл приаттачил.

Ужас. Ни у вас, ни у Vladimir Lettiev этот файл имеет некорректное
содержимое. У вас оно совершенно неправильное, а у Владимира оно
содержит ошибки.

Вот спецификация на него:

http://www.freedesktop.org/software/systemd/man/os-release.html

Тот кто положил в этот файл такой мусор сломал всех кто хочет
использовать этот файл.

-- 
Rgrds, legion

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

[Андрей Черепанов]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

28.02.2013 12:12, Vladimir Lettiev пишет:
> On Thu, Feb 28, 2013 at 11:11:08AM +0400, Alexey Gladkov wrote:
>> 28.02.2013 09:30, Sergei Epiphanov wrote:
>>> Сегодня наткнулся на одну баго-фичу: недавнее обновление
>>> make-initrd научилось работать с файлом /etc/os-release. Но в
>>> моем файле почему-то вместо правильной информации (в виде
>>> shell-переменных) оказалась строчка
>> 
>> Вы можете приаттачить ваш /etc/os-release ?
> 
> У меня, например, в первой строке не хватает кавычек. При
> обновлении ядра make-initrd не смог обработать файл.
> 
> NAME=ALT Linux VERSION="6.9.0 alpha (Cheiron)"
Уже исправлено в branding-altlinux-centaurus-6.9.0-alt17.src.rpm
* Вт фев 26 2013 Andrey Cherepanov <cas@altlinux.org> 6.9.0-alt17
- - Fix Name in os-release

Обновитесь.
> ID=altlinux VERSION_ID=6.9.0 PRETTY_NAME="ALT Linux 6.9.0 Centaurus
> alpha (Cheiron)" ANSI_COLOR="1;33" CPE_NAME="cpe:/o:alt
> linux:centaurus:6.9.0" HOME_URL="http://altlinux.ru/" 
> BUG_REPORT_URL="https://bugs.altlinux.org/"
> 


- -- 
Андрей Черепанов
ALT Linux
cas@altlinux.ru
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.13 (GNU/Linux)

iEYEARECAAYFAlEvMOsACgkQSGPA9Knr8THpawCeLpvI8QkSxSdlhDtJK76sRper
ks8AoKG3XBDMEDWJMdeGTMjfaYyHHH7k
=m/2L
-----END PGP SIGNATURE-----

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

[Michael Shigorin]

On Thu, Feb 28, 2013 at 02:25:12PM +0400, Alexey Gladkov wrote:
> Тот кто положил в этот файл такой мусор сломал всех кто хочет
> использовать этот файл.

Видимо, всё-таки source-ить придётся с обработкой ошибок.

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

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

[Alexey Gladkov]

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

Видимо, заполнять его нужно с обработкой ошибок.

-- 
Rgrds, legion

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

[Pavel Vainerman]

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-ить придётся с обработкой ошибок.
>
> Видимо, заполнять его нужно с обработкой ошибок.

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

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



-- 
Pavel Vaynerman
www.etersot.ru

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

[Alexey Gladkov]

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

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

[Pavel Vainerman]

28.02.2013 15:30, Alexey Gladkov пишет:
> 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.
>
>>    Вроде общее правило в "безопасном программировании" - "не доверяй
>> ВХОДНЫМ данным"
>
> У паранои должны быть пределы иначе можно перестать доверять выводу
> системных утилит и содержимому конфигов.
    согласен.. при условии (того, что вы написали ниже).

 > Я могу добавить красочное сообщение о том, что конфигурация битая
   это наверно то, чего не хватает..
если из этого сообщения будет понятно что битый файл os-release
(ну или что где то в нём ошибка)..


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

   Речь собственно шла не о "продолжить работу", а о диагностировании ошибки
( и понятном сообщении..)


-- 
Pavel Vaynerman
www.etersot.ru

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

[Aleksey Novodvorsky]

28 февраля 2013 г., 15:30 пользователь Alexey Gladkov
<legion@altlinux.ru> написал:
> Логичнее не писать фигню в конфиги. Даже если программы их
> использующие продолжат работу, они будут действовать возможно не так
> как ожидаете (выбор за ними).

+1

Rgrds, Алексей

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

[Alexey Gladkov]

28.02.2013 15:35, Pavel Vainerman wrote:
>> У паранои должны быть пределы иначе можно перестать доверять выводу
>> системных утилит и содержимому конфигов.
>     согласен.. при условии (того, что вы написали ниже).

Я продемонстрирую почему я так толсто троллингово возмущаюсь:

$ dd if=/dev/urandom of=/tmp/badfile count=1 bs=512

$ cat /tmp/test.sh
#!/bin/sh -efu

. /tmp/badfile || echo "BAD FILE"
echo "END"

$ /tmp/test.sh
/tmp/badfile: line 1: unexpected EOF while looking for matching ``'

$ echo "Foo bar baz" > /tmp/badfile

$ LANG=C /tmp/test.sh
/tmp/badfile: line 1: Foo: command not found


Шеллы могут быть любыми, но результат будет один: синтаксическая ошибка.

Таким образом нарушается базовое условие формата этого конфига и
обработать такой сорс можно только через subshell:

$ cat /tmp/test.sh
#!/bin/sh -efu

( . /tmp/badfile ) || echo "BAD FILE"
echo "END"

$ /tmp/test.sh
/tmp/zzz: line 1: Foo: command not found
BAD FILE
END

Но в этом случае переменные из этого конфига не забрать простым
способом и это всё равно, что выуживать их sed'ом по одной. Как раз
этого хотели избежать создатели формата этого файла.

А учитывая, что формат допускает вот такие конструкции:

Example: NAME=Fedora or NAME="Debian GNU/Linux".

получение значений sed'ом делается ещё более весёлой задачей.

> если из этого сообщения будет понятно что битый файл os-release
> (ну или что где то в нём ошибка)..

Как я показал выше показать что именно файл битый можно, но выглядеть
эта проверка будет очень забавно. Безусловно я добавлю её раз возник
прецедент с записью туда мусора.

-- 
Rgrds, legion

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

[Pavel Vainerman]

28.02.2013 15:59, Alexey Gladkov пишет:
...
> Шеллы могут быть любыми, но результат будет один: синтаксическая ошибка.
>
   Убедили. В Shell-скриптах действительно своя специфика.

...
> Как я показал выше показать что именно файл битый можно, но выглядеть
> эта проверка будет очень забавно. Безусловно я добавлю её раз возник
> прецедент с записью туда мусора.
  +1

Если это всё-таки возможно.. то было бы хорошо.


-- 
Pavel Vaynerman
www.etersot.ru

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

[Michael Shigorin]

On Thu, Feb 28, 2013 at 03:59:26PM +0400, Alexey Gladkov wrote:
> обработать такой сорс можно только через subshell:

А, вот как.  Спасибо.

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

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

[Alexey Gladkov]

28.02.2013 16:03, Pavel Vainerman wrote:
>> Как я показал выше показать что именно файл битый можно, но выглядеть
>> эта проверка будет очень забавно. Безусловно я добавлю её раз возник
>> прецедент с записью туда мусора.
>   +1
> 
> Если это всё-таки возможно.. то было бы хорошо.

Нет ничего невозможного, есть трудно достижимое :)

http://git.altlinux.org/people/legion/packages/make-initrd.git?p=make-initrd.git;a=commitdiff;h=891b6c03bc3c574db56c0f48342bd18f5ab0bc3f

-- 
Rgrds, legion

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

[Pavel Vainerman]

28.02.2013 16:26, Alexey Gladkov пишет:
> 28.02.2013 16:03, Pavel Vainerman wrote:
>>> Как я показал выше показать что именно файл битый можно, но выглядеть
>>> эта проверка будет очень забавно. Безусловно я добавлю её раз возник
>>> прецедент с записью туда мусора.
>>    +1
>>
>> Если это всё-таки возможно.. то было бы хорошо.
>
> Нет ничего невозможного, есть трудно достижимое :)
>
> http://git.altlinux.org/people/legion/packages/make-initrd.git?p=make-initrd.git;a=commitdiff;h=891b6c03bc3c574db56c0f48342bd18f5ab0bc3f

   Элегантно.. Спасибо.


-- 
Pavel Vaynerman
www.etersot.ru

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

[Sergei Epiphanov]

On 28 февраля 2013 14:25 Alexey Gladkov wrote:
> Ужас. Ни у вас, ни у Vladimir Lettiev этот файл имеет некорректное
> содержимое. У вас оно совершенно неправильное, а у Владимира оно
> содержит ошибки.

О том и речь. Причём эту ошибку создал скрипт/файлтриггер/что-то ещё, 
упакованный _системно_ в Сизиф. Я просто хочу обратить внимание на 
аккуратность написания _системных_ конфигов, отвечающих за работу _базовых_ 
элементов системы. Ошибка в конфигурации демона сломает только сам демон.

Но узнал-то я об этом только постфактум.

-- 
С уважением, Епифанов Сергей

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

[Sergei Epiphanov]

On 28 февраля 2013 15:30 Alexey Gladkov wrote:
> В данном случае произошло прекращение выполнения. Я могу добавить
> красочное сообщение о том, что конфигурация битая, но сути это не
> изменит и то что не работает работать не начнёт.

Нужно сделать хотя бы это. Я еле разобрался, откуда ноги растут.

-- 
С уважением, Епифанов Сергей

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

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 977 bytes --]

On Thu, Feb 28, 2013 at 04:26:27PM +0400, Alexey Gladkov wrote:
> 28.02.2013 16:03, Pavel Vainerman wrote:
> >> Как я показал выше показать что именно файл битый можно, но выглядеть
> >> эта проверка будет очень забавно. Безусловно я добавлю её раз возник
> >> прецедент с записью туда мусора.
> >   +1
> > 
> > Если это всё-таки возможно.. то было бы хорошо.
> 
> Нет ничего невозможного, есть трудно достижимое :)
> 
> http://git.altlinux.org/people/legion/packages/make-initrd.git?p=make-initrd.git;a=commitdiff;h=891b6c03bc3c574db56c0f48342bd18f5ab0bc3f

Файл с синтаксисом shell-скрипта, конечно, засорсится, но если там
окажется синтаксически корректный мусор, то результат может оказаться
еще веселее.  Например:

echo halt > /etc/os-release

Другими словами, мусор в системных файлах легко может разрушить всю
систему.  Проверка синтаксиса - это, конечно, хорошо, но не отменяет
необходимости поддерживать системные файлы в порядке.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]