On Mon, Nov 13, 2017 at 08:21:59AM +0100, Florian Westphal wrote: > Reason is that some places do this: > > rtnl_register(pf, RTM_FOO, doit, NULL, 0); > rtnl_register(pf, RTM_FOO, NULL, dumpit, 0);
Sure, however, > (from different call sites in the stack). > > - if (doit) > > - tab[msgindex].doit = doit; > > - if (dumpit) > > - tab[msgindex].dumpit = dumpit; > > Which is the reason for these if () tests. then we assign NULL, which is fine, no?