upstream module / need to check pipe field

Maxim Dounin mdounin at mdounin.ru
Mon Dec 21 15:33:41 UTC 2015


Hello!

On Mon, Dec 21, 2015 at 03:07:10PM +0000, David CARLIER wrote:

> I was wondering if it might be better to check if pipe field before
> attempting to clear the temp file ? Anyway it is checked as well line
> 4015 and below ... in this case u->pipe checks is sufficient
> u->pipe->temp_file is checked inside ngx_http_file_cache_free
> function.

The check for u->pipe is needed only in places where we don't know 
if u->pipe exists or not.

-- 
Maxim Dounin
http://nginx.org/



More information about the nginx-devel mailing list