]> 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)
commit41d8ea8c8d268555d3cbd8dd2ab32dcc06658209
tree04abf1948e0f8a3a52267f84a2f33bc97c337dd7
parentcba23f88ec6740b7f8d6a1383254708a91d37625
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