From: Paul Mackerras <paulus@samba.org>
Date: Mon, 26 May 2008 09:18:08 +0000 (+0000)
Subject: Fix bug causing duplicate radius interim accounting messages (PR #1661)
X-Git-Tag: v2.4.5~38
X-Git-Url: https://git.ozlabs.org/?a=commitdiff_plain;h=638647916bb20f23221b0fd5fd6a8bb79a020093;p=ppp.git

Fix bug causing duplicate radius interim accounting messages (PR #1661)

This fixes a bug where we didn't cancel a timeout in the radius code
when the link goes down, leaving us with duplicate timeouts if it
comes back up again.  Bug report and patch from Richard Kojedzinszky.
---

diff --git a/pppd/plugins/radius/radius.c b/pppd/plugins/radius/radius.c
index 3c773cb..44d1911 100644
--- a/pppd/plugins/radius/radius.c
+++ b/pppd/plugins/radius/radius.c
@@ -24,7 +24,7 @@
 *
 ***********************************************************************/
 static char const RCSID[] =
-"$Id: radius.c,v 1.31 2006/05/22 00:01:40 paulus Exp $";
+"$Id: radius.c,v 1.32 2008/05/26 09:18:08 paulus Exp $";
 
 #include "pppd.h"
 #include "chap-new.h"
@@ -943,6 +943,9 @@ radius_acct_stop(void)
 	return;
     }
 
+    if (rstate.acct_interim_interval)
+	UNTIMEOUT(radius_acct_interim, NULL);
+
     rstate.accounting_started = 0;
     rc_avpair_add(&send, PW_ACCT_SESSION_ID, rstate.session_id,
 		   0, VENDOR_NONE);