From 40a8f669c2dbdc70a53009b97597b1982a5f5d65 Mon Sep 17 00:00:00 2001 From: Renato Botelho Date: Fri, 18 Oct 2013 17:32:13 -0300 Subject: [PATCH] Prevent a possible division by zero. it fixes #3212 --- etc/inc/captiveportal.inc | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/etc/inc/captiveportal.inc b/etc/inc/captiveportal.inc index 0a7cd622f7..7d683b0064 100644 --- a/etc/inc/captiveportal.inc +++ b/etc/inc/captiveportal.inc @@ -726,8 +726,9 @@ function captiveportal_prune_old() { else $interval = 0; $past_interval_min = ($session_time > $interval); - $within_interval = ($session_time % $interval >= 0 && $session_time % $interval <= 59); - if (($interval > 0 && $past_interval_min && $within_interval) || $interval === 0) { + if ($interval != 0) + $within_interval = ($session_time % $interval >= 0 && $session_time % $interval <= 59); + if ($interval === 0 || ($interval > 0 && $past_interval_min && $within_interval)) { RADIUS_ACCOUNTING_STOP($cpentry[1], // ruleno $cpentry[4], // username $cpentry[5], // sessionid