diff mbox

It is invalid to send Acct-Terminate-Cause in Accounting-On and Accounting-Off

Message ID CAGnO3dp6MrtqdZQ0+d=BS7rPjdxwRrXtNAmODJPAqjKU=vXvbw@mail.gmail.com
State Accepted
Headers show

Commit Message

Nick Lowe Jan. 24, 2016, 11:38 a.m. UTC 
Attached without the whitespace broken...

Comments

Jouni Malinen Feb. 5, 2016, 4:31 p.m. UTC | #1 
On Sun, Jan 24, 2016 at 11:38:21AM +0000, Nick Lowe wrote:
> Attached without the whitespace broken...

Thanks, applied.
diff mbox

Patch

From b169a2269b8f3422a73ba9100085de8100eab89f Mon Sep 17 00:00:00 2001
From: Nick Lowe <nick.lowe@lugatech.com>
Date: Sun, 24 Jan 2016 10:20:33 +0000
Subject: [PATCH 2/3] It is invalid to send Acct-Terminate-Cause in
 Accounting-On and Accounting-Off.

Signed-off-by: Nick Lowe <nick.lowe@lugatech.com>
---
 src/ap/accounting.c | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/src/ap/accounting.c b/src/ap/accounting.c
index 95dc366..e585b78 100644
--- a/src/ap/accounting.c
+++ b/src/ap/accounting.c
@@ -448,14 +448,6 @@  static void accounting_report_state(struct hostapd_data *hapd, int on)
 	if (!msg)
 		return;
 
-	if (!radius_msg_add_attr_int32(msg, RADIUS_ATTR_ACCT_TERMINATE_CAUSE,
-				       RADIUS_ACCT_TERMINATE_CAUSE_NAS_REBOOT))
-	{
-		wpa_printf(MSG_INFO, "Could not add Acct-Terminate-Cause");
-		radius_msg_free(msg);
-		return;
-	}
-
 	if (radius_client_send(hapd->radius, msg, RADIUS_ACCT, NULL) < 0)
 		radius_msg_free(msg);
 }
-- 
2.5.0