[PATCH] Fix a memory invalid read issue in ngx_http_gzip_ok

Maxim Dounin mdounin at mdounin.ru
Thu Dec 22 13:21:28 UTC 2011


Hello!

On Thu, Dec 22, 2011 at 08:18:41PM +0800, agentzh wrote:

> On Thu, Dec 22, 2011 at 5:00 PM, Maxim Dounin <mdounin at mdounin.ru> wrote:
> > I suppose the problem is instead in your code which (incorrectly)
> > adds headers to r->headers_in.
> >
> 
> I do strongly suggest we *should* support manipulating request
> headers, it's often desired by many people in real world settings :)

As I already said more than once, I don't think we should.  If 
it's needed - it means something else needs fixing.  We do support 
arbitrary modification of headers passed to upstream servers, and 
this should be enough.

But, actually, nobody stops you from adding headers correctly, 
i.e. null-terminated.

Maxim Dounin



More information about the nginx-devel mailing list