diff options
author | Gustavo A. R. Silva <gustavoars@kernel.org> | 2024-05-01 20:09:30 +0200 |
---|---|---|
committer | Luiz Augusto von Dentz <luiz.von.dentz@intel.com> | 2024-05-14 16:51:07 +0200 |
commit | d6bb8782b4b6ec14fb6e336f7f8c6a1ccce613b1 (patch) | |
tree | 07ef4349d6970fe8c7b0a63c35533987a45641d7 /net | |
parent | Bluetooth: qca: clean up defines (diff) | |
download | linux-d6bb8782b4b6ec14fb6e336f7f8c6a1ccce613b1.tar.xz linux-d6bb8782b4b6ec14fb6e336f7f8c6a1ccce613b1.zip |
Bluetooth: hci_conn: Use struct_size() in hci_le_big_create_sync()
Use struct_size() instead of the open-coded version. Similarly to
this other patch[1].
Link: https://lore.kernel.org/linux-hardening/ZiwwPmCvU25YzWek@neat/ [1]
Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
Reviewed-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
Diffstat (limited to 'net')
-rw-r--r-- | net/bluetooth/hci_conn.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c index 8375f7bd58ee..fc1aeb513b08 100644 --- a/net/bluetooth/hci_conn.c +++ b/net/bluetooth/hci_conn.c @@ -2132,7 +2132,7 @@ int hci_le_big_create_sync(struct hci_dev *hdev, struct hci_conn *hcon, memcpy(pdu->bis, bis, num_bis); return hci_send_cmd(hdev, HCI_OP_LE_BIG_CREATE_SYNC, - sizeof(*pdu) + num_bis, pdu); + struct_size(pdu, bis, num_bis), pdu); } static void create_big_complete(struct hci_dev *hdev, void *data, int err) |