netdev - Re: [PATCH iproute2-next 3/7] bridge: mdb: Add UDP destination port support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <d5df68ec-8edf-e12c-48a8-832304716d67@blackwall.org>
Date:   Thu, 23 Mar 2023 17:18:18 +0200
From:   Nikolay Aleksandrov <razor@...ckwall.org>
To:     Ido Schimmel <idosch@...dia.com>, netdev@...r.kernel.org
Cc:     dsahern@...il.com, stephen@...workplumber.org, petrm@...dia.com,
        mlxsw@...dia.com
Subject: Re: [PATCH iproute2-next 3/7] bridge: mdb: Add UDP destination port
 support

On 21/03/2023 15:01, Ido Schimmel wrote:
> In a similar fashion to VXLAN FDB entries, allow user space to program
> and view the UDP destination port of VXLAN MDB entries. Specifically,
> add support for the 'MDBE_ATTR_DST_PORT' and 'MDBA_MDB_EATTR_DST_PORT'
> attributes in request and response messages, respectively.
> 
> Use the keyword "dst_port" instead of "port" as the latter is already
> used to specify the net device associated with the MDB entry.
> 
> Example:
> 
>  # bridge mdb add dev vxlan0 port vxlan0 grp 239.1.1.1 permanent dst 198.51.100.1 dst_port 1234
> 
>  $ bridge -d -s mdb show
>  dev vxlan0 port vxlan0 grp 239.1.1.1 permanent filter_mode exclude proto static dst 198.51.100.1 dst_port 1234    0.00
> 
>  $ bridge -d -s -j -p mdb show
>  [ {
>          "mdb": [ {
>                  "index": 15,
>                  "dev": "vxlan0",
>                  "port": "vxlan0",
>                  "grp": "239.1.1.1",
>                  "state": "permanent",
>                  "filter_mode": "exclude",
>                  "protocol": "static",
>                  "flags": [ ],
>                  "dst": "198.51.100.1",
>                  "dst_port": 1234,
>                  "timer": "   0.00"
>              } ],
>          "router": {}
>      } ]
> 
> Signed-off-by: Ido Schimmel <idosch@...dia.com>
> ---
>  bridge/mdb.c      | 40 ++++++++++++++++++++++++++++++++++++++++
>  man/man8/bridge.8 | 10 +++++++++-
>  2 files changed, 49 insertions(+), 1 deletion(-)
> 

Reviewed-by: Nikolay Aleksandrov <razor@...ckwall.org>