diff options
author | Mark Michelson <mmichelson@digium.com> | 2014-11-20 16:36:54 +0000 |
---|---|---|
committer | Mark Michelson <mmichelson@digium.com> | 2014-11-20 16:36:54 +0000 |
commit | 8e64f6a0af9c62b41c29b8e94b961f3e49b154a6 (patch) | |
tree | bbcc9d4efd627be6859f6b211e824d7f49be5479 | |
parent | e2181139c985ae35174f956020e69f1ee07fa2f8 (diff) |
Fix error with mixed address family ACLs.
Prior to this commit, the address family of the first item in an ACL
was used to compare all incoming traffic. This could lead to traffic
of other IP address families bypassing ACLs.
ASTERISK-24469 #close
Reported by Matt Jordan
Patches:
ASTERISK-24469-11.diff uploaded by Matt Jordan (License #6283)
AST-2014-012
........
Merged revisions 428402 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 428417 from http://svn.asterisk.org/svn/asterisk/branches/11
........
Merged revisions 428422 from http://svn.asterisk.org/svn/asterisk/branches/12
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/13@428425 65c4cc65-6c06-0410-ace0-fbb531ad65f3
-rw-r--r-- | main/acl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/main/acl.c b/main/acl.c index 5c3b633d7..87776b33b 100644 --- a/main/acl.c +++ b/main/acl.c @@ -743,7 +743,7 @@ enum ast_acl_sense ast_apply_ha(const struct ast_ha *ha, const struct ast_sockad ast_copy_string(iabuf2, ast_inet_ntoa(ha->netaddr), sizeof(iabuf2)); ast_debug(1, "##### Testing %s with %s\n", iabuf, iabuf2); #endif - if (ast_sockaddr_is_ipv4(&ha->addr)) { + if (ast_sockaddr_is_ipv4(¤t_ha->addr)) { if (ast_sockaddr_is_ipv6(addr)) { if (ast_sockaddr_is_ipv4_mapped(addr)) { /* IPv4 ACLs apply to IPv4-mapped addresses */ |