Attention is currently required from: laforge, pespin.
fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/libosmocore/+/42406?usp=email )
Change subject: netdev: netdev_mnl_set_mtu(): drop unneeded variables ......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
The man documentation states the struct is passed, so imho looks like a good thing to set flags fiel […]
The man-page for `mnl_nlmsg_put_extra_header()`:
https://manpages.debian.org/experimental/libmnl-doc/mnl_nlmsg_put_extra_head...
states that "this function sets to zero the room that is required to put the extra header", so it does zero-initialize the extra header internally. I confirmed that by looking at the source code.
However, I am fine with setting both fields to 0 explicitly. I'll update the patch.