# HG changeset patch # User Roy Marples # Date 1601362824 -3600 # Node ID a3d5af47a4e32445fc583cab3870841cc2b13c29 # Parent 1712c2fd62a48d1cbeb04522df55bedbf58988dd Add a comment to prior incase I do something dumb like removing it in the future if I forgot why it was there. diff -r 1712c2fd62a4 -r a3d5af47a4e3 src/dhcp.c --- a/src/dhcp.c Tue Sep 29 07:30:27 2020 +0100 +++ b/src/dhcp.c Tue Sep 29 08:00:24 2020 +0100 @@ -3477,6 +3477,13 @@ #ifdef PRIVSEP const struct dhcp_state *state = D_CSTATE(ifp); + /* It's possible that an interface departs and arrives in short + * order to receive a BPF frame out of order. + * There is a similar check in ARP, but much lower down the stack. + * It's not needed for other inet protocols because we send the + * message as a whole and select the interface off that and then + * check state. BPF on the other hand is very interface + * specific and we do need this check. */ if (state == NULL) return;