Bug 21229

Summary: Разломан read
Product: Sisyphus Reporter: Andrey Rahmatullin <wrar>
Component: ashAssignee: Alexey Gladkov <legion>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: critical    
Priority: P3 CC: glebfm, ldv, legion, placeholder, rider
Version: unstable   
Hardware: all   
OS: Linux   
URL: http://comments.gmane.org/gmane.comp.shells.dash/174
Attachments:
Description Flags
tuxonice.conf none

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
Исправлено в апстриме.