On Wed, Jul 16, 2025 at 04:39:54PM +0200, Pablo Neira Ayuso wrote: > On Wed, Jul 16, 2025 at 12:01:55PM +0200, Phil Sutter wrote: > > Hi, > > > > On Wed, Jul 16, 2025 at 11:44:59AM +0200, Florian Westphal wrote: > > > Phil Sutter <phil@xxxxxx> wrote: > > > > #include <string.h> > > > > #include <stdlib.h> > > > > +#include <libmnl/libmnl.h> > > > > > > Why is this include needed? > > > > Because of: > > > > | In file included from udata.c:9: > > | ../include/utils.h:88:40: warning: 'struct nlattr' declared inside parameter list will not be visible outside of this definition or declaration > > | 88 | const char *mnl_attr_get_ifname(struct nlattr *attr); > > | | ^~~~~~ > > I think this helper belongs to src/netlink.c Not a deal breaker, Florian's proposal is also fine.
