d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove an extra "break" which could incorrectly terminate an

Browse Source 
STAILQ_FOREACH() loop when an AF_INET6 rule matched.

Spotted by:	cem
This commit is contained in:
Hiroki Sato 2016-12-13 20:55:46 +00:00
parent 1521ca71d3
commit 442f3ba311
1 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
usr.sbin/syslogd/syslogd.c
View File

@ -2465,7 +2465,6 @@ validate(struct sockaddr *sa, const char *hname)
struct allowedpeer *ap;
struct sockaddr_in *sin4, *a4p = NULL, *m4p = NULL;
#ifdef INET6
int reject;
struct sockaddr_in6 *sin6, *a6p = NULL, *m6p = NULL;
#endif
struct addrinfo hints, *res;
@ -2533,13 +2532,8 @@ validate(struct sockaddr *sa, const char *hname)
dprintf("rejected in rule %d due to scope mismatch.\n", i);
continue;
}
reject = 0;
if (IN6_ARE_MASKED_ADDR_EQUAL(&sin6->sin6_addr,
&a6p->sin6_addr, &m6p->sin6_addr) != 0) {
++reject;
break;
}
if (reject) {
dprintf("rejected in rule %d due to IP mismatch.\n", i);
continue;
}