lldpd: note about capabilities
[openwrt/staging/stintel.git] / package / network / services / lldpd / files / lldpd.init
index e13128e2d6bfcbca8bc6716a8c62ec6e727883ad..6707476e553bf6607a592b11c2d7c78600cda792 100644 (file)
@@ -1,5 +1,6 @@
 #!/bin/sh /etc/rc.common
 # Copyright (C) 2008-2015 OpenWrt.org
+# shellcheck disable=1091,2034,3037,3043,3045
 
 START=90
 STOP=01
@@ -39,38 +40,38 @@ get_config_restart_hash() {
        config_load 'lldpd'
 
        config_get      v 'config' 'lldp_class'; append _string "$v" ","
-       if [ "$CONFIG_LLDPD_WITH_SNMP" == "y" ]; then
+       if [ "$CONFIG_LLDPD_WITH_SNMP" = "y" ]; then
                config_get      v 'config' 'agentxsocket'; append _string "$v" ","
        fi
        config_get      v 'config' 'cid_interface'; append _string "$v" ","
        config_get      v 'config' 'filter'; append _string "$v" ","
        config_get_bool v 'config' 'readonly_mode'; append _string "$v" ","
        config_get_bool v 'config' 'lldp_no_version'; append _string "$v" ","
-       if [ "$CONFIG_LLDPD_WITH_LLDPMED" == "y" ]; then
+       if [ "$CONFIG_LLDPD_WITH_LLDPMED" = "y" ]; then
                config_get_bool v 'config' 'lldpmed_no_inventory'; append _string "$v" ","
        fi
        config_get_bool v 'config' 'enable_lldp' 1; append _string "$v" ","
        config_get_bool v 'config' 'force_lldp'; append _string "$v" ","
-       if [ "$CONFIG_LLDPD_WITH_CDP" == "y" ]; then
+       if [ "$CONFIG_LLDPD_WITH_CDP" = "y" ]; then
                config_get_bool v 'config' 'enable_cdp'; append _string "$v" ","
                config_get      v 'config' 'cdp_version'; append _string "$v" ","
                config_get_bool v 'config' 'force_cdp'; append _string "$v" ","
                config_get_bool v 'config' 'force_cdpv2'; append _string "$v" ","
        fi
-       if [ "$CONFIG_LLDPD_WITH_EDP" == "y" ]; then
+       if [ "$CONFIG_LLDPD_WITH_EDP" = "y" ]; then
                config_get_bool v 'config' 'enable_edp'; append _string "$v" ","
                config_get_bool v 'config' 'force_edp'; append _string "$v" ","
        fi
-       if [ "$CONFIG_LLDPD_WITH_FDP" == "y" ]; then
+       if [ "$CONFIG_LLDPD_WITH_FDP" = "y" ]; then
                config_get_bool v 'config' 'enable_fdp'; append _string "$v" ","
                config_get_bool v 'config' 'force_fdp'; append _string "$v" ","
        fi
-       if [ "$CONFIG_LLDPD_WITH_SONMP" == "y" ]; then
+       if [ "$CONFIG_LLDPD_WITH_SONMP" = "y" ]; then
                config_get_bool v 'config' 'enable_sonmp'; append _string "$v" ","
                config_get_bool v 'config' 'force_sonmp'; append _string "$v" ","
        fi
 
-       _hash=`echo -n "${_string}" | md5sum | awk '{ print \$1 }'`
+       _hash=$(echo -n "${_string}" | md5sum | awk '{ print $1 }')
        export -n "$var=$_hash"
 }
 
@@ -105,15 +106,26 @@ write_lldpd_conf()
        local lldp_mgmt_ip
        config_get lldp_mgmt_ip 'config' 'lldp_mgmt_ip'
 
+       # Configurable capabilities in lldpd >= v1.0.15
        local lldp_syscapabilities
        config_get lldp_syscapabilities 'config' 'lldp_syscapabilities'
 
-       if [ "$CONFIG_LLDPD_WITH_LLDPMED" == "y" ]; then
+       if [ "$CONFIG_LLDPD_WITH_LLDPMED" = "y" ]; then
                local lldpmed_fast_start
                config_get_bool lldpmed_fast_start 'config' 'lldpmed_fast_start' 0
 
                local lldpmed_fast_start_tx_interval
                config_get lldpmed_fast_start_tx_interval 'config' 'lldpmed_fast_start_tx_interval' 0
+
+               local lldp_location
+               config_get lldp_location 'config' 'lldp_location'
+
+               local lldp_class
+               config_get lldp_class 'config' 'lldp_class'
+
+               local lldp_policy
+               config_get lldp_policy 'config' 'lldp_policy'
+
        fi
 
        local lldp_agenttype
