Hello!

On Wed, Mar 03, 2010 at 11:04:17PM +0300, Igor Sysoev wrote:

> On Wed, Mar 03, 2010 at 10:58:19PM +0300, Maxim Dounin wrote:
>
> > Hello!
> >
> > On Wed, Mar 03, 2010 at 10:48:10PM +0300, Igor Sysoev wrote:
> >
> > > On Wed, Mar 03, 2010 at 10:46:05PM +0300, Igor Sysoev wrote:
> > >
> > > > On Wed, Mar 03, 2010 at 09:36:56PM +0300, Vladimir Sopot wrote:
> > > >
> > > > > Приветствую!
> > > > >
> > > > > А для проблемки c memcached (http://www.lexa.ru/nginx-ru/msg32094.html) снова надо будет накатывать патч?
> > > >
> > > > Пока да. Мне нужно
> > > > 1) воспроизвести ошибку на 0.6.14
> > >
> > > Вернее, воспроизвести ту ошибку на 0.6.13, которая была пофикшена в 0.6.14.
> >
> > Вроде ж я ещё тогда тест нарисовал и вроде даже ссылку на него
> > давал, нет?
> >
> > http://mdounin.ru/hg/nginx-tests/rev/25047dc6ae9b
> >
> > Или ты обязательно хочешь с бесконечным циклом во writer'е?
>
> Да, мне нужно убедиться, что твой патч фиксит обе баги.

Ну я вроде бы объяснил - это одна и та же бага, только результаты
на выходе разные. Бесконечный цикл возникает когда на выход
отправляется буфер отрицательного размера. А до кода чтения
остатков trailer'а там вообще дело не доходит, он в 0.6.13 был
правильный (почти - выход всё-таки лучше делать).

Но если очень хочется - почему нет? Только в test suite я это
включать не буду, он у меня глупый и SIGKILL'ом убежавшие
процессы бить не умеет. :)

Тест прилагается.

На 0.6.13 даёт цикл:

2010/03/04 03:27:14 [debug] 45924#0: *1 memcached filter bytes:2006 size:4016 length:2015 rest:7
2010/03/04 03:27:14 [error] 45924#0: *1 memcached sent invalid trailer while reading upstream, client: 127.0.0.1, server: loc
alhost, URL: "/", upstream: "memcached://127.0.0.1:8081", host: "localhost"
2010/03/04 03:27:14 [debug] 45924#0: *1 http output filter "/?"
2010/03/04 03:27:14 [debug] 45924#0: *1 copy filter: "/?"
2010/03/04 03:27:14 [debug] 45924#0: *1 http postpone filter "/?" 080F19C8
2010/03/04 03:27:14 [debug] 45924#0: *1 http postpone filter out "/?"
2010/03/04 03:27:14 [debug] 45924#0: *1 write old buf t:0 f:0 00000000, pos 080F6400, size: 986 file: 0, size: 0
2010/03/04 03:27:14 [debug] 45924#0: *1 write new buf t:0 f:0 00000000, pos 080F67DA, size: -2 file: 0, size: 0
....
2010/03/04 03:27:14 [debug] 45924#0: *1 writev: 984 of 984
2010/03/04 03:27:14 [debug] 45924#0: *1 writev: 0 of 0
2010/03/04 03:27:14 [debug] 45924#0: *1 writev: 0 of 0
2010/03/04 03:27:14 [debug] 45924#0: *1 writev: 0 of 0
2010/03/04 03:27:14 [debug] 45924#0: *1 writev: 0 of 0
2010/03/04 03:27:14 [debug] 45924#0: *1 writev: 0 of 0
2010/03/04 03:27:14 [debug] 45924#0: *1 writev: 0 of 0
....

На 0.8.34 без патча - цикла нет, но есть ругань на invalid trailer
при его дочитывании:

....
2010/03/04 03:31:22 [debug] 45970#0: *1 recv: eof:0, avail:4, err:0
2010/03/04 03:31:22 [debug] 45970#0: *1 recv: fd:8 4 of 7
2010/03/04 03:31:22 [error] 45970#0: *1 memcached sent invalid trailer while reading upstream, client: 127.0.0.1, server: loc
alhost, request: "GET / HTTP/1.0", upstream: "memcached://127.0.0.1:8081", host: "localhost"
....

На 0.8.34 с патчем - ни циклов, ни ругани.

Maxim Dounin
_______________________________________________
nginx-ru mailing list
nginx-ru@nginx.org
http://nginx.org/mailman/listinfo/nginx-ru