[PATCH] http_request: fix a segmentation fault

[Igor Sysoev]

On Thu, May 28, 2009 at 08:43:24AM +0400, Igor Sysoev wrote:

> On Wed, May 27, 2009 at 01:57:32PM +0400, Kirill A. Korinskiy wrote:
> 
> > At Wed, 27 May 2009 04:19:05 +0400,
> > Maxim Dounin <mdounin at mdounin.ru> wrote:
> > > 
> > > 
> > > Не то чтобы совсем не попал, но не попал в шарик.
> > > 
> > 
> > В общем третий вариант патча, и итоговый вариант. Да, с ночным
> > хаченьем пора завязывать. Ну или ждать до утра.
> 
> Я пару раз наблюдал nginx в ситуации low memory и каждый раз он работал
> продолжительное время без падений (это не относится к nginx'у с перловым
> модулем, потому что перл в таких случаях падает очень легко). 
> 
> На самом деле, в этом месте проверки косметические, потому что выделения
> небольшие и делаются из только что выделенного пула, в котором по
> умолчанию 16К. Поэтому, чтобы уронить nginx в этом месте, нужно или создать
> около 1000 (для amd64) переменных в конфиге для дефолтного request_pool_size,
> или уменьшать пул: для 4K понадобится 250 переменных, что тоже не мало.

Вернее, для 16К пула нужно тоже 250 переменных, а не 1000.


-- 
Игорь Сысоев
http://sysoev.ru