]> git.kaiwu.me - nginx.git/commit
Fixed handling of non-null-terminated unix sockets.
authorMaxim Dounin <mdounin@mdounin.ru>
Wed, 4 Oct 2017 18:19:38 +0000 (21:19 +0300)
committerMaxim Dounin <mdounin@mdounin.ru>
Wed, 4 Oct 2017 18:19:38 +0000 (21:19 +0300)
commitf57527df8122eb8a99cc3bf419c28aa4d2a3cf9b
treed3e68baf3d19feb4aeb6874025ee6009cc4653ba
parentcb4dc2720acf49cb8870e5ff4ebd33ad08668e67
Fixed handling of non-null-terminated unix sockets.

At least FreeBSD, macOS, NetBSD, and OpenBSD can return unix sockets
with non-null-terminated sun_path.  Additionally, the address may become
non-null-terminated if it does not fit into the buffer provided and was
truncated (may happen on macOS, NetBSD, and Solaris, which allow unix socket
addresess larger than struct sockaddr_un).  As such, ngx_sock_ntop() might
overread the sockaddr provided, as it used "%s" format and thus assumed
null-terminated string.

To fix this, the ngx_strnlen() function was introduced, and it is now used
to calculate correct length of sun_path.
src/core/ngx_inet.c
src/core/ngx_string.c
src/core/ngx_string.h