diff options
| author |   yasuoka <yasuoka@openbsd.org> | 2020-07-29 02:32:13 +0000 |
|---|---|---|
| committer | yasuoka <yasuoka@openbsd.org> | 2020-07-29 02:32:13 +0000 |
| commit | 511c53b1292838559a33b3d8e51ffa1f0e25a599 (patch) | |
| tree | 5d2631bc6a18b50b88fcf7c8bcc2a67370091f5c | |
| parent | SC_DEBUG() and friends are for debugging the SCSI code, not for scattered random (diff) | |
| download | wireguard-openbsd-511c53b1292838559a33b3d8e51ffa1f0e25a599.tar.xz wireguard-openbsd-511c53b1292838559a33b3d8e51ffa1f0e25a599.zip | |
Fix previous commit which referred wrong address and returned wrong
value.
ok sashan
| -rw-r--r-- | sys/net/pf_lb.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/sys/net/pf_lb.c b/sys/net/pf_lb.c index 096f4ea596d..8793de95fea 100644 --- a/sys/net/pf_lb.c +++ b/sys/net/pf_lb.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pf_lb.c,v 1.66 2020/07/28 16:47:41 yasuoka Exp $ */ +/* $OpenBSD: pf_lb.c,v 1.67 2020/07/29 02:32:13 yasuoka Exp $ */ /* * Copyright (c) 2001 Daniel Hartmeier @@ -322,13 +322,13 @@ pf_map_addr_sticky(sa_family_t af, struct pf_rule *r, struct pf_addr *saddr, return (-1); } - if ((rpool->opts & PF_POOL_TYPEMASK) == PF_POOL_LEASTSTATES) { - if (pf_map_addr_states_increase(af, rpool, naddr) == -1) - return (-1); - } - if (!PF_AZERO(cached, af)) + if (!PF_AZERO(cached, af)) { pf_addrcpy(naddr, cached, af); + if ((rpool->opts & PF_POOL_TYPEMASK) == PF_POOL_LEASTSTATES && + pf_map_addr_states_increase(af, rpool, cached) == -1) + return (-1); + } if (pf_status.debug >= LOG_DEBUG) { log(LOG_DEBUG, "pf: pf_map_addr: " "src tracking (%u) maps ", type); @@ -651,7 +651,7 @@ pf_map_addr_states_increase(sa_family_t af, struct pf_pool *rpool, pf_print_host(naddr, 0, af); addlog(". Failed to increase count!\n"); } - return (1); + return (-1); } } else if (rpool->addr.type == PF_ADDR_DYNIFTL) { if (pfr_states_increase(rpool->addr.p.dyn->pfid_kt, @@ -663,7 +663,7 @@ pf_map_addr_states_increase(sa_family_t af, struct pf_pool *rpool, pf_print_host(naddr, 0, af); addlog(". Failed to increase count!\n"); } - return (1); + return (-1); } } return (0); |