ПРОЕКТЫ 


  АРХИВ 


Apache-Talk @lexa.ru 

Inet-Admins @info.east.ru 

Filmscanners @halftone.co.uk 

Security-alerts @yandex-team.ru 

nginx-ru @sysoev.ru 


  СТАТЬИ 


  ПЕРСОНАЛЬНОЕ 


  ПРОГРАММЫ 



ПИШИТЕ
ПИСЬМА












     АРХИВ :: nginx-ru
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



 




Copyright © Lexa Software, 1996-2009.