Bluetooth: L2CAP: Fix possible multiple reject send

[platform/kernel/linux-starfive.git] / net / ethtool / bitset.c

diff --git a/net/ethtool/bitset.c b/net/ethtool/bitset.c
index 883ed9b..0515d66 100644 (file)
--- a/net/ethtool/bitset.c
+++ b/net/ethtool/bitset.c
@@ -431,10 +431,8 @@ ethnl_update_bitset32_verbose(u32 *bitmap, unsigned int nbits,
                              ethnl_string_array_t names,
                              struct netlink_ext_ack *extack, bool *mod)
 {
-       u32 *orig_bitmap, *saved_bitmap = NULL;
        struct nlattr *bit_attr;
        bool no_mask;
-       bool dummy;
        int rem;
        int ret;
 
@@ -450,22 +448,8 @@ ethnl_update_bitset32_verbose(u32 *bitmap, unsigned int nbits,
        }
 
        no_mask = tb[ETHTOOL_A_BITSET_NOMASK];
-       if (no_mask) {
-               unsigned int nwords = DIV_ROUND_UP(nbits, 32);
-               unsigned int nbytes = nwords * sizeof(u32);
-
-               /* The bitmap size is only the size of the map part without
-                * its mask part.
-                */
-               saved_bitmap = kcalloc(nwords, sizeof(u32), GFP_KERNEL);
-               if (!saved_bitmap)
-                       return -ENOMEM;
-               memcpy(saved_bitmap, bitmap, nbytes);
-               ethnl_bitmap32_clear(bitmap, 0, nbits, &dummy);
-               orig_bitmap = saved_bitmap;
-       } else {
-               orig_bitmap = bitmap;
-       }
+       if (no_mask)
+               ethnl_bitmap32_clear(bitmap, 0, nbits, mod);
 
        nla_for_each_nested(bit_attr, tb[ETHTOOL_A_BITSET_BITS], rem) {
                bool old_val, new_val;
@@ -474,14 +458,13 @@ ethnl_update_bitset32_verbose(u32 *bitmap, unsigned int nbits,
                if (nla_type(bit_attr) != ETHTOOL_A_BITSET_BITS_BIT) {
                        NL_SET_ERR_MSG_ATTR(extack, bit_attr,
                                            "only ETHTOOL_A_BITSET_BITS_BIT allowed in ETHTOOL_A_BITSET_BITS");
-                       ret = -EINVAL;
-                       goto out;
+                       return -EINVAL;
                }
                ret = ethnl_parse_bit(&idx, &new_val, nbits, bit_attr, no_mask,
                                      names, extack);
                if (ret < 0)
-                       goto out;
-               old_val = orig_bitmap[idx / 32] & ((u32)1 << (idx % 32));
+                       return ret;
+               old_val = bitmap[idx / 32] & ((u32)1 << (idx % 32));
                if (new_val != old_val) {
                        if (new_val)
                                bitmap[idx / 32] |= ((u32)1 << (idx % 32));
@@ -491,10 +474,7 @@ ethnl_update_bitset32_verbose(u32 *bitmap, unsigned int nbits,
                }
        }
 
-       ret = 0;
-out:
-       kfree(saved_bitmap);
-       return ret;
+       return 0;
 }
 
 static int ethnl_compact_sanity_checks(unsigned int nbits,