Attention is currently required from: laforge, pespin.

View Change

1 comment:

• Patchset:

  • Patch Set #1:

    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_header.3.en.html

    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.

To view, visit change 42406. To unsubscribe, or for help writing mail filters, visit settings.