Bug 21229 - Разломан read
Summary: Разломан read
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: ash (show other bugs)
Version: unstable
Hardware: all Linux
: P3 critical
Assignee: Alexey Gladkov
QA Contact: qa-sisyphus
URL: http://comments.gmane.org/gmane.comp....
Keywords:
Depends on:
Blocks:
 
Reported: 2009-08-26 11:11 MSD by Andrey Rahmatullin
Modified: 2009-08-31 14:30 MSD (History)
5 users (show)

See Also:

Attachments
tuxonice.conf (1.25 KB, text/plain)
2009-08-26 20:38 MSD, Andrey Rahmatullin 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Rahmatullin 2009-08-26 11:11:30 MSD 
С ash-0.5.5.1-alt4 перестал работать hibernate, потому что читает из конфигов мусор.

#!/bin/dash -x
while true ; do
  read option params
  [ $? -ne 0 ] && [ -z "$option" ] && break
  [ -z "$option" ] && continue
  echo $option $params
done < /etc/hibernate/tuxonice.conf

читает, например,
FullSpeedCPU yesFle.
и
EnableEscape yesE
Comment 1 Alexey Gladkov 2009-08-26 13:17:34 MSD 
Конфиг в аттачмент добавьте.
Comment 2 Andrey Rahmatullin 2009-08-26 20:38:45 MSD 
Created attachment 3789 [details]
tuxonice.conf

Да, конечно.
Comment 3 Dmitry V. Levin 2009-08-27 00:10:21 MSD 
/usr/share/mkimage/tools/mki-sh-image-prepare тоже сломался от этого изменения 55c46b7286f5d9f2d8291158203e2b61d2494420.
На будущее: стоит дополнительно тестировать ash с помощью mkimage.
Comment 4 Alexey Gladkov 2009-08-27 01:10:18 MSD 
Исправление уже давно находится в очереди на сборку (кстати, уж очень долго).

Этот коммит 55c46b7 сломан довольно неочевидно в районе ifsbreakup. При этом проявляется эта ошибка проявляется не на всех строчках при чтении. Тест на эту ошибку должен быть хитрым.
Comment 5 Repository Robot 2009-08-27 02:31:24 MSD 
ash-0.5.5.1-alt5 -> sisyphus:

* Wed Aug 26 2009 Alexey Gladkov <legion@altlinux> 0.5.5.1-alt5

- Revert "Honor tab as IFS whitespace when splitting fields in readcmd" (ALT#21229).
Comment 6 Alexey Gladkov 2009-08-31 14:30:02 MSD 
Исправлено в апстриме.