[Comm] alm2.4 "watch" bug?

[Comm] alm2.4 "watch" bug?

[Dmitry Vodennikov]

Добрый день.

Сейчас совершенно случайно обнаружил следующее:

[vod@mse vod]$ set | grep LANG
LANG=ru_RU.CP1251
LANGUAGE=ru_RU.CP1251

[vod@mse vod]$ ls /
bin   dev  home  mnt  proc  sbin  sys  usr
boot  etc  lib   opt  root  swap  tmp  var

[vod@mse vod]$ watch ls /
ну собственно содержимое корня, обновляется каждые 2 секунды

[vod@mse vod]$ ls ~/
bin      Documents  ipt.log   Mail  RPM       tmp     watch.log
Desktop  files      ipt.log~  mbox  test.log  vmware

[vod@mse vod]$ watch ls ~/
Segmentation fault

Во как.... Но нас голыми руками не возьмешь...

[vod@mse vod]$ export LANG=POSIX
[vod@mse vod]$ export LANGUAGE=POSIX

[vod@mse vod]$ watch ls ~/
и все работает как положено....


Хм. А если так:

[vod@mse vod]$ export LANG=ru_RU.KOI8-R
[vod@mse vod]$ export LANUAGE=ru_RU.KOI8-R
[vod@mse vod]$ watch ls ~/
и опять-таки все работает как положено....

[vod@mse vod]$ uname -a
Linux mse.elecom.ru 2.4.26-std-up-alt13 #1 Mon Nov 14 00:25:14 MSK 2005 
i686 unknown unknown GNU/Linux
[vod@mse vod]$ rpm -qf /usr/bin/watch
procps-2.0.10-alt4.1

sudo apt-get update, sudo apt-get upgrade сделан,
из backports и тем более сизифа ничего не берется.

На что бы такое повесить багу???

Re: [Comm] alm2.4 "watch" bug?

[Evgenii Terechkov]

Dmitry Vodennikov пишет:

> [vod@mse vod]$ set | grep LANG
> LANG=ru_RU.CP1251
> LANGUAGE=ru_RU.CP1251
> [vod@mse vod]$ ls /
> bin   dev  home  mnt  proc  sbin  sys  usr
> boot  etc  lib   opt  root  swap  tmp  var
> [vod@mse vod]$ watch ls /
> ну собственно содержимое корня, обновляется каждые 2 секунды
> [vod@mse vod]$ ls ~/
> bin      Documents  ipt.log   Mail  RPM       tmp     watch.log
> Desktop  files      ipt.log~  mbox  test.log  vmware
> [vod@mse vod]$ watch ls ~/
> Segmentation fault
> Во как.... Но нас голыми руками не возьмешь...
> [vod@mse vod]$ export LANG=POSIX
> [vod@mse vod]$ export LANGUAGE=POSIX

Насколько помню, переменная LANGUAGE уже не используется.

> [vod@mse vod]$ watch ls ~/
> и все работает как положено....
> Хм. А если так:
> [vod@mse vod]$ export LANG=ru_RU.KOI8-R
> [vod@mse vod]$ export LANUAGE=ru_RU.KOI8-R
> [vod@mse vod]$ watch ls ~/
> и опять-таки все работает как положено....
> [vod@mse vod]$ uname -a
> Linux mse.elecom.ru 2.4.26-std-up-alt13 #1 Mon Nov 14 00:25:14 MSK 2005 
> i686 unknown unknown GNU/Linux
> [vod@mse vod]$ rpm -qf /usr/bin/watch
> procps-2.0.10-alt4.1
> sudo apt-get update, sudo apt-get upgrade сделан,
> из backports и тем более сизифа ничего не берется.

Могу только подтвердить такой глюк. Только у меня локаль KOI8-R. И ядро
2.6.8. Пакет procps тот-же.

> На что бы такое повесить багу???

Или на procps или на glibc-locales :-).

Re: [Comm] alm2.4 "watch" bug?

[Andrey Rahmatullin]

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

* Срд Авг 11 2004 Stanislav Ievlev <inger@altlinux> 2.0.10-alt4.1
- setlocale in watch to see output with national symbols (e.g. russian)

Не оно?

-- 
WBR, wRAR (ALT Linux Team)
Powered by the ALT Linux fortune(8):

