The branch stable/13 has been updated by donner:
URL:
https://cgit.FreeBSD.org/src/commit/?id=791035c8da5e8a693b3b954db67ff50b4f8695cb
commit 791035c8da5e8a693b3b954db67ff50b4f8695cb
Author: Stefan Eßer <s...@freebsd.org>
AuthorDate: 2021-07-10 11:00:56 +0000
Commit: Lutz Donnerhacke <don...@freebsd.org>
CommitDate: 2021-07-14 11:49:21 +0000
libalias: fix divide by zero causing panic
The packet_limit can fall to 0, leading to a divide by zero abort in
the "packets % packet_limit".
An possible solution would be to apply a lower limit of 1 after the
calculation of packet_limit, but since any number modulo 1 gives 0,
the more efficient solution is to skip the modulo operation for
packet_limit <= 1.
Reported by: Karl Denninger <k...@denninger.net>
(cherry picked from commit 58080fbca09fda6d5f011d37059edbca8ceb4c58)
---
sys/netinet/libalias/alias_db.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sys/netinet/libalias/alias_db.c b/sys/netinet/libalias/alias_db.c
index 783a8ca3baa8..ed222c4133d4 100644
--- a/sys/netinet/libalias/alias_db.c
+++ b/sys/netinet/libalias/alias_db.c
@@ -1769,7 +1769,7 @@ HouseKeeping(struct libalias *la)
* Reduce the amount of house keeping work substantially by
* sampling over the packets.
*/
- if (packets % packet_limit == 0) {
+ if (packet_limit <= 1 || packets % packet_limit == 0) {
time_t now;
#ifdef _KERNEL
_______________________________________________
dev-commits-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/dev-commits-src-all
To unsubscribe, send any mail to "dev-commits-src-all-unsubscr...@freebsd.org"
