Used but not initialised: $ grep arp_list if_ether.c LIST_ENTRY(llinfo_arp) la_list; /* [mN] global arp_list */ LIST_HEAD(, llinfo_arp) arp_list; /* [mN] list of all llinfo_arp structures */ /* Net lock is exclusive, no arp mutex needed for arp_list here. */ LIST_FOREACH_SAFE(la, &arp_list, la_list, nla) { LIST_INSERT_HEAD(&arp_list, la, la_list);
It only works because arp_list is global and thus zero initialised, so #define LIST_INIT(head) do { \ LIST_FIRST(head) = LIST_END(head); \ } while (0) is practically a NOOP, but we must not rely on that. Use proper queue(9) init (in analogy to nd6_init() and nd6_list). OK? Index: netinet/if_ether.c =================================================================== RCS file: /cvs/src/sys/netinet/if_ether.c,v retrieving revision 1.256 diff -u -p -r1.256 if_ether.c --- netinet/if_ether.c 31 Jan 2023 13:41:54 -0000 1.256 +++ netinet/if_ether.c 2 Mar 2023 09:19:53 -0000 @@ -146,6 +146,7 @@ arpinit(void) { static struct timeout arptimer_to; + LIST_INIT(&arp_list); pool_init(&arp_pool, sizeof(struct llinfo_arp), 0, IPL_SOFTNET, 0, "arp", NULL);