]> git.ipfire.org Git - thirdparty/iptables.git/commit
ebtables: Include 'bitmask' value when comparing rules
authorPhil Sutter <phil@nwl.cc>
Wed, 12 Jun 2024 12:22:54 +0000 (14:22 +0200)
committerPhil Sutter <phil@nwl.cc>
Wed, 12 Jun 2024 15:14:27 +0000 (17:14 +0200)
commit9d0f4d239ab9d530120d3bc885d7ea41161ddf0b
tree3c553c29023fde0ff181e7eba345839a5b46030f
parent0234117d24609070f08ef36a11795c3c8e4c19bf
ebtables: Include 'bitmask' value when comparing rules

The former FIXME comment pointed at the fact that struct ebt_entry does
not have a 'flags' field (unlike struct ipt_ip). In fact, ebt_entry's
equivalent is 'bitmask' field. Comparing that instead is the right
thing to do, even though it does not seem to make a difference in
practice: No rule options alter just the bitmask value, nor is it
possible to fill an associated field with default values (e.g. all-zero
MAC and mask).

Since the situation described above might change and there is a slight
performance improvement in some cases (e.g. comparing rules differing
only by specified/omitted source/dest MAC address), add the check
anyway.

Suggested-by: Michael Estner <michaelestner@web.de>
Signed-off-by: Phil Sutter <phil@nwl.cc>
iptables/nft-bridge.c