[PATCH] Acct-Authentic should not be sent in Accounting-On and Accounting-Off

Nick Lowe nick.lowe at lugatech.com
Mon Jan 25 05:08:07 PST 2016


Version without broken whitespace attached.

Acct-Authentic should not be sent in Accounting-On and
 Accounting-Off

Signed-off-by: Nick Lowe <nick.lowe at lugatech.com>
---
 src/ap/accounting.c | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/ap/accounting.c b/src/ap/accounting.c
index babfe2d..7636cfc 100644
--- a/src/ap/accounting.c
+++ b/src/ap/accounting.c
@@ -76,17 +76,17 @@ static struct radius_msg * accounting_msg(struct
hostapd_data *hapd,
         goto fail;
     }

-    if (!hostapd_config_get_radius_attr(hapd->conf->radius_acct_req_attr,
-                        RADIUS_ATTR_ACCT_AUTHENTIC) &&
-        !radius_msg_add_attr_int32(msg, RADIUS_ATTR_ACCT_AUTHENTIC,
-                       hapd->conf->ieee802_1x ?
-                       RADIUS_ACCT_AUTHENTIC_RADIUS :
-                       RADIUS_ACCT_AUTHENTIC_LOCAL)) {
-        wpa_printf(MSG_INFO, "Could not add Acct-Authentic");
-        goto fail;
-    }
-
     if (sta) {
+        if (!hostapd_config_get_radius_attr(hapd->conf->radius_acct_req_attr,
+                            RADIUS_ATTR_ACCT_AUTHENTIC) &&
+            !radius_msg_add_attr_int32(msg, RADIUS_ATTR_ACCT_AUTHENTIC,
+                           hapd->conf->ieee802_1x ?
+                           RADIUS_ACCT_AUTHENTIC_RADIUS :
+                           RADIUS_ACCT_AUTHENTIC_LOCAL)) {
+            wpa_printf(MSG_INFO, "Could not add Acct-Authentic");
+            goto fail;
+        }
+
         /* Use 802.1X identity if available */
         val = ieee802_1x_get_identity(sta->eapol_sm, &len);

-- 
2.5.0
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0006-Acct-Authentic-should-not-be-sent-in-Accounting-On-a.patch
Type: text/x-patch
Size: 1519 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/hostap/attachments/20160125/75d4415c/attachment.bin>


More information about the Hostap mailing list