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 | d84f598f5c080c730b3b5f4bbd284acf06b23002 (patch) | |
| tree | 73113fe9689d44f93ed527df8c2b4b99a7b90ee0 /src/selftest/allowedips.h | |
| parent | cc2113b5b27c7ee45711df05c49a7d05bd0a934e (diff) | |
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>
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
Diffstat (limited to 'src/selftest/allowedips.h')
| -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); |