diff options
| author | Jim Pryor <profjim@jimpryor.net> | 2009-11-29 02:47:16 +0100 |
|---|---|---|
| committer | James Rayner <james@archlinux.org> | 2009-12-03 21:28:31 +0100 |
| commit | e9875e80160831253b58937f8fb0f4e39faa6ca9 (patch) | |
| tree | 3695e2bdff32d8c7035498ec0f2de4d231320a58 | |
| parent | 86675e091987dc7ffb049aa86f701a45c5191fd2 (diff) | |
| download | netctl-e9875e80160831253b58937f8fb0f4e39faa6ca9.tar.gz netctl-e9875e80160831253b58937f8fb0f4e39faa6ca9.tar.xz | |
connections/wireless: add some quotes
Sometimes just for coding style (inside [[ ]]), othertimes a necessary
precaution.
Signed-off-by: Jim Pryor <profjim@jimpryor.net>
| -rw-r--r-- | src/connections/wireless | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/connections/wireless b/src/connections/wireless index 6c51068..638b51b 100644 --- a/src/connections/wireless +++ b/src/connections/wireless @@ -27,7 +27,7 @@ iwconfig_up() { quirk prescan && iwlist "$INTERFACE" scan &> /dev/null # bcm43xx # 'none' uses iwconfig like wep. Use sane default if WEP_OPTS="" - if [[ -z $WEP_OPTS ]]; then + if [[ -z "$WEP_OPTS" ]]; then if [[ "$SECURITY" = "wep-old" ]]; then if [[ -n "$AP" ]]; then WEP_OPTS="ap \"$AP\" key $KEY" @@ -35,7 +35,7 @@ iwconfig_up() { WEP_OPTS="essid \"$ESSID\" key $KEY" fi elif [[ "$SECURITY" = "none-old" ]]; then - if [[ -n $AP ]]; then + if [[ -n "$AP" ]]; then WEP_OPTS="ap \"$AP\"" else WEP_OPTS="essid \"$ESSID\"" @@ -63,7 +63,7 @@ iwconfig_up() { wireless_up() { - PROFILE=$1 + PROFILE="$1" load_profile "$PROFILE" enable_rf || return 1 @@ -140,7 +140,7 @@ wireless_up() { fi fi - if ! $CONN_DIR/ethernet up "$PROFILE"; then + if ! "$CONN_DIR/ethernet" up "$PROFILE"; then wireless_down "$PROFILE" YES return 1 fi @@ -159,7 +159,7 @@ wireless_down() { else report_debug wireless_down stop_wpa "$INTERFACE" stop_wpa "$INTERFACE" - rm -rf ${TMPDIR:-/tmp}/wpa.$INTERFACE + rm -rf "${TMPDIR:-/tmp}/wpa.$INTERFACE" fi bring_interface down "$INTERFACE" |