atom feed9 messages in ru.sysoev.nginxRe: [PATH] support for UNIX socket in...
FromSent OnAttachments
Roberto De IorisMay 23, 2011 7:18 am.patch
Igor SysoevMay 23, 2011 9:35 am 
Roberto De IorisMay 23, 2011 10:41 am 
Roberto De IorisMay 23, 2011 10:43 am 
Igor SysoevMay 23, 2011 10:48 am 
Igor SysoevMay 23, 2011 10:51 am.zero
Roberto De IorisMay 23, 2011 11:17 am 
Igor SysoevMay 23, 2011 11:36 am 
nri.plSep 17, 2012 12:04 am 
Subject:Re: [PATH] support for UNIX socket in abstract namespace
From:Igor Sysoev (ig@sysoev.ru)
Date:May 23, 2011 11:36:46 am
List:ru.sysoev.nginx

On Mon, May 23, 2011 at 08:17:46PM +0200, Roberto De Ioris wrote:

On Mon, May 23, 2011 at 07:42:02PM +0200, Roberto De Ioris wrote:

On Mon, May 23, 2011 at 04:18:30PM +0200, Roberto De Ioris wrote:

This is the same code i made after you suggestion, but in the url parsing code (ngx_parse_unix_domain_url), ngx_cpystrn() is used and it takes '\0' in account.

Changing ngx_cpystrn to not use \0 fixes the problem. Probably substituting it with ngx_memcpy is the way to go.

Another thing to do is "fixing"

u->addrs[0].socklen = sizeof(struct sockaddr_un);

u->addrs[0].socklen = sizeof(saun->sun_family)+len;

nxg_memcpy should work here.