Nginx-ru mailing list archive (nginx-ru@sysoev.ru)
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] http_request: fix a segmentation fault
On Thu, May 28, 2009 at 02:07:02PM +0400, Kirill A. Korinskiy wrote:
> At Thu, 28 May 2009 08:55:18 +0400,
> Igor Sysoev <is@xxxxxxxxxxxxx> wrote:
> >
> > On Thu, May 28, 2009 at 08:43:24AM +0400, Igor Sysoev wrote:
> >
> > > Я пару раз наблюдал nginx в ситуации low memory и каждый раз он работал
> > > продолжительное время без падений (это не относится к nginx'у с перловым
> > > модулем, потому что перл в таких случаях падает очень легко).
> > >
> > > На самом деле, в этом месте проверки косметические, потому что выделения
> > > небольшие и делаются из только что выделенного пула, в котором по
> > > умолчанию 16К. Поэтому, чтобы уронить nginx в этом месте, нужно или
> > > создать
> > > около 1000 (для amd64) переменных в конфиге для дефолтного
> > > request_pool_size,
> > > или уменьшать пул: для 4K понадобится 250 переменных, что тоже не мало.
> >
> > Вернее, для 16К пула нужно тоже 250 переменных, а не 1000.
>
> собственно на этом месте оно у меня и бабахнуло.
То есть, в конфигурации было описано 250 переменных ?
> Т.е. это граничная ситуация, ага, до которой мало кто доберется. Но
> все-таки иметь даже ??косметические?? проверки которые дают сегфол ??? это
> не хорошо.
>
> Т.е. если сюда попасть нельзя, даже теоретически, то зачем там
> условие? А если можно ??? то все-таки нужно иметь корректную обработку.
Я согласен, что проверка там нужна и должно быть правильное завершение,
но на практике я такой ситуации просто не видел.
--
Игорь Сысоев
http://sysoev.ru
|
