[arch-projects] [netcfg] [PATCH] Exit from stop_wpa() if there's nothing to terminate.

[Dave Reisner]

On Tue, Oct 02, 2012 at 10:27:24PM +0400, Ivan Shapovalov wrote:
> Check "$WPA_CONF_DIR/$INTERFACE" for existence; exit if it's absent
> (that is, wpa_supplicant isn't running).
> Just avoids an error message from wpa_cli.
> 
> Signed-off-by: Ivan Shapovalov <intelfx100 at gmail.com>
> ---
>  src/8021x | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/src/8021x b/src/8021x
> index 608d683..bce4e8c 100644
> --- a/src/8021x
> +++ b/src/8021x
> @@ -73,6 +73,9 @@ stop_wpa()
>      # we need this as long as wpa_cli has a different default than netcfg
>      [[ -z "$WPA_CTRL_DIR" && -z "$WPA_CONF" ]] && WPA_CTRL_DIR="/run/wpa_supplicant"
>  
> +    # exit if wpa_supplicant is already terminated for this interface
> +    [ -e "$WPA_CTRL_DIR/$INTERFACE" ] || return

It baffles me why people blatently disregard syntax. [ is not identical
to [[ in bash.

> +
>      wpa_call "$INTERFACE" terminate > /dev/null
>  
>      # wait up to one second for the pid file to be removed
> -- 
> 1.7.12.2
> 
>