> Что это такое случилось с приходом WindowMaker-0.80?
Нет, это я экспериментирую. :)
		-- ldv in devel@

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

Re: [Comm] alm2.4 "watch" bug?

[Evgenii Terechkov]

Andrey Rahmatullin пишет:

> * Срд Авг 11 2004 Stanislav Ievlev <inger@altlinux> 2.0.10-alt4.1
> - setlocale in watch to see output with national symbols (e.g. russian)
> Не оно?

Похоже оно. Пересборка из Сизифа разрешает вопрос (и наверно добавляет
других :-). И top новый, типа цветной :-)

Re: [Comm] alm2.4 "watch" bug?

[Dmitry Vodennikov]

Andrey Rahmatullin пишет:
> * Срд Авг 11 2004 Stanislav Ievlev <inger@altlinux> 2.0.10-alt4.1
> - setlocale in watch to see output with national symbols (e.g. russian)
> 
> Не оно?
> 
Не похоже.
[vod@alm24 procps-2.0.10]$  rpm --changelog procps | less
начинается именно с этого
------------------------------
* Срд Авг 11 2004 Stanislav Ievlev <inger@altlinux.org> 2.0.10-alt4.1

- setlocale in watch to see output with national symbols (e.g. russian)
------------------------------
И тем не менее, проблема есть. Резюмируя, дело обстоит так
watch ls /
у меня работает во всех трех случаях - LANG=POSIX, LANG=ru_RU.KOI8-R, 
LANG=ru_RU.CP1251
а вот
watch ls ~/
работает только при LANG=POSIX, LANG=ru_RU.KOI8-R

Попробую на выходных через gdb
strace говорит вот что
------------------------------
7145  getuid32()                        = 500
7145  geteuid32()                       = 500
7145  access("/home/vod/.terminfo/x/xterm", R_OK) = -1 ENOENT (No such 
file or d
irectory)
7145  access("/lib/terminfo/x/xterm", R_OK) = 0
7145  open("/lib/terminfo/x/xterm", O_RDONLY) = 5
7145  read(5, "\32\1\34\0\35\0\17\0i\1\232\4", 12) = 12
7145  --- SIGSEGV (Segmentation fault) @ 0 (0) ---
7145  +++ killed by SIGSEGV +++

Re: [Comm] alm2.4 "watch" bug?

[Andrey Rahmatullin]

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

On Fri, Mar 31, 2006 at 09:53:36PM -0800, Dmitry Vodennikov wrote:
> watch ls /
> у меня работает во всех трех случаях - LANG=POSIX, LANG=ru_RU.KOI8-R, 
> LANG=ru_RU.CP1251
> а вот
> watch ls ~/
> работает только при LANG=POSIX, LANG=ru_RU.KOI8-R
wrar@wrars-comp ~ $ echo $LANG
ru_RU.CP1251
wrar@wrars-comp ~ $ rpm -q procps
procps-3.2.5-alt6

Всё работает.
А с другими данными с кириллицей оно себя так же ведет?

-- 
WBR, wRAR (ALT Linux Team)
Powered by the ALT Linux fortune(8):

И вообще, если бы все [опенсорсовые] девелоперы вначале читали всю
документацию, а потом садились писать... Камня на камне бы не осталось
:-)
		-- morozov in devel@

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

Re: [Comm] alm2.4 "watch" bug?

[Michael Shigorin]

On Fri, Mar 31, 2006 at 09:53:36PM -0800, Dmitry Vodennikov wrote:
> > * Срд Авг 11 2004 Stanislav Ievlev <inger@altlinux> 2.0.10-alt4.1
> > - setlocale in watch to see output with national symbols (e.g. russian)
> > Не оно?
> Не похоже.

Как по мне -- так похоже, в корне-то не-ASCII нет небось.
Повесьте эти письма в bugzilla на ALM2.4, что ли.

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

Re: [Comm] alm2.4 "watch" bug?

[Andrey Rahmatullin]

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

On Mon, Apr 03, 2006 at 12:43:13PM +0300, Michael Shigorin wrote:
> Как по мне -- так похоже, в корне-то не-ASCII нет небось.
Потому и вспомнил, что до того изменения watch показывал вместо рус. букв
то ли вопросы, то ли пробелы. Не падал, впрочем.

-- 
WBR, wRAR (ALT Linux Team)
Powered by the ALT Linux fortune(8):

> Стебетесь?
да. а Вы?
		-- gns in devel@

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

Re: [Comm] alm2.4 "watch" bug?

[Evgenii Terechkov]

Andrey Rahmatullin пишет:

>> Как по мне -- так похоже, в корне-то не-ASCII нет небось.
> Потому и вспомнил, что до того изменения watch показывал вместо рус.
> букв
> то ли вопросы, то ли пробелы. Не падал, впрочем.

Оно вроде. Testcase автора оригинального поста у меня полностью
повторяем (был). Система тоже как у него. Обновление procps из сизифа
помогло.

P.S.: Письмо, блин, не дошло :-( предыдущее.

Re: [Comm] alm2.4 "watch" bug?

[Dmitry Vodennikov]

Michael Shigorin пишет:
> On Fri, Mar 31, 2006 at 09:53:36PM -0800, Dmitry Vodennikov wrote:
>>> * Срд Авг 11 2004 Stanislav Ievlev <inger@altlinux> 2.0.10-alt4.1
>>> - setlocale in watch to see output with national symbols (e.g. russian)
>>> Не оно?
>> Не похоже.
> 
> Как по мне -- так похоже, в корне-то не-ASCII нет небось.
> Повесьте эти письма в bugzilla на ALM2.4, что ли.
> 
Я щас повешу багу с локализованной проблемой и решением.

Проблема с watch.c состоит в следующем. Смотрим watch.c
-------------------------------------------------
         command = strdup(argv[optind++]);
         command_length = strlen(command);
         for (; optind < argc; optind++) {
                 int s = strlen(argv[optind]);
                 char *endp = &command[command_length];
                 *endp = ' ';
                 command_length += s + 1;
                 command = realloc(command, command_length + 1);
                 strcpy(endp + 1, argv[optind]);
         }
-------------------------------------------------
man realloc говорит нам вот что
realloc() returns a pointer to the newly  allocated  memory,  which  is
suitably  aligned  for  any  kind of variable and may be different from
ptr
собственно, так оно и видно под отладчиком. realloc возвращает 
совершенно другой указатель, что в сочетании с установкой endp ДО 
realloc и записью в *endp ПОСЛЕ realloc приводит к неожиданным эффектам.

у меня например это приводило к segmentation fault, если параметр у ls 
длиннее 8 символов. Т.е.
export LANG=ru_RU.CP1251 watch ls 12345678
работает, а
export LANG=ru_RU.CP1251 watch ls 123456789
уже нет

Ну собственно остается только сказать что в сизифном procsp этот кусок 
уже переделан вот так:
-------------------------------------------------
                 char *endp;
                 int s = strlen(argv[optind]);
                 command = realloc(command, command_length + s + 2); 
  /* space and \0 */
                 endp = command + command_length;
                 *endp = ' ';
                 memcpy(endp + 1, argv[optind], s);
                 command_length += 1 + s;        /* space then string 
length */
                 command[command_length] = '\0';
-------------------------------------------------

Re: [Comm] alm2.4 "watch" bug?

[Michael Shigorin]

On Tue, Apr 04, 2006 at 01:36:28AM -0700, Dmitry Vodennikov wrote:
> >Повесьте эти письма в bugzilla на ALM2.4, что ли.
> Я щас повешу багу с локализованной проблемой и решением.

Дмитрий, вот таких людей в команде никогда не будет хватать!
(а ещё -- нам -- клиентов, которые говорят о проблеме и добавляют
"я тут погуглил..." :)

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

Re: [Comm] alm2.4 "watch" bug?

[Dmitry Vodennikov]

Michael Shigorin пишет:
> On Tue, Apr 04, 2006 at 01:36:28AM -0700, Dmitry Vodennikov wrote:
>>> Повесьте эти письма в bugzilla на ALM2.4, что ли.
>> Я щас повешу багу с локализованной проблемой и решением.
> 
> Дмитрий, вот таких людей в команде никогда не будет хватать!
> (а ещё -- нам -- клиентов, которые говорят о проблеме и добавляют
> "я тут погуглил..." :)
> 
Да ладно вам :)

P.S.
Видимо, тред считаем законченым.