diff options
| author |   Jason A. Donenfeld <Jason@zx2c4.com> | 2018-09-19 04:06:27 +0200 |
|---|---|---|
| committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2018-09-20 02:07:56 +0200 |
| commit | 470a0c58d85d451cadae663c3a1d8c8c2538ef31 (patch) | |
| tree | 43996382f2f2370906bfab6ede75aade0a8caca1 /src/selftest/allowedips.h | |
| parent | chacha20: prefer crypto_xor_cpy to avoid memmove (diff) | |
| download | wireguard-monolithic-historical-470a0c58d85d451cadae663c3a1d8c8c2538ef31.tar.xz wireguard-monolithic-historical-470a0c58d85d451cadae663c3a1d8c8c2538ef31.zip | |
allowedips: change from BUG_ON to WARN_ON
This is never going to hit anyway, and if it does, it's a development
problem that will be caught with the selftests anyway. So don't make
Andrew Lunn upset, and just change it to a WARN_ON.
Suggested-by: Andrew Lunn <andrew@lunn.ch>
Diffstat (limited to '')
| -rw-r--r-- | src/selftest/allowedips.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/selftest/allowedips.h b/src/selftest/allowedips.h index 83cfb34..dc04543 100644 --- a/src/selftest/allowedips.h +++ b/src/selftest/allowedips.h @@ -600,7 +600,7 @@ bool __init allowedips_selftest(void) allowedips_remove_by_peer(&t, a, &mutex); test_negative(4, a, 192, 168, 0, 1); - /* These will hit the BUG_ON(len >= 128) in free_node if something goes wrong. */ + /* These will hit the WARN_ON(len >= 128) in free_node if something goes wrong. */ for (i = 0; i < 128; ++i) { part = cpu_to_be64(~(1LLU << (i % 64))); memset(&ip, 0xff, 16); |