diff options
Diffstat (limited to 'network_io')
-rw-r--r-- | network_io/unix/sockopt.c | 21 |
1 files changed, 19 insertions, 2 deletions
diff --git a/network_io/unix/sockopt.c b/network_io/unix/sockopt.c index 487d30957..8ba553230 100644 --- a/network_io/unix/sockopt.c +++ b/network_io/unix/sockopt.c @@ -397,8 +397,25 @@ apr_status_t apr_socket_accept_filter(apr_socket_t *sock, const char *name, const char *args) { struct accept_filter_arg af; - strncpy(af.af_name, name, 16); - strncpy(af.af_arg, args, 256 - 16); + socklen_t optlen = sizeof(af); + + /* FreeBSD returns an error if the filter is already set; ignore + * this call if we previously set it to the same value. + */ + if ((getsockopt(sock->socketdes, SOL_SOCKET, SO_ACCEPTFILTER, + &af, &optlen)) == 0) { + if (!strcmp(name, af.af_name) && !strcmp(args, af.af_arg)) { + return APR_SUCCESS; + } + } + + /* Uhh, at least in FreeBSD 9 the fields are declared as arrays of + * these lengths; did sizeof not work in some ancient release? + * + * FreeBSD kernel sets the last byte to a '\0'. + */ + apr_cpystrn(af.af_name, name, 16); + apr_cpystrn(af.af_arg, args, 256 - 16); if ((setsockopt(sock->socketdes, SOL_SOCKET, SO_ACCEPTFILTER, &af, sizeof(af))) < 0) { |