@@ -138,21 +150,34 @@ write_lldpd_conf()
        [ -n "$lldp_hostname" ] && echo "configure system hostname" "\"$lldp_hostname\"" >> "$LLDPD_CONF"
        [ -n "$lldp_mgmt_ip" ] && echo "configure system ip management pattern" "\"$lldp_mgmt_ip\"" >> "$LLDPD_CONF"
        [ -n "$lldp_syscapabilities" ] && echo "configure system capabilities enabled $lldp_syscapabilities" >> "$LLDPD_CONF"
-       if [ "$CONFIG_LLDPD_WITH_LLDPMED" == "y" ] && [ $lldpmed_fast_start -gt 0 ]; then
-               if [ $lldpmed_fast_start_tx_interval -gt 0 ]; then
+       if [ "$CONFIG_LLDPD_WITH_LLDPMED" = "y" ] && [ "$lldpmed_fast_start" -gt 0 ]; then
+               if [ "$lldpmed_fast_start_tx_interval" -gt 0 ]; then
                        echo "configure med fast-start tx-interval $lldpmed_fast_start_tx_interval" >> "$LLDPD_CONF"
                else
                        echo "configure med fast-start enable" >> "$LLDPD_CONF"
                fi
        fi
-       [ -n "$lldp_agenttype" ] && echo "configure lldp agent-type $lldp_agenttype" >> "$LLDPD_CONF"
-       [ -n "$lldp_portidsubtype" ] && echo "configure lldp portidsubtype $lldp_portidsubtype" >> "$LLDPD_CONF"
+       if [ "$CONFIG_LLDPD_WITH_LLDPMED" = "y" ]; then
+               # other 'configure med xxx' statements go here
+
+               [ -n "$lldp_location" ] && echo "configure med location" "$lldp_location" >> "$LLDPD_CONF"
+
+               # Manual states that if Class (-M) is 2 or 3, at least one network policy must be defined
+               case "$lldp_class" in
+                       2 | 3 ) [ -n "$lldp_policy" ] && echo "configure med policy $lldp_policy" >> "$LLDPD_CONF"
+                       ;;
+               esac
+
+       fi
+
+       [ -n "$lldp_agenttype" ] && echo "configure lldp agent-type" "\"$lldp_agenttype\"" >> "$LLDPD_CONF"
+       [ -n "$lldp_portidsubtype" ] && echo "configure lldp portidsubtype" "\"$lldp_portidsubtype\"" >> "$LLDPD_CONF"
        [ -n "$lldp_platform" ] && echo "configure system platform" "\"$lldp_platform\"" >> "$LLDPD_CONF"
-       [ $lldp_tx_interval -gt 0 ] && echo "configure lldp tx-interval $lldp_tx_interval" >> "$LLDPD_CONF"
-       [ $lldp_tx_hold -gt 0 ] && echo "configure lldp tx-hold $lldp_tx_hold" >> "$LLDPD_CONF"
+       [ -n "$lldp_tx_interval" ] && echo "configure lldp tx-interval $lldp_tx_interval" >> "$LLDPD_CONF"
+       [ "$lldp_tx_hold" -gt 0 ] && echo "configure lldp tx-hold $lldp_tx_hold" >> "$LLDPD_CONF"
 
        # Since lldpd's sysconfdir is /tmp, we'll symlink /etc/lldpd.d to /tmp/$LLDPD_CONFS_DIR
-       [ -e $LLDPD_CONFS_DIR ] || ln -s /etc/lldpd.d $LLDPD_CONFS_DIR
+       [ -e "$LLDPD_CONFS_DIR" ] || ln -s /etc/lldpd.d "$LLDPD_CONFS_DIR"
 }
 
 start_service() {
@@ -170,7 +195,6 @@ start_service() {
        local enable_edp
        local force_edp
        local lldp_class
-       local lldp_location
        local lldp_no_version
        local lldpmed_no_inventory
        local readonly_mode
@@ -180,32 +204,31 @@ start_service() {
        config_load 'lldpd'
        config_get_bool enable_lldp 'config' 'enable_lldp' 1
        config_get_bool force_lldp 'config' 'force_lldp' 0
-       if [ "$CONFIG_LLDPD_WITH_CDP" == "y" ]; then
+       if [ "$CONFIG_LLDPD_WITH_CDP" = "y" ]; then
                config_get_bool enable_cdp 'config' 'enable_cdp' 0
                config_get cdp_version 'config' 'cdp_version' 'cdpv1v2'
                config_get_bool force_cdp 'config' 'force_cdp' 0
                config_get_bool force_cdpv2 'config' 'force_cdpv2' 0
        fi
-       if [ "$CONFIG_LLDPD_WITH_FDP" == "y" ]; then
+       if [ "$CONFIG_LLDPD_WITH_FDP" = "y" ]; then
                config_get_bool enable_fdp 'config' 'enable_fdp' 0
                config_get_bool force_fdp 'config' 'force_fdp' 0
        fi
-       if [ "$CONFIG_LLDPD_WITH_SONMP" == "y" ]; then
+       if [ "$CONFIG_LLDPD_WITH_SONMP" = "y" ]; then
                config_get_bool enable_sonmp 'config' 'enable_sonmp' 0
                config_get_bool force_sonmp 'config' 'force_sonmp' 0
        fi
-       if [ "$CONFIG_LLDPD_WITH_EDP" == "y" ]; then
+       if [ "$CONFIG_LLDPD_WITH_EDP" = "y" ]; then
                config_get_bool enable_edp 'config' 'enable_edp' 0
                config_get_bool force_edp 'config' 'force_edp' 0
        fi
        config_get lldp_class 'config' 'lldp_class'
-       config_get lldp_location 'config' 'lldp_location'
        config_get_bool lldp_no_version 'config' 'lldp_no_version' 0
-       if [ "$CONFIG_LLDPD_WITH_LLDPMED" == "y" ]; then
+       if [ "$CONFIG_LLDPD_WITH_LLDPMED" = "y" ]; then
                config_get_bool lldpmed_no_inventory 'config' 'lldpmed_no_inventory' 0
        fi
        config_get_bool readonly_mode 'config' 'readonly_mode' 0
-       if [ "$CONFIG_LLDPD_WITH_SNMP" == "y" ]; then
+       if [ "$CONFIG_LLDPD_WITH_SNMP" = "y" ]; then
                config_get agentxsocket 'config' 'agentxsocket'
        fi
        config_get filter 'config' 'filter' 15
@@ -220,8 +243,8 @@ start_service() {
        procd_set_param command ${LLDPDBIN}
        procd_append_param command -d
 
-       if [ $enable_lldp -gt 0 ]; then
-               if [ $force_lldp -gt 0 ]; then
+       if [ "$enable_lldp" -gt 0 ]; then
+               if [ "$force_lldp" -gt 0 ]; then
                        procd_append_param command '-l'
                fi
        else
@@ -229,20 +252,20 @@ start_service() {
                procd_append_param command '-ll'
        fi
 
-       if [ "$CONFIG_LLDPD_WITH_CDP" == "y" ] && [ $enable_cdp -gt 0 ]; then
-               if [ $cdp_version == "cdpv2" ]; then
-                       if [ $force_cdp -gt 0 ]; then
+       if [ "$CONFIG_LLDPD_WITH_CDP" = "y" ] && [ "$enable_cdp" -gt 0 ]; then
+               if [ "$cdp_version" = "cdpv2" ]; then
+                       if [ "$force_cdp" -gt 0 ]; then
                                # CDPv1 disabled, CDPv2 forced
                                procd_append_param command '-ccccc'
                        else
                                # CDPv1 disabled, CDPv2 enabled
                                procd_append_param command '-cccc'
                        fi
-               elif [ $cdp_version == "cdpv1v2" ]; then
-                       if [ $force_cdp -gt 0 ] && [ $force_cdpv2 -gt 0 ]; then
+               elif [ "$cdp_version" = "cdpv1v2" ]; then
+                       if [ "$force_cdp" -gt 0 ] && [ "$force_cdpv2" -gt 0 ]; then
                                # CDPv1 enabled, CDPv2 forced
                                procd_append_param command '-ccc'
-                       elif [ $force_cdp -gt 0 ]; then
+                       elif [ "$force_cdp" -gt 0 ]; then
                                # CDPv1 forced, CDPv2 enabled
                                procd_append_param command '-cc'
                        else
@@ -252,8 +275,8 @@ start_service() {
                fi
        fi
 
-       if [ "$CONFIG_LLDPD_WITH_FDP" == "y" ] && [ $enable_fdp -gt 0 ]; then
-               if [ $force_fdp -gt 0 ]; then
+       if [ "$CONFIG_LLDPD_WITH_FDP" = "y" ] && [ "$enable_fdp" -gt 0 ]; then
+               if [ "$force_fdp" -gt 0 ]; then
                        # FDP enabled and forced
                        procd_append_param command '-ff'
                else
@@ -262,8 +285,8 @@ start_service() {
                fi
        fi
 
-       if [ "$CONFIG_LLDPD_WITH_SONMP" == "y" ] && [ $enable_sonmp -gt 0 ]; then
-               if [ $force_sonmp -gt 0 ]; then
+       if [ "$CONFIG_LLDPD_WITH_SONMP" = "y" ] && [ "$enable_sonmp" -gt 0 ]; then
+               if [ "$force_sonmp" -gt 0 ]; then
                        # SONMP enabled and forced
                        procd_append_param command '-ss'
                else
@@ -272,8 +295,8 @@ start_service() {
                fi
        fi
 
-       if [ "$CONFIG_LLDPD_WITH_EDP" == "y" ] && [ $enable_edp -gt 0 ]; then
-               if [ $force_edp -gt 0 ]; then
+       if [ "$CONFIG_LLDPD_WITH_EDP" = "y" ] && [ "$enable_edp" -gt 0 ]; then
+               if [ "$force_edp" -gt 0 ]; then
                        # EDP enabled and forced
                        procd_append_param command '-ee'
                else
@@ -282,11 +305,11 @@ start_service() {
                fi
        fi
 
-       [ $readonly_mode -gt 0 ] && procd_append_param command '-r'
-       [ $lldp_no_version -gt 0 ] && procd_append_param command '-k'
-       [ "$CONFIG_LLDPD_WITH_LLDPMED" == "y" ] && [ $lldpmed_no_inventory -gt 0 ] && procd_append_param command '-i'
+       [ "$readonly_mode" -gt 0 ] && procd_append_param command '-r'
+       [ "$lldp_no_version" -gt 0 ] && procd_append_param command '-k'
+       [ "$CONFIG_LLDPD_WITH_LLDPMED" = "y" ] && [ "$lldpmed_no_inventory" -gt 0 ] && procd_append_param command '-i'
        [ -n "$lldp_class" ] && procd_append_param command -M "$lldp_class"
-       [ "$CONFIG_LLDPD_WITH_SNMP" == "y" ] && [ -n "$agentxsocket" ] && procd_append_param command -x -X "$agentxsocket"
+       [ "$CONFIG_LLDPD_WITH_SNMP" = "y" ] && [ -n "$agentxsocket" ] && procd_append_param command -x -X "$agentxsocket"
        [ -n "$filter" ] && procd_append_param command -H "$filter"
 
     # ChassisID interfaces
@@ -300,7 +323,7 @@ start_service() {
 
        local restart_hash
        get_config_restart_hash restart_hash
-       echo -n "$restart_hash" > $LLDPD_RESTART_HASH
+       echo -n "$restart_hash" > "$LLDPD_RESTART_HASH"
 
        # set auto respawn behavior
        procd_set_param respawn
@@ -318,7 +341,7 @@ reload_service() {
        local config_hash=""
 
        get_config_restart_hash config_hash
-       if [ -f ${LLDPD_RESTART_HASH} ]; then running_hash=`cat $LLDPD_RESTART_HASH`; fi
+       [ -f ${LLDPD_RESTART_HASH} ] && running_hash=$(cat "$LLDPD_RESTART_HASH")
 
        if [ "x$running_hash" != "x$config_hash" ]; then
                # Restart LLDPd
@@ -327,7 +350,7 @@ reload_service() {
                return 0
        fi
 
-       $LLDPCLI -u $LLDPSOCKET &> /dev/null <<-EOF
+       $LLDPCLI -u "$LLDPSOCKET" >/dev/null 2>&1 <<-EOF
                pause
                unconfigure lldp custom-tlv
                unconfigure system interface pattern
@@ -336,8 +359,8 @@ reload_service() {
                unconfigure system ip management pattern
                unconfigure system platform
        EOF
-       if [ "$CONFIG_LLDPD_WITH_LLDPMED" == "y" ]; then
-               $LLDPCLI -u $LLDPSOCKET &> /dev/null <<-EOF
+       if [ "$CONFIG_LLDPD_WITH_LLDPMED" = "y" ]; then
+               $LLDPCLI -u "$LLDPSOCKET" >/dev/null 2>&1 <<-EOF
                        unconfigure med fast-start
                EOF
 
@@ -345,9 +368,9 @@ reload_service() {
        # Rewrite lldpd.conf
        # If something changed it should be included by the lldpcli call
        write_lldpd_conf
-       $LLDPCLI -u $LLDPSOCKET -c $LLDPD_CONF -c $LLDPD_CONFS_DIR &> /dev/null
+       $LLDPCLI -u "$LLDPSOCKET" -c "$LLDPD_CONF" -c "$LLDPD_CONFS_DIR" >/dev/null 2>&1 
        # Broadcast update over the wire
-       $LLDPCLI -u $LLDPSOCKET &> /dev/null <<-EOF
+       $LLDPCLI -u "$LLDPSOCKET" >/dev/null 2>&1 <<-EOF
                resume
                update
        EOF
@@ -355,6 +378,6 @@ reload_service() {
 }
 
 stop_service() {
-       rm -rf ${LLDPD_RUN} $LLDPSOCKET 2>/dev/null
+       rm -rf ${LLDPD_RUN} "$LLDPSOCKET" >/dev/null 2>&1
 }