From 6a019f77f23fc9d214d1a1679f90606a31396184 Mon Sep 17 00:00:00 2001
From: James Rayner <james@archlinux.org>
Date: Mon, 10 Aug 2009 18:02:31 +1000
Subject: Added basic rfkill support; prototype connection query support

---
 src/connections/ethernet         |  7 +++++++
 src/connections/ethernet-iproute |  7 +++++++
 src/connections/wireless         | 45 +++++++++++++++++++++++++++++++++++++++-
 3 files changed, 58 insertions(+), 1 deletion(-)

(limited to 'src/connections')

diff --git a/src/connections/ethernet b/src/connections/ethernet
index e24e9f5..38b341c 100644
--- a/src/connections/ethernet
+++ b/src/connections/ethernet
@@ -124,6 +124,13 @@ ethernet_down() {
     esac
 }
 
+# Returns status of profile - is it still functional?
+ethernet_status() {
+    if ! ip link show dev ra0|grep -q "state UP"; then
+        return 1
+    fi
+}
+
 ethernet_$1 $2
 exit $?
 # vim: set ts=4 et sw=4:
diff --git a/src/connections/ethernet-iproute b/src/connections/ethernet-iproute
index 1a4a6e4..a6c4804 100644
--- a/src/connections/ethernet-iproute
+++ b/src/connections/ethernet-iproute
@@ -112,6 +112,13 @@ ethernet_down() {
 
 }
 
+# Returns status of profile - is it still functional?
+ethernet_status() {
+    if ! ip link show dev ra0|grep -q "state UP"; then
+        return 1
+    fi
+}
+
 ethernet_$1 $2
 exit $?
 # vim: set ts=4 et sw=4:
diff --git a/src/connections/wireless b/src/connections/wireless
index 8b43ed1..7394a20 100644
--- a/src/connections/wireless
+++ b/src/connections/wireless
@@ -1,12 +1,34 @@
 #! /bin/bash
 . /usr/lib/network/network
 
