Re: [PATCH] dbus: Add SsidTempDisabled signal

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Feb 25, 2021 at 05:38:46PM +0100, Peter Malo wrote:
> diff --git a/doc/dbus.doxygen b/doc/dbus.doxygen
> +      <li>
> +	<h3>SsidTempDisabled ( s : ssid, u : auth_failures, s : reason )</h3>
> +	<p>Information about SSID being temporarily disabled.</p>
> +	<h4>Arguments</h4>
> +	<dl>
> +	  <dt>s : ssid</dt>
> +	  <dd>SSID of the network that has been temp disabled.</dd>
> +	  <dt>u : auth_failures</dt>
> +	  <dd>Number of authentication failures</dd>
> +	  <dt>s : reason</dt>
> +	  <dd>Reason for the disabling of the network e.g., "WRONG_KEY"</dd>
> +	</dl>
> +      </li>

This looks a bit strange.. Why would a D-Bus signal include the SSID?
Shouldn't this instead be identifying the network object that is being
temporarily disabled?

If there is a signal notifying that a network got temporarily disabled,
shouldn't there be a signal to notify that such temporarily disablement
was removed?

-- 
Jouni Malinen                                            PGP id EFC895FA

_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap



[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux