On Tue, May 13, 2025 at 15:30:20 +0200, Michal Prívozník wrote:
> On 5/13/25 09:53, Peter Krempa wrote:
> > On Mon, May 12, 2025 at 15:37:13 +0200, Michal Privoznik via Devel wrote:
> >> From: Michal Privoznik <mpriv...@redhat.com>
> >>
> >> In virnetlink.c there are two sections: the first one when
> >> building WITH_LIBNL support, the other that provides stubs for
> >> functions declared in the corresponding header file when building
> >> without netlink support. But the stub implementation for
> >> virNetlinkBridgeVlanFilterSet() was missing.
> >>
> >> Signed-off-by: Michal Privoznik <mpriv...@redhat.com>
> >> ---
> >>  src/util/virnetlink.c | 11 +++++++++++
> >>  1 file changed, 11 insertions(+)
> >>
> >> diff --git a/src/util/virnetlink.c b/src/util/virnetlink.c
> >> index 206646d9d7..2555457cd2 100644
> >> --- a/src/util/virnetlink.c
> >> +++ b/src/util/virnetlink.c
> >> @@ -1344,6 +1344,17 @@ virNetlinkNewLink(const char *ifname G_GNUC_UNUSED,
> >>  }
> >>  
> >>  
> >> +int
> >> +virNetlinkBridgeVlanFilterSet(const char *ifname G_GNUC_UNUSED,
> >> +                              int cmd G_GNUC_UNUSED,
> >> +                              const unsigned short fflags G_GNUC_UNUSED,
> > 
> > s/fflags/flags/
> 
> No, this needs to be anything else but 'flags' otherwise our
> sc_flags_usage syntax-check rule will complain. unusedflags perhaps?

Ah ... d'oh. I mentioned it just because some static analysis tool
complained about argument names differing between definition and
declaration;

Anyways 'unusedflags' would likely be preferrable.

Reply via email to