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.tdbrUhHBdz and /tmp/tmp.XNqjuuWbcg
Tree base:
23f573b50117 ("neigh: discard lladdr bcast/null (bcast poison)")
Now at:
dd7e8c72cdf8 ("selftests/net: add no NDP bcast/null poison test")
 ====== Checking before the patch ======
 ====== Checking the tree with the patch ======
Checking tools/testing/selftests/net/arp_ndisc_no_bcastnull_poison.sh - 4368c3b053e8875bfae1d0cb6192a6eb1c97529f8d98210b0725e6e4fe875dd9


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


In arp_ndisc_no_bcastnull_poison.sh line 35:
	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_ndisc_no_bcastnull_poison.sh line 36:
	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_ndisc_no_bcastnull_poison.sh line 39:
	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_ndisc_no_bcastnull_poison.sh line 40:
	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_ndisc_no_bcastnull_poison.sh line 42:
	ip netns exec ${PEER_NS} ip addr add ${V6_ADDR1}/64 dev veth1
                      ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.

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


In arp_ndisc_no_bcastnull_poison.sh line 43:
	ip netns exec ${PEER_NS} ip route add default via ${V6_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 ${V6_ADDR0} dev veth1


In arp_ndisc_no_bcastnull_poison.sh line 48:
	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_ndisc_no_bcastnull_poison.sh line 51:
	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_ndisc_no_bcastnull_poison.sh line 56:
	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_ndisc_no_bcastnull_poison.sh line 63:
	cleanup_ns ${PEER_NS}
                   ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	cleanup_ns "${PEER_NS}"


In arp_ndisc_no_bcastnull_poison.sh line 74:
	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_ndisc_no_bcastnull_poison.sh line 75:
	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_ndisc_no_bcastnull_poison.sh line 78:
	./arp_send ${veth0_ifindex} ${BCAST_MAC} ${VALID_MAC} ${op} \
                   ^--------------^ 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}" ${BCAST_MAC} ${VALID_MAC} "${op}" \


In arp_ndisc_no_bcastnull_poison.sh line 81:
	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_ndisc_no_bcastnull_poison.sh line 85:
		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_ndisc_no_bcastnull_poison.sh line 91:
	./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_ndisc_no_bcastnull_poison.sh line 92:
		${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_ndisc_no_bcastnull_poison.sh line 94:
	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_ndisc_no_bcastnull_poison.sh line 95:
		grep ${tmac})
                     ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting.

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


In arp_ndisc_no_bcastnull_poison.sh line 98:
		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_ndisc_no_bcastnull_poison.sh line 118:
	ip netns exec ${PEER_NS} ip -6 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 -6 neigh flush dev veth1 >/dev/null 2>&1


In arp_ndisc_no_bcastnull_poison.sh line 119:
	ip netns exec ${PEER_NS} ping -c 1 ${V6_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 ${V6_ADDR0} >/dev/null 2>&1


In arp_ndisc_no_bcastnull_poison.sh line 122:
	./ndisc_send ${veth0_ifindex} ${l2_dmac} ${VALID_MAC} ${dst_ip} \
                     ^--------------^ 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:
	./ndisc_send "${veth0_ifindex}" "${l2_dmac}" ${VALID_MAC} "${dst_ip}" \


In arp_ndisc_no_bcastnull_poison.sh line 123:
		${V6_ADDR0} ${tip} ${op} ${VALID_MAC}
                                   ^---^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
		${V6_ADDR0} ${tip} "${op}" ${VALID_MAC}


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

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


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

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


In arp_ndisc_no_bcastnull_poison.sh line 134:
	./ndisc_send ${veth0_ifindex} ${l2_dmac} ${VALID_MAC} ${dst_ip} \
                     ^--------------^ 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:
	./ndisc_send "${veth0_ifindex}" "${l2_dmac}" ${VALID_MAC} "${dst_ip}" \


In arp_ndisc_no_bcastnull_poison.sh line 135:
		${V6_ADDR0} ${tip} ${op} ${tmac}
                                   ^---^ SC2086 (info): Double quote to prevent globbing and word splitting.
                                         ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
		${V6_ADDR0} ${tip} "${op}" "${tmac}"


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

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


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

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


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

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


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

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


In arp_ndisc_no_bcastnull_poison.sh line 182:
	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_ndisc_no_bcastnull_poison.sh line 186:
	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_ndisc_no_bcastnull_poison.sh line 190:
	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_ndisc_no_bcastnull_poison.sh line 194:
	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_ndisc_no_bcastnull_poison.sh line 267:
	run_no_ndp_poisoning ${veth1_mac} ${V6_ADDR1} ${NA} ${BCAST_MAC}
                             ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	run_no_ndp_poisoning "${veth1_mac}" ${V6_ADDR1} ${NA} ${BCAST_MAC}


In arp_ndisc_no_bcastnull_poison.sh line 271:
	run_no_ndp_poisoning ${veth1_mac} ${V6_ADDR1} ${NA} ${NULL_MAC}
                             ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	run_no_ndp_poisoning "${veth1_mac}" ${V6_ADDR1} ${NA} ${NULL_MAC}


In arp_ndisc_no_bcastnull_poison.sh line 311:
	run_no_ndp_poisoning ${veth1_mac} ${V6_ADDR1} ${NS} ${BCAST_MAC}
                             ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	run_no_ndp_poisoning "${veth1_mac}" ${V6_ADDR1} ${NS} ${BCAST_MAC}


In arp_ndisc_no_bcastnull_poison.sh line 315:
	run_no_ndp_poisoning ${veth1_mac} ${V6_ADDR1} ${NS} ${NULL_MAC}
                             ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
	run_no_ndp_poisoning "${veth1_mac}" ${V6_ADDR1} ${NS} ${NULL_MAC}

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 ...