mirror of
https://github.com/Telecominfraproject/wlan-ap.git
synced 2025-11-03 11:57:58 +00:00
uspot: radius: regroup redundant checks
Clarifies intent. Signed-off-by: Thibaut VARÈNE <hacks@slashdirt.org>
This commit is contained in:
committed by
John Crispin
parent
3219fceb48
commit
1cded2e983
@@ -308,17 +308,13 @@ radius(void)
|
|||||||
if (rc_avpair_add(rh, &send, PW_PROXY_STATE, blobmsg_get_string(tb[RADIUS_PROXY_STATE_ACCT]), -1, 0) == NULL)
|
if (rc_avpair_add(rh, &send, PW_PROXY_STATE, blobmsg_get_string(tb[RADIUS_PROXY_STATE_ACCT]), -1, 0) == NULL)
|
||||||
goto fail;
|
goto fail;
|
||||||
}
|
}
|
||||||
|
if (rc_acct(rh, 0, send) == OK_RC)
|
||||||
|
return result(rh, 1, NULL);
|
||||||
} else {
|
} else {
|
||||||
if (tb[RADIUS_PROXY_STATE_AUTH]) {
|
if (tb[RADIUS_PROXY_STATE_AUTH]) {
|
||||||
if (rc_avpair_add(rh, &send, PW_PROXY_STATE, blobmsg_get_string(tb[RADIUS_PROXY_STATE_AUTH]), -1, 0) == NULL)
|
if (rc_avpair_add(rh, &send, PW_PROXY_STATE, blobmsg_get_string(tb[RADIUS_PROXY_STATE_AUTH]), -1, 0) == NULL)
|
||||||
goto fail;
|
goto fail;
|
||||||
}
|
}
|
||||||
}
|
|
||||||
|
|
||||||
if (tb[RADIUS_ACCT] && blobmsg_get_bool(tb[RADIUS_ACCT])) {
|
|
||||||
if (rc_acct(rh, 0, send) == OK_RC)
|
|
||||||
return result(rh, 1, NULL);
|
|
||||||
} else {
|
|
||||||
if (rc_auth(rh, 0, send, &received, NULL) == OK_RC)
|
if (rc_auth(rh, 0, send, &received, NULL) == OK_RC)
|
||||||
return result(rh, 1, received);
|
return result(rh, 1, received);
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user