On Thursday 17 March 2016 18:04:56 Matthias Schiffer wrote:
And here is the new version of the PoC userspace tool. Build with:
gcc -o batnl batnl.c $$(pkg-config --cflags --libs libnl-1) -Wall
Thanks for the patches.
I've only looked at the test querier for a second and I am a little bit confused that you access things like attrs[BATADV_ATTR_ALGO_NAME] without checking if this is available or that the received message is valid (according to the nla_parse policy).
I know that this is only a PoC but wanted to mention it in case somebody wants to use it as reference for a batctl/alfred implementation.
I like that the netlink interface doesn't introduce complex structs at the moment (but I have to think a little bit about it). The unnamed enums freak me out a little bit (names would be nice so it is easier to write kernel-doc for it).
I just ran the build_test stuff through your branch so you get an early feedback about some common problems. See the attached mail (the unused symbols seem to be only symptoms of the build failures with old kernels < 3.12).
Kind regards, Sven