diff options
| author |   mikeb <mikeb@openbsd.org> | 2016-06-15 11:36:06 +0000 |
|---|---|---|
| committer | mikeb <mikeb@openbsd.org> | 2016-06-15 11:36:06 +0000 |
| commit | 367b0410d3271136951f6c323ac43106e15f266c (patch) | |
| tree | e6a7f72cc9185ae7ac339b6d92c0fca0c4dfcee8 /sys/net/pf_lb.c | |
| parent | Copy mode needs to keep the original grid intact so it can copy from it (diff) | |
| download | wireguard-openbsd-367b0410d3271136951f6c323ac43106e15f266c.tar.xz wireguard-openbsd-367b0410d3271136951f6c323ac43106e15f266c.zip | |
There's no need to convert values returned by arc4random to the network
byte order. Spotted by Gleb Smirnoff (glebius@FreeBSD.org), thanks!
ok tedu
Diffstat (limited to 'sys/net/pf_lb.c')
| -rw-r--r-- | sys/net/pf_lb.c | 16 |
1 files changed, 6 insertions, 10 deletions
diff --git a/sys/net/pf_lb.c b/sys/net/pf_lb.c index 9a449005157..6ac74760a7c 100644 --- a/sys/net/pf_lb.c +++ b/sys/net/pf_lb.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pf_lb.c,v 1.52 2015/11/24 13:37:16 mpi Exp $ */ +/* $OpenBSD: pf_lb.c,v 1.53 2016/06/15 11:36:06 mikeb Exp $ */ /* * Copyright (c) 2001 Daniel Hartmeier @@ -415,28 +415,24 @@ pf_map_addr(sa_family_t af, struct pf_rule *r, struct pf_addr *saddr, } else if (init_addr != NULL && PF_AZERO(init_addr, af)) { switch (af) { case AF_INET: - rpool->counter.addr32[0] = htonl(arc4random()); + rpool->counter.addr32[0] = arc4random(); break; #ifdef INET6 case AF_INET6: if (rmask->addr32[3] != 0xffffffff) - rpool->counter.addr32[3] = - htonl(arc4random()); + rpool->counter.addr32[3] = arc4random(); else break; if (rmask->addr32[2] != 0xffffffff) - rpool->counter.addr32[2] = - htonl(arc4random()); + rpool->counter.addr32[2] = arc4random(); else break; if (rmask->addr32[1] != 0xffffffff) - rpool->counter.addr32[1] = - htonl(arc4random()); + rpool->counter.addr32[1] = arc4random(); else break; if (rmask->addr32[0] != 0xffffffff) - rpool->counter.addr32[0] = - htonl(arc4random()); + rpool->counter.addr32[0] = arc4random(); break; #endif /* INET6 */ default: |