net: ieee802154: hwsim: Ensure frame checksum are valid
authorMiquel Raynal <miquel.raynal@bootlin.com>
Tue, 25 Jan 2022 12:25:38 +0000 (13:25 +0100)
committerStefan Schmidt <stefan@datenfreihafen.org>
Fri, 28 Jan 2022 10:23:08 +0000 (11:23 +0100)
There is no point in accepting frames with a wrong or missing checksum,
at least not outside of a promiscuous setting. Set the right flag by
default in the hwsim driver to ensure checksums are not ignored.

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Acked-by: Alexander Aring <aahringo@redhat.com>
Link: https://lore.kernel.org/r/20220125122540.855604-2-miquel.raynal@bootlin.com
Signed-off-by: Stefan Schmidt <stefan@datenfreihafen.org>
drivers/net/ieee802154/mac802154_hwsim.c

index 8caa61ec718f503d0731262446a88805ff5e06f3..62ced7a30d92892345edb6baecb36d500f3a4f09 100644 (file)
@@ -790,7 +790,7 @@ static int hwsim_add_one(struct genl_info *info, struct device *dev,
        phy->idx = idx;
        INIT_LIST_HEAD(&phy->edges);
 
-       hw->flags = IEEE802154_HW_PROMISCUOUS;
+       hw->flags = IEEE802154_HW_PROMISCUOUS | IEEE802154_HW_RX_DROP_BAD_CKSUM;
        hw->parent = dev;
 
        err = ieee802154_register_hw(hw);