[PATCH] hostapd: Report error on unknown ACCEPT_ACL/DENY_ACL commands
Jouni Malinen
j at w1.fi
Wed Feb 1 08:56:20 PST 2023
On Wed, Jan 04, 2023 at 04:53:50PM +0100, Raphaël Mélotte wrote:
> Currently when using ACCEPT_ACL or DENY_ACL, no error is reported if
> the rest of the command is unknown (e.g. 'ACCEPT_ACL FOOBAR' reports
> 'OK').
>
> On the other hand, hostapd_cli makes it possible to use 'accept_acl'
> and 'deny_acl' in lowercase, but the rest of the command (i.e. 'SHOW',
> 'ADD_MAC', 'CLEAR', etc) must be in uppercase.
>
> As a result, the command 'accept_acl clear' could seem valid when
> using hostapd_cli (as it reports 'OK'), while it actually does not do
> anything (because 'clear' must be in uppercase).
>
> To let users know whether the command really succeeded or not, report
> an error when the command was not understood.
>
> Note that this is also consistent with the way it is currently
> implemented in wpa_supplicant.
Thanks, applied.
--
Jouni Malinen PGP id EFC895FA
More information about the Hostap
mailing list