aboutsummaryrefslogtreecommitdiffstats
path: root/src/ap/accounting.c
diff options
context:
space:
mode:
authorJouni Malinen <j@w1.fi>2014-10-18 07:35:33 (GMT)
committerJouni Malinen <j@w1.fi>2014-10-18 07:35:33 (GMT)
commitaff039fada2f10b0956312a53e1484b73314daa2 (patch)
treeb1420214f7969d6075d9f708721e20b0df2ef621 /src/ap/accounting.c
parent414236b5db78d0495893d9ee5a335a07a4b5a307 (diff)
downloadhostap-aff039fada2f10b0956312a53e1484b73314daa2.zip
hostap-aff039fada2f10b0956312a53e1484b73314daa2.tar.gz
hostap-aff039fada2f10b0956312a53e1484b73314daa2.tar.bz2
Remove duplicated Acct-Session-Id from Accounting-Request
Commit 8b2486115479582b2ab164a4508f22ed23a9a4cb ('Add Acct-Session-Id into Access-Request messages') added Acct-Session-Id building into the helper function shared between authentication and accounting messages. However, it forgot to remove the same code from the generation of accounting messages and as such, ended up with Accounting-Request messages containing two copies of this attribute. Fix this by removing the addition of this attribute from the accounting specific function. Signed-off-by: Jouni Malinen <j@w1.fi>
Diffstat (limited to 'src/ap/accounting.c')
-rw-r--r--src/ap/accounting.c8
1 files changed, 0 insertions, 8 deletions
diff --git a/src/ap/accounting.c b/src/ap/accounting.c
index 6290d3f..1fb1e92 100644
--- a/src/ap/accounting.c
+++ b/src/ap/accounting.c
@@ -49,14 +49,6 @@ static struct radius_msg * accounting_msg(struct hostapd_data *hapd,
if (sta) {
radius_msg_make_authenticator(msg, (u8 *) sta, sizeof(*sta));
-
- os_snprintf(buf, sizeof(buf), "%08X-%08X",
- sta->acct_session_id_hi, sta->acct_session_id_lo);
- if (!radius_msg_add_attr(msg, RADIUS_ATTR_ACCT_SESSION_ID,
- (u8 *) buf, os_strlen(buf))) {
- wpa_printf(MSG_INFO, "Could not add Acct-Session-Id");
- goto fail;
- }
} else {
radius_msg_make_authenticator(msg, (u8 *) hapd, sizeof(*hapd));
}