[PATCH] http_request: fix a segmentation fault

[Igor Sysoev]

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 переменных, что тоже не мало.


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