diff options
author | Gustavo A. R. Silva <gustavo@embeddedor.com> | 2019-08-29 20:12:11 -0500 |
---|---|---|
committer | Marcel Holtmann <marcel@holtmann.org> | 2019-09-05 17:27:22 +0200 |
commit | 72bb169e024a20203e6044a81d5e41ae6ee0645b (patch) | |
tree | 8f59e5200d2f2e88496fa2690de4abaa8db2c299 /drivers/bluetooth | |
parent | f1300c0340872d7d02617f439433d70c6f47700f (diff) |
Bluetooth: mgmt: Use struct_size() helper
One of the more common cases of allocation size calculations is finding
the size of a structure that has a zero-sized array at the end, along
with memory for some number of elements for that array. For example:
struct mgmt_rp_get_connections {
...
struct mgmt_addr_info addr[0];
} __packed;
Make use of the struct_size() helper instead of an open-coded version
in order to avoid any potential type mistakes.
So, replace the following form:
sizeof(*rp) + (i * sizeof(struct mgmt_addr_info));
with:
struct_size(rp, addr, i)
Also, notice that, in this case, variable rp_len is not necessary,
hence it is removed.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Diffstat (limited to 'drivers/bluetooth')
0 files changed, 0 insertions, 0 deletions