]> git.kaiwu.me - nginx.git/commit
Fixed handling of unix sockets in $binary_remote_addr.
authorMaxim Dounin <mdounin@mdounin.ru>
Wed, 4 Oct 2017 18:19:42 +0000 (21:19 +0300)
committerMaxim Dounin <mdounin@mdounin.ru>
Wed, 4 Oct 2017 18:19:42 +0000 (21:19 +0300)
commit80a2771762f0411300ff01ec24d6481e8518dfff
tree9697d3c852d680ac930f31a1b909d05ad0eb8223
parentf57527df8122eb8a99cc3bf419c28aa4d2a3cf9b
Fixed handling of unix sockets in $binary_remote_addr.

Previously, unix sockets were treated as AF_INET ones, and this may
result in buffer overread on Linux, where unbound unix sockets have
2-byte addresses.

Note that it is not correct to use just sun_path as a binary representation
for unix sockets.  This will result in an empty string for unbound unix
sockets, and thus behaviour of limit_req and limit_conn will change when
switching from $remote_addr to $binary_remote_addr.  As such, normal text
representation is used.

Reported by Stephan Dollberg.
src/http/ngx_http_variables.c
src/stream/ngx_stream_variables.c