========== shellcheck - FAILED ====== Checking before the patch ====== ====== Checking the tree with the patch ====== tools/testing/selftests/net/arp_ndisc_no_bcastnull_poison.sh is a new file, but not shellcheck compliant New errors added --- /tmp/tmp.tdbrUhHBdz 2025-12-21 22:03:52.757103815 -0500 +++ /tmp/tmp.XNqjuuWbcg 2025-12-21 22:03:53.185099736 -0500 @@ -0,0 +1,337 @@ + +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 ...