ShellCheck - shell script analysis tool
version: 0.11.0
license: GNU General Public License, version 3
website: https://www.shellcheck.net
Redirect to /tmp/tmp.nflWCR3qqH and /tmp/tmp.V6eeV3DiuD
Tree base:
beca60725e5f ("arp: discard invalid sha addr (b/mcast ARP poison)")
Now at:
c93253430f90 ("selftests/net: add no ARP b/mcast,null poison test")
 ====== Checking before the patch ======
 ====== Checking the tree with the patch ======
Checking tools/testing/selftests/net/arp_no_invalid_sha_poision.sh - 9c13f1c0fbec73aeed13795ec5b2f0434a33d9a8e6a3bbd5d8785ed599b9e107


In arp_no_invalid_sha_poision.sh line 18:
nsid=100
^--^ SC2034 (warning): nsid appears unused. Verify use (or export if used externally).


In arp_no_invalid_sha_poision.sh line 28:
	ip link set dev veth1 netns ${PEER_NS}
                                    ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	ip link set dev veth1 netns "${PEER_NS}"


In arp_no_invalid_sha_poision.sh line 29:
	ip netns exec ${PEER_NS} ip link set dev veth1 up
                      ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	ip netns exec "${PEER_NS}" ip link set dev veth1 up


In arp_no_invalid_sha_poision.sh line 31:
	ip netns exec ${PEER_NS} ip addr add ${V4_ADDR1}/24 dev veth1
                      ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	ip netns exec "${PEER_NS}" ip addr add ${V4_ADDR1}/24 dev veth1


In arp_no_invalid_sha_poision.sh line 32:
	ip netns exec ${PEER_NS} ip route add default via ${V4_ADDR0} dev veth1
                      ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	ip netns exec "${PEER_NS}" ip route add default via ${V4_ADDR0} dev veth1


In arp_no_invalid_sha_poision.sh line 37:
	ip netns exec ${PEER_NS} \
                      ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	ip netns exec "${PEER_NS}" \


In arp_no_invalid_sha_poision.sh line 40:
	ip netns exec ${PEER_NS} \
                      ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	ip netns exec "${PEER_NS}" \


In arp_no_invalid_sha_poision.sh line 45:
	veth1_mac="$(ip netns exec ${PEER_NS} ip -j link show veth1 | \
                                   ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	veth1_mac="$(ip netns exec "${PEER_NS}" ip -j link show veth1 | \


In arp_no_invalid_sha_poision.sh line 52:
	cleanup_ns ${PEER_NS}
                   ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	cleanup_ns "${PEER_NS}"


In arp_no_invalid_sha_poision.sh line 63:
	ip netns exec ${PEER_NS} ip neigh flush dev veth1 >/dev/null 2>&1
                      ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	ip netns exec "${PEER_NS}" ip neigh flush dev veth1 >/dev/null 2>&1


In arp_no_invalid_sha_poision.sh line 64:
	ip netns exec ${PEER_NS} ping -c 1 ${V4_ADDR0} >/dev/null 2>&1
                      ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	ip netns exec "${PEER_NS}" ping -c 1 ${V4_ADDR0} >/dev/null 2>&1


In arp_no_invalid_sha_poision.sh line 67:
	./arp_send ${veth0_ifindex} ${l2_dmac} ${VALID_MAC} ${op} \
                   ^--------------^ SC2086 (info): Double quote to prevent globbing and word splitting.
                                    ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.
                                                            ^---^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	./arp_send "${veth0_ifindex}" "${l2_dmac}" ${VALID_MAC} "${op}" \


In arp_no_invalid_sha_poision.sh line 70:
	neigh=$(ip netns exec ${PEER_NS} ip neigh show ${V4_ADDR0} | \
                              ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	neigh=$(ip netns exec "${PEER_NS}" ip neigh show ${V4_ADDR0} | \


In arp_no_invalid_sha_poision.sh line 74:
		ip netns exec ${PEER_NS} ip neigh show ${V4_ADDR0}
                              ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
		ip netns exec "${PEER_NS}" ip neigh show ${V4_ADDR0}


In arp_no_invalid_sha_poision.sh line 80:
	./arp_send ${veth0_ifindex} ${l2_dmac} ${VALID_MAC} ${op} \
                   ^--------------^ SC2086 (info): Double quote to prevent globbing and word splitting.
                                    ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.
                                                            ^---^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	./arp_send "${veth0_ifindex}" "${l2_dmac}" ${VALID_MAC} "${op}" \


In arp_no_invalid_sha_poision.sh line 81:
		${V4_ADDR0} ${tmac} ${V4_ADDR0} ${tmac}
                            ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting.
                                                ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
		${V4_ADDR0} "${tmac}" ${V4_ADDR0} "${tmac}"


In arp_no_invalid_sha_poision.sh line 83:
	neigh=$(ip netns exec ${PEER_NS} ip neigh show ${V4_ADDR0} | \
                              ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	neigh=$(ip netns exec "${PEER_NS}" ip neigh show ${V4_ADDR0} | \


In arp_no_invalid_sha_poision.sh line 84:
		grep ${tmac})
                     ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
		grep "${tmac}")


In arp_no_invalid_sha_poision.sh line 87:
		ip netns exec ${PEER_NS} ip neigh show ${V4_ADDR0}
                              ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
		ip netns exec "${PEER_NS}" ip neigh show ${V4_ADDR0}


In arp_no_invalid_sha_poision.sh line 97:
	if [ ${rc} == 0 ]; then
             ^---^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	if [ "${rc}" == 0 ]; then


In arp_no_invalid_sha_poision.sh line 137:
	run_no_arp_poisoning ${veth1_mac} ${BCAST_MAC} ${ARP_REPLY}
                             ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	run_no_arp_poisoning "${veth1_mac}" ${BCAST_MAC} ${ARP_REPLY}


In arp_no_invalid_sha_poision.sh line 141:
	run_no_arp_poisoning ${veth1_mac} ${NULL_MAC} ${ARP_REPLY}
                             ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	run_no_arp_poisoning "${veth1_mac}" ${NULL_MAC} ${ARP_REPLY}


In arp_no_invalid_sha_poision.sh line 145:
	run_no_arp_poisoning ${veth1_mac} ${BCAST_MAC} ${ARP_REQ}
                             ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	run_no_arp_poisoning "${veth1_mac}" ${BCAST_MAC} ${ARP_REQ}


In arp_no_invalid_sha_poision.sh line 149:
	run_no_arp_poisoning ${veth1_mac} ${NULL_MAC} ${ARP_REQ}
                             ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	run_no_arp_poisoning "${veth1_mac}" ${NULL_MAC} ${ARP_REQ}


In arp_no_invalid_sha_poision.sh line 153:
	run_no_arp_poisoning ${veth1_mac} ${MCAST_MAC} ${ARP_REQ}
                             ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	run_no_arp_poisoning "${veth1_mac}" ${MCAST_MAC} ${ARP_REQ}


In arp_no_invalid_sha_poision.sh line 157:
	run_no_arp_poisoning ${veth1_mac} ${MCAST_MAC} ${ARP_REPLY}
                             ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	run_no_arp_poisoning "${veth1_mac}" ${MCAST_MAC} ${ARP_REPLY}

For more information:
  https://www.shellcheck.net/wiki/SC2034 -- nsid appears unused. Verify use (...
  https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing ...