diff options
| author | Stefan Fritsch <sf@cvs.openbsd.org> | 2018-07-22 09:09:19 +0000 |
|---|---|---|
| committer | Stefan Fritsch <sf@cvs.openbsd.org> | 2018-07-22 09:09:19 +0000 |
| commit | bf73814aa8b64c70896f54ac8e33046728b26274 (patch) | |
| tree | 6ea56e2f1cb1eabd9b8c309e33c0aca93c4eac1d /sys | |
| parent | 227a3e24eca40ff2ca9d91aa22e00d174d3f81db (diff) | |
Fix arguments of pf_purge_expired_{src_nodes,rules}()
Due to the missing "void", this
extern void pf_purge_expired_src_nodes();
is no prototype but a declaration. It is enough to suppress the
'implicit declaration' warning but it does not allow the compiler to
check the arguments passed to the calls of the function.
Fix the prototypes and don't pass the waslocked argument anymore. It has
been removed a year ago.
ok sashan henning
Diffstat (limited to 'sys')
| -rw-r--r-- | sys/net/pf.c | 4 | ||||
| -rw-r--r-- | sys/net/pf_ioctl.c | 8 | ||||
| -rw-r--r-- | sys/net/pfvar.h | 6 |
3 files changed, 9 insertions, 9 deletions
diff --git a/sys/net/pf.c b/sys/net/pf.c index 00187c9268c..5f39f2947bd 100644 --- a/sys/net/pf.c +++ b/sys/net/pf.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pf.c,v 1.1072 2018/07/12 14:28:07 sashan Exp $ */ +/* $OpenBSD: pf.c,v 1.1073 2018/07/22 09:09:18 sf Exp $ */ /* * Copyright (c) 2001 Daniel Hartmeier @@ -1257,7 +1257,7 @@ pf_purge(void *xnloops) /* purge other expired types every PFTM_INTERVAL seconds */ if (++(*nloops) >= pf_default_rule.timeout[PFTM_INTERVAL]) { - pf_purge_expired_src_nodes(0); + pf_purge_expired_src_nodes(); pf_purge_expired_rules(); } PF_UNLOCK(); diff --git a/sys/net/pf_ioctl.c b/sys/net/pf_ioctl.c index 5ce121c1022..a639bd60704 100644 --- a/sys/net/pf_ioctl.c +++ b/sys/net/pf_ioctl.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pf_ioctl.c,v 1.335 2018/07/10 09:28:27 henning Exp $ */ +/* $OpenBSD: pf_ioctl.c,v 1.336 2018/07/22 09:09:18 sf Exp $ */ /* * Copyright (c) 2001 Daniel Hartmeier @@ -794,7 +794,7 @@ pf_commit_rules(u_int32_t ticket, char *anchor) u_int32_t old_rcount; /* Make sure any expired rules get removed from active rules first. */ - pf_purge_expired_rules(1); + pf_purge_expired_rules(); rs = pf_find_ruleset(anchor); if (rs == NULL || !rs->rules.inactive.open || @@ -2580,7 +2580,7 @@ pfioctl(dev_t dev, u_long cmd, caddr_t addr, int flags, struct proc *p) pf_src_tree_remove_state(state); RB_FOREACH(n, pf_src_tree, &tree_src_tracking) n->expire = 1; - pf_purge_expired_src_nodes(1); + pf_purge_expired_src_nodes(); PF_UNLOCK(); break; } @@ -2613,7 +2613,7 @@ pfioctl(dev_t dev, u_long cmd, caddr_t addr, int flags, struct proc *p) } if (killed > 0) - pf_purge_expired_src_nodes(1); + pf_purge_expired_src_nodes(); psnk->psnk_killed = killed; PF_UNLOCK(); diff --git a/sys/net/pfvar.h b/sys/net/pfvar.h index fea4cca9bc8..faf29f0f487 100644 --- a/sys/net/pfvar.h +++ b/sys/net/pfvar.h @@ -1,4 +1,4 @@ -/* $OpenBSD: pfvar.h,v 1.481 2018/07/11 09:05:51 henning Exp $ */ +/* $OpenBSD: pfvar.h,v 1.482 2018/07/22 09:09:18 sf Exp $ */ /* * Copyright (c) 2001 Daniel Hartmeier @@ -1697,9 +1697,9 @@ extern int pf_tbladdr_setup(struct pf_ruleset *, extern void pf_tbladdr_remove(struct pf_addr_wrap *); extern void pf_tbladdr_copyout(struct pf_addr_wrap *); extern void pf_calc_skip_steps(struct pf_rulequeue *); -extern void pf_purge_expired_src_nodes(); +extern void pf_purge_expired_src_nodes(void); extern void pf_purge_expired_states(u_int32_t); -extern void pf_purge_expired_rules(); +extern void pf_purge_expired_rules(void); extern void pf_remove_state(struct pf_state *); extern void pf_remove_divert_state(struct pf_state_key *); extern void pf_free_state(struct pf_state *); |