RADIUS: Do not include Acct-Terminate-Cause in Accounting-On/Off
authorNick Lowe <nick.lowe@lugatech.com>
Sun, 24 Jan 2016 10:20:33 +0000 (10:20 +0000)
committerJouni Malinen <j@w1.fi>
Fri, 5 Feb 2016 15:59:07 +0000 (17:59 +0200)
Per RFC 2866, 5.10, it is invalid to send Acct-Terminate-Cause in
Accounting-On and Accounting-Off (this is included only when
Acct-Status-Type is set to Stop).

Signed-off-by: Nick Lowe <nick.lowe@lugatech.com>
src/ap/accounting.c

index c60b3a6..9b06221 100644 (file)
@@ -446,14 +446,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);
 }