On Wed, Dec 06, 2023 at 10:17:26AM +0100, Claudio Jeker wrote: > On Wed, Dec 06, 2023 at 12:57:57AM +0100, Alexander Bluhm wrote: > > On Wed, Dec 06, 2023 at 01:39:40AM +0300, Vitaliy Makkoveev wrote: > > > > Diff makes sense in any case. > > > > > > > > > > Just checked, socket6_send() is identical to socket_send() and needs > > > to be reworked in the same way. > > > > New diff for v4 and v6. The other callers seem to be correct. I > > will run this through regress and commit regardless whether it fixes > > the reported bug. The current code is wrong anyway. > > > > ok? > > AFAIK there is no ip_mroute.c regress coverage. Diff is OK claudio@
I have written tests in 2019 which use a small dummy multicast routing daemon. Fix passes all regress with witness kernel. /usr/src/regress/sys/netinet/mcast /usr/src/regress/sys/netinet6/mcast6 Diff is commited. Jo Geraerts: Please test anyway so we know whether it fixes your bug. bluhm