+
+rfkill_from_name() {
+    local name=$1
+    for rfkill in /sys/class/rfkill/*; do
+        if [[ "$(cat $rfkill/name)" == $name ]]; then
+            echo $rfkill
+            return 0
+        fi
+    done
+    echo "none"
+    return 1
+}
+
 wireless_up() {
 
     load_profile $1
     . ${SUBR_DIR}/8021x
     . ${SUBR_DIR}/wireless
 
+    # If rfkill is specified, enable device.
+    if [[ -n "$RFKILL_NAME" ]]; then
+        path=$(rfkill_from_name $RFKILL_NAME)
+        if [[ $? -ne 0 ]]; then
+            err_append "no rfkill switch with the name $RFKILL_NAME";
+        fi 
+        echo 1 > ${path}/state
+    fi
+
     # Check if interface exists
     if [[ ! -e /sys/class/net/"$INTERFACE" ]]; then
         if ! echo "$INTERFACE"|grep ":"; then
@@ -17,7 +39,7 @@ wireless_up() {
 
     # Kill any lingering wpa_supplicants.
     stop_wpa $INTERFACE 
-
+            
     # Most drivers (mac80211) need mode set before device is brought up
     # Drivers generally default to managed, but set this to be sure.
     if [[ "$(iwgetid -sm $INTERFACE)" -ne "Managed" ]]; then
@@ -117,6 +139,27 @@ wireless_down() {
 	[[ "$SECURITY" == "wpa" ]] && rm -f "/tmp/wpa.${1// /}" # remove wpa config
     iwconfig $INTERFACE essid off key off &> /dev/null 
     ifconfig $INTERFACE down
+    
+    # If rfkill is specified, disable device.
+    if [[ -n "$RFKILL_NAME" ]]; then
+        path=$(rfkill_from_name $RFKILL_NAME)
+        if [[ $? -ne 0 ]]; then
+            err_append "no rfkill switch with the name $RFKILL_NAME";
+        fi 
+        echo 0 > ${path}/state
+    fi
+    
+}
+
+# Returns status of profile - is it still functional?
+wireless_status() {
+    load_profile $1
+    if [[ "$(iwgetid -r)" -ne $ESSID ]]; then
+        return 1
+    elif ! ip link show dev ra0|grep -q "state UP"; then
+        return 1
+    fi
+
 }
 
 wireless_$1 $2
-- 
cgit v1.2.3-24-g4f1b


From 65c7253e9f27421e61c0764dcf8843797611db00 Mon Sep 17 00:00:00 2001
From: James Rayner <james@archlinux.org>
Date: Mon, 10 Aug 2009 18:10:19 +1000
Subject: Add wireless+iproute support, extend wait for some slower cards

---
 src/connections/ethernet         |  2 +-
 src/connections/ethernet-iproute |  2 +-
 src/connections/wireless         | 12 ++++++++----
 3 files changed, 10 insertions(+), 6 deletions(-)

(limited to 'src/connections')

diff --git a/src/connections/ethernet b/src/connections/ethernet
index 38b341c..006dd42 100644
--- a/src/connections/ethernet
+++ b/src/connections/ethernet
@@ -12,7 +12,7 @@ ethernet_up() {
     fi  
 
     ip link set $INTERFACE up
-    sleep 1
+    sleep 2
 
     if ip link show $INTERFACE|grep -q "NO-CARRIER"; then
        err_append "No connection"
diff --git a/src/connections/ethernet-iproute b/src/connections/ethernet-iproute
index a6c4804..3d5f5d3 100644
--- a/src/connections/ethernet-iproute
+++ b/src/connections/ethernet-iproute
@@ -19,7 +19,7 @@ ethernet_up() {
     fi  
 
     ip link set $INTERFACE up
-    sleep 1
+    sleep 2
 
     if ip link show $INTERFACE|grep -q "NO-CARRIER"; then
        err_append "No connection"
diff --git a/src/connections/wireless b/src/connections/wireless
index 7394a20..0fcf56e 100644
--- a/src/connections/wireless
+++ b/src/connections/wireless
@@ -122,7 +122,9 @@ wireless_up() {
         ;;        
     esac
     
-    if ! ${CONN_DIR}/ethernet up $1; then
+    conn=ethernet
+    checkyesno ${IPROUTE:-no} && conn=ethernet-iproute
+    if ! ${CONN_DIR}/$conn up $1; then
         wireless_down $1 YES
         return 1
     fi
@@ -131,9 +133,11 @@ wireless_up() {
 wireless_down() {
     load_profile $1
     . ${SUBR_DIR}/8021x
-    PROFILE=$1 NOETHERNETDOWN=$2
-    if ! checkyesno $2; then
-        ${CONN_DIR}/ethernet down $1    
+    PROFILE=$1 NOETHERNETDOWN=$2 
+    if ! checkyesno $NOETHERNETDOWN; then
+        conn=ethernet
+        checkyesno ${IPROUTE:-no} && conn=ethernet-iproute
+        $CONN_DIR/$conn down $1  
     fi
     stop_wpa $INTERFACE 
 	[[ "$SECURITY" == "wpa" ]] && rm -f "/tmp/wpa.${1// /}" # remove wpa config
-- 
cgit v1.2.3-24-g4f1b


From 74fe64cec00d359ce98938754e9cb1b0c96e49a8 Mon Sep 17 00:00:00 2001
From: James Rayner <james@archlinux.org>
Date: Mon, 10 Aug 2009 20:09:14 +1000
Subject: merge logging code from Jim Pryor

---
 src/connections/ethernet         | 44 ++++++++++++++++++++++---------
 src/connections/ethernet-iproute | 42 ++++++++++++++++++-----------
 src/connections/wireless         | 57 +++++++++++++++++++++++++++++-----------
 3 files changed, 100 insertions(+), 43 deletions(-)

(limited to 'src/connections')

diff --git a/src/connections/ethernet b/src/connections/ethernet
index 006dd42..2105ec2 100644
--- a/src/connections/ethernet
+++ b/src/connections/ethernet
@@ -6,16 +6,17 @@ ethernet_up() {
 
     if [[ ! -e /sys/class/net/"$INTERFACE" ]]; then
         if ! echo "$INTERFACE"|grep ":"; then
-            err_append "interface $INTERFACE does not exist"
+            report_fail "interface $INTERFACE does not exist"
             return 1
         fi
     fi  
 
+    report_debug ethernet_up ifup
     ip link set $INTERFACE up
     sleep 2
 
     if ip link show $INTERFACE|grep -q "NO-CARRIER"; then
-       err_append "No connection"
+       report_fail "No connection"
        return 1
     fi
 
@@ -23,7 +24,11 @@ ethernet_up() {
         . "${SUBR_DIR}"/8021x
         [[ -z "$WPA_CONF" ]] && WPA_CONF="/etc/wpa_supplicant.conf"
         [[ -z "$WPA_OPTS" ]] && WPA_OPTS="-Dwired"
-        start_wpa "$INTERFACE" "$WPA_CONF" "$WPA_OPTS"
+        report_debug ethernet_up start_wpa "$INTERFACE" "$WPA_CONF" "$WPA_OPTS"
+        if ! start_wpa $INTERFACE $WPA_CONF $WPA_OPTS; then
+            report_fail "wpa_supplicant did not start, possible configuration error"
+            return 1
+        fi
         if ! wpa_check "$INTERFACE"; then
             ifconfig "$INTERFACE" down
             return 1
@@ -34,8 +39,9 @@ ethernet_up() {
     dhcp)
         if checkyesno "${DHCLIENT:-no}"; then
             rm -r /var/run/dhclient-${INTERFACE}.pid >/dev/null 2>&1
+            report_debug ethernet_up dhclient -q -e TIMEOUT="${DHCP_TIMEOUT:-10}" -pf "/var/run/dhclient-$INTERFACE.pid" "$INTERFACE"
             if ! dhclient -q -e TIMEOUT="${DHCP_TIMEOUT:-10}" -pf /var/run/dhclient-${INTERFACE}.pid $INTERFACE; then
-                err_append "DHCP IP lease attempt failed."
+                report_fail "DHCP IP lease attempt failed."
                 return 1
             fi
         else
@@ -44,37 +50,44 @@ ethernet_up() {
             # If using own dns, tell dhcpcd to NOT replace resolv.conf
             [[ -n "$DNS1" ]] && DHCP_OPTIONS="-C resolv.conf $DHCP_OPTIONS"
             # Start dhcpcd
+            report_debug ethernet_up dhcpcd -qL -t "${DHCP_TIMEOUT:-10}" $DHCP_OPTIONS "$INTERFACE"
             if ! dhcpcd -qL -t "${DHCP_TIMEOUT:-10}" $DHCP_OPTIONS "$INTERFACE"; then
-                err_append "DHCP IP lease attempt failed."
+                report_fail "DHCP IP lease attempt failed."
                 return 1
             fi 
         fi
-        [[ -n "$IFOPTS" ]] && ifconfig "$INTERFACE" $IFOPTS
+        if [[ -n "$IFOPTS" ]]; then
+            report_debug ethernet_up ifup $IFOPTS    
+            ifconfig "$INTERFACE" $IFOPTS
+        fi
         ;;
     static)
+        report_debug ethernet_up ifup $IFOPTS
         if ! ifconfig "$INTERFACE" $IFOPTS up; then
-            err_append "Bringing interface up failed."
+            report_fail "Bringing interface up failed."
             return 1
         fi
 
         # bring up the default route (gateway)
         if [[ -n "$GATEWAY" ]]; then
+            report_debug ethernet_up route add default gw "$GATEWAY"
             if ! route add default gw $GATEWAY; then
-                err_append "Adding gateway failed."
+                report_fail "Adding gateway failed."
                 return 1
             fi 
         fi
         ;;
       *)
-        err_append "IP=\"\" must be either 'dhcp' or 'static'."
+        report_fail "IP=\"\" must be either 'dhcp' or 'static'."
         return 1
         ;;
     esac
 
     # set the hostname
     if [[ -n "$HOSTNAME" ]]; then
+        report_debug ethernet_up hostname "$HOSTNAME"
         if ! hostname "$HOSTNAME"; then
-            err_append "Setting hostname failed."
+            report_fail "Setting hostname failed."
             return 1
         fi
     fi
@@ -103,18 +116,25 @@ ethernet_down() {
     dhcp)
         if checkyesno "${DHCLIENT:-no}"; then
             if [[ -f /var/run/dhclient-${INTERFACE}.pid ]]; then
-                kill `cat /var/run/dhclient-${INTERFACE}.pid`
+                report_debug ethernet_down kill dhclient
+                kill $(cat /var/run/dhclient-${INTERFACE}.pid)
             fi
         else
             if [[ -f /var/run/dhcpcd-${INTERFACE}.pid ]]; then
+                report_debug ethernet_down dhcpcd -qx "$INTERFACE"
                 dhcpcd -qx "$INTERFACE"
             fi
         fi
         ;;
     static)
-        [[ -n "$GATEWAY" ]] && route del default gw $GATEWAY
+        if [[ -n "$GATEWAY" ]]; then
+            report_debug ethernet_down route del default gw "$GATEWAY"
+            route del default gw $GATEWAY
+        fi
         ;;
     esac
+    
+    report_debug ethernet_down ifdown
     ifconfig $INTERFACE 0.0.0.0
     
     case "$CONNECTION" in # Keep interface up for wireless
diff --git a/src/connections/ethernet-iproute b/src/connections/ethernet-iproute
index 3d5f5d3..75c99f3 100644
--- a/src/connections/ethernet-iproute
+++ b/src/connections/ethernet-iproute
@@ -1,9 +1,9 @@
 #! /bin/bash
 . /usr/lib/network/network
 
-error()
+report_iproute()
 {
-    err_append "$*"
+    report_fail "$*"
     ip addr flush $INTERFACE &>/dev/null
     quirk "nodown" || ip link set $INTERFACE down &>/dev/null 
     exit 1
@@ -14,15 +14,16 @@ ethernet_up() {
 
     if [[ ! -e /sys/class/net/$INTERFACE ]]; then
         if ! echo "$INTERFACE"|grep ":"; then
-            error "Interface $INTERFACE does not exist"
+            report_iproute "Interface $INTERFACE does not exist"
         fi
     fi  
 
+    report_debug ethernet_iproute_up ifup
     ip link set $INTERFACE up
     sleep 2
 
-    if ip link show $INTERFACE|grep -q "NO-CARRIER"; then
-       err_append "No connection"
+    if ip link show $INTERFACE|grep -Fq "NO-CARRIER"; then
+       report_fail "No connection"
        return 1
     fi
      
@@ -31,7 +32,11 @@ ethernet_up() {
         [[ -z "$WPA_CONF" ]] && WPA_CONF="/etc/wpa_supplicant.conf"
         [[ -z "$WPA_OPTS" ]] && WPA_OPTS="-Dwired"
 
-        start_wpa "$INTERFACE" "$WPA_CONF" "$WPA_OPTS"
+        if ! start_wpa $INTERFACE $WPA_CONF $WPA_OPTS; then
+            report_fail "wpa_supplicant did not start, possible configuration error"
+            return 1
+        fi
+        
         if ! wpa_check "$INTERFACE"; then
             ip link set $INTERFACE down
             return 1
@@ -45,41 +50,46 @@ ethernet_up() {
 
         # If using own dns, tell dhcpcd to NOT replace resolv.conf
         [[ -n "$DNS" ]] && DHCP_OPTIONS="-C resolv.conf $DHCP_OPTIONS"
-       
+        
+        report_debug ethernet_iproute_up dhcpcd -qL -t "${DHCP_TIMEOUT:-10}" $DHCP_OPTIONS "$INTERFACE"       
         if ! dhcpcd -qL -t ${DHCP_TIMEOUT:-10} $DHCP_OPTIONS $INTERFACE; then
-            error "DHCP IP lease attempt failed"
+            report_iproute "DHCP IP lease attempt failed"
         fi
         ;;
     static)
         if [[ -n "$ADDR" ]]; then
+            report_debug ethernet_iproute_up ip addr add "$ADDR/24" brd + dev "$INTERFACE"
             if ! ip addr add ${ADDR}/24 brd + dev $INTERFACE; then
-                error "Could not configure interface"
+                report_iproute "Could not configure interface"
             fi
         fi
         if [[ -n "$GATEWAY" ]]; then
+            report_debug ethernet_iproute_up ip route add default via "$GATEWAY"
             if ! ip route add default via $GATEWAY; then
-                error "Adding gateway failed"
+                report_iproute "Adding gateway failed"
             fi
         fi
         ;;
       *)
-        error "Profile error: IP must be either 'dhcp' or 'static'"
+        report_iproute "Profile report_iproute: IP must be either 'dhcp' or 'static'"
         ;;
     esac
 
     if [[ -n "$IPCFG" ]]; then
         for line in "${IPCFG[@]}"; do
-            
+        
+            report_debug ethernet_iproute_up ip "$line"            
             if ! ip $line; then
-                error "Could not configure interface"
+                report_iproute "Could not configure interface"
             fi
         done
     fi
 
     # Set hostname
     if [[ -n "$HOSTNAME" ]]; then
+        report_debug ethernet_iproute_up hostname "$HOSTNAME"
         if ! hostname $HOSTNAME; then
-            error "Cannot set hostname"
+            report_iproute "Cannot set hostname"
         fi
     fi
 
@@ -103,10 +113,12 @@ ethernet_down() {
 
     if [[ "$IP" == "dhcp" ]]; then
         if [[ -f /var/run/dhcpcd-${INTERFACE}.pid ]]; then
+            report_debug ethernet_iproute_down dhcpcd -qx "$INTERFACE"
             dhcpcd -qx $INTERFACE
         fi
     fi
-
+    
+    report_debug ethernet_iproute_down if_down
     ip addr flush $INTERFACE
     quirk "nodown" || ip link set $INTERFACE down
 
diff --git a/src/connections/wireless b/src/connections/wireless
index 0fcf56e..2516042 100644
--- a/src/connections/wireless
+++ b/src/connections/wireless
@@ -24,42 +24,50 @@ wireless_up() {
     if [[ -n "$RFKILL_NAME" ]]; then
         path=$(rfkill_from_name $RFKILL_NAME)
         if [[ $? -ne 0 ]]; then
-            err_append "no rfkill switch with the name $RFKILL_NAME";
+            report_fail "no rfkill switch with the name $RFKILL_NAME";
         fi 
         echo 1 > ${path}/state
+        sleep 1
     fi
 
     # Check if interface exists
     if [[ ! -e /sys/class/net/"$INTERFACE" ]]; then
         if ! echo "$INTERFACE"|grep ":"; then
-            err_append "interface $INTERFACE does not exist"
+            report_fail "interface $INTERFACE does not exist"
             return 1
         fi
     fi  
 
     # Kill any lingering wpa_supplicants.
+    report_debug wireless_up stop_wpa "$INTERFACE"
     stop_wpa $INTERFACE 
             
     # Most drivers (mac80211) need mode set before device is brought up
     # Drivers generally default to managed, but set this to be sure.
     if [[ "$(iwgetid -sm $INTERFACE)" -ne "Managed" ]]; then
+        report_debug wireless_up iwconfig "$INTERFACE" mode managed
         iwconfig $INTERFACE mode managed
     fi
 
-    ifconfig $INTERFACE up 
+    report_debug wireless_up ifup
+    ifconfig $INTERFACE up || return 1
 
     quirk "prescan" && iwlist $INTERFACE scan &> /dev/null # bcm43xx
     quirk "preessid" && eval "iwconfig $INTERFACE mode managed essid \"$ESSID\"" # ipw3945
  
     if checkyesno ${SCAN:-no}; then
+        report_debug wireless_up scanning
         if ! find_essid $INTERFACE "$ESSID"; then
-            err_append "Network not present."
+            report_fail "Network not present."
             return 1
         fi 
     fi
   
     # Manually set iwconfig options
-    [[ "$IWCONFIG" ]] && iwconfig $INTERFACE $IWCONFIG
+    if [[ "$IWCONFIG" ]]; then
+        report_debug wireless_up iwconfig "$INTERFACE" $IWCONFIG        
+        iwconfig $INTERFACE $IWCONFIG
+    fi
 
     # Set to 'none' if not set
     [[ -z "$SECURITY" ]] && SECURITY="none"
@@ -74,15 +82,21 @@ wireless_up() {
         fi 
               
         quirk "predown" && ifconfig $INTERFACE down # madwifi FS#10585
-
+        
+        report_debug wireless_up iwconfig "$INTERFACE" $WEP_OPTS
         if ! eval iwconfig $INTERFACE $WEP_OPTS; then
-            err_append "Could not set wireless configuration."
+            report_fail "Could not set wireless configuration."
             return 1
         fi
 
         quirk "predown" && ifconfig $INTERFACE up # madwifi FS#10585
        
-        wep_check $INTERFACE $TIMEOUT||return 1
+        report_debug ethernet_up wep_check
+        
+        if ! wep_check $INTERFACE $TIMEOUT; then
+            report_fail "WEP Association Failed"
+            return 1
+        fi
         ;;    
     wpa)
 
@@ -98,14 +112,17 @@ wireless_up() {
         if [[ "${#KEY}" == "64" ]]; then
             echo -e "network={ \nssid=\"$ESSID\" \npsk=$KEY \n}">> $WPA_CONF
         elif ! echo "$KEY" | wpa_passphrase "$ESSID" >> $WPA_CONF; then
-            err_append "Configuration generation failed. $(cat $WPA_CONF)"
+            report_fail "Configuration generation failed. $(cat $WPA_CONF)"
             return 1
         fi
 
         # Connect!
         [[ -z "$WPA_OPTS" ]] && WPA_OPTS="-Dwext"
+        report_debug wireless_up start_wpa "$INTERFACE" "$WPA_CONF/wpa.conf" "$WPA_OPTS"
         start_wpa $INTERFACE $WPA_CONF $WPA_OPTS || return 1
+        report_debug wireless_up wpa_check
         if ! wpa_check $INTERFACE $TIMEOUT; then
+            report_fail "WPA Authentication/Association Failed"
             stop_wpa $INTERFACE
             return 1
         fi
@@ -114,8 +131,14 @@ wireless_up() {
         . ${SUBR_DIR}/8021x 
         [[ -z "$WPA_CONF" ]] && WPA_CONF="/etc/wpa_supplicant.conf" # defaults
         [[ -z "$WPA_OPTS" ]] && WPA_OPTS="-Dwext"
-        start_wpa $INTERFACE $WPA_CONF $WPA_OPTS || return 1
+        report_debug wireless_up start_wpa "$INTERFACE" "$WPA_CONF" "$WPA_OPTS"
+        if ! start_wpa $INTERFACE $WPA_CONF $WPA_OPTS; then
+            report_fail "wpa_supplicant did not start, possible configuration error"
+            return 1
+        fi
+        report_debug wireless_up wpa_check
         if ! wpa_check $INTERFACE $TIMEOUT; then
+            report_fail "WPA Authentication/Association Failed"
             stop_wpa $INTERFACE
             return 1
         fi
@@ -131,16 +154,18 @@ wireless_up() {
 }
  
 wireless_down() {
-    load_profile $1
-    . ${SUBR_DIR}/8021x
-    PROFILE=$1 NOETHERNETDOWN=$2 
+    PROFILE=$1 NOETHERNETDOWN=$2
+    load_profile $PROFILE
+    . ${SUBR_DIR}/8021x 
     if ! checkyesno $NOETHERNETDOWN; then
         conn=ethernet
         checkyesno ${IPROUTE:-no} && conn=ethernet-iproute
-        $CONN_DIR/$conn down $1  
+        $CONN_DIR/$conn down $PROFILE  
     fi
+    report_debug wireless_down stop_wpa "$INTERFACE"
     stop_wpa $INTERFACE 
-	[[ "$SECURITY" == "wpa" ]] && rm -f "/tmp/wpa.${1// /}" # remove wpa config
+	[[ "$SECURITY" == "wpa" ]] && rm -f "/tmp/wpa.${PROFILE// /}" # remove wpa config
+    report_debug wireless_down iwconfig "$INTERFACE" essid off key off
     iwconfig $INTERFACE essid off key off &> /dev/null 
     ifconfig $INTERFACE down
     
@@ -148,7 +173,7 @@ wireless_down() {
     if [[ -n "$RFKILL_NAME" ]]; then
         path=$(rfkill_from_name $RFKILL_NAME)
         if [[ $? -ne 0 ]]; then
-            err_append "no rfkill switch with the name $RFKILL_NAME";
+            report_fail "no rfkill switch with the name $RFKILL_NAME";
         fi 
         echo 0 > ${path}/state
     fi
-- 
cgit v1.2.3-24-g4f1b


From 5f43ba3cfab8851b2a380d609029b8061ae51bca Mon Sep 17 00:00:00 2001
From: James Rayner <james@archlinux.org>
Date: Mon, 10 Aug 2009 20:20:12 +1000
Subject: merge improved find_ap/find_essid

---
 src/connections/wireless | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

(limited to 'src/connections')

diff --git a/src/connections/wireless b/src/connections/wireless
index 2516042..f7b4b4c 100644
--- a/src/connections/wireless
+++ b/src/connections/wireless
@@ -32,7 +32,7 @@ wireless_up() {
 
     # Check if interface exists
     if [[ ! -e /sys/class/net/"$INTERFACE" ]]; then
-        if ! echo "$INTERFACE"|grep ":"; then
+        if ! echo "$INTERFACE"|grep -Fq ":"; then
             report_fail "interface $INTERFACE does not exist"
             return 1
         fi
@@ -44,13 +44,13 @@ wireless_up() {
             
     # Most drivers (mac80211) need mode set before device is brought up
     # Drivers generally default to managed, but set this to be sure.
-    if [[ "$(iwgetid -sm $INTERFACE)" -ne "Managed" ]]; then
+    if [[ $(iwgetid -sm $INTERFACE) -ne Managed ]]; then
         report_debug wireless_up iwconfig "$INTERFACE" mode managed
         iwconfig $INTERFACE mode managed
     fi
 
     report_debug wireless_up ifup
-    ifconfig $INTERFACE up || return 1
+    set_interface up $INTERFACE || return 1
 
     quirk "prescan" && iwlist $INTERFACE scan &> /dev/null # bcm43xx
     quirk "preessid" && eval "iwconfig $INTERFACE mode managed essid \"$ESSID\"" # ipw3945
-- 
cgit v1.2.3-24-g4f1b


From 1400a8362753016dacc699679e8b9daebd3c8513 Mon Sep 17 00:00:00 2001
From: James Rayner <james@archlinux.org>
Date: Mon, 10 Aug 2009 20:37:58 +1000
Subject: Add interface setting, a debug mode and interface setting hooks

---
 src/connections/ethernet         | 11 ++---------
 src/connections/ethernet-iproute |  6 ++----
 src/connections/wireless         |  2 +-
 3 files changed, 5 insertions(+), 14 deletions(-)

(limited to 'src/connections')

diff --git a/src/connections/ethernet b/src/connections/ethernet
index 2105ec2..86d8b96 100644
--- a/src/connections/ethernet
+++ b/src/connections/ethernet
@@ -12,8 +12,7 @@ ethernet_up() {
     fi  
 
     report_debug ethernet_up ifup
-    ip link set $INTERFACE up
-    sleep 2
+    set_interface up $INTERFACE
 
     if ip link show $INTERFACE|grep -q "NO-CARRIER"; then
        report_fail "No connection"
@@ -135,13 +134,7 @@ ethernet_down() {
     esac
     
     report_debug ethernet_down ifdown
-    ifconfig $INTERFACE 0.0.0.0
-    
-    case "$CONNECTION" in # Keep interface up for wireless
-        ethernet|ethernet-old) 
-            quirk "nodown" || ifconfig $INTERFACE down
-            ;;
-    esac
+    set_interface down $INTERFACE
 }
 
 # Returns status of profile - is it still functional?
diff --git a/src/connections/ethernet-iproute b/src/connections/ethernet-iproute
index 75c99f3..2f0c485 100644
--- a/src/connections/ethernet-iproute
+++ b/src/connections/ethernet-iproute
@@ -19,8 +19,7 @@ ethernet_up() {
     fi  
 
     report_debug ethernet_iproute_up ifup
-    ip link set $INTERFACE up
-    sleep 2
+    set_interface up $INTERFACE
 
     if ip link show $INTERFACE|grep -Fq "NO-CARRIER"; then
        report_fail "No connection"
@@ -119,8 +118,7 @@ ethernet_down() {
     fi
     
     report_debug ethernet_iproute_down if_down
-    ip addr flush $INTERFACE
-    quirk "nodown" || ip link set $INTERFACE down
+    set_interface down $INTERFACE
 
 }
 
diff --git a/src/connections/wireless b/src/connections/wireless
index f7b4b4c..10f751e 100644
--- a/src/connections/wireless
+++ b/src/connections/wireless
@@ -167,7 +167,7 @@ wireless_down() {
 	[[ "$SECURITY" == "wpa" ]] && rm -f "/tmp/wpa.${PROFILE// /}" # remove wpa config
     report_debug wireless_down iwconfig "$INTERFACE" essid off key off
     iwconfig $INTERFACE essid off key off &> /dev/null 
-    ifconfig $INTERFACE down
+    set_interface down $INTERFACE
     
     # If rfkill is specified, disable device.
     if [[ -n "$RFKILL_NAME" ]]; then
-- 
cgit v1.2.3-24-g4f1b