Message ID | 20211216044839.v9.3.If37d23d1dd8b765d8a6c8eca71ac1c29df591565@changeid (mailing list archive) |
---|---|
State | Superseded |
Headers | show |
Series | [v9,1/3] bluetooth: msft: Handle MSFT Monitor Device Event | expand |
Context | Check | Description |
---|---|---|
tedd_an/checkpatch | success | Checkpatch PASS |
tedd_an/gitlint | success | Gitlint PASS |
Hi Manish, On Thu, Dec 16, 2021 at 4:50 AM Manish Mandlik <mmandlik@google.com> wrote: > > Use correct sizeof() parameter while allocating skb. > > Signed-off-by: Manish Mandlik <mmandlik@google.com> > --- > > (no changes since v8) > > Changes in v8: > - New patch in the series. > > net/bluetooth/mgmt.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c > index c65247b5896c..5fd29bd399f1 100644 > --- a/net/bluetooth/mgmt.c > +++ b/net/bluetooth/mgmt.c > @@ -9709,7 +9709,7 @@ void mgmt_device_found(struct hci_dev *hdev, bdaddr_t *bdaddr, u8 link_type, > > /* Allocate skb. The 5 extra bytes are for the potential CoD field */ > skb = mgmt_alloc_skb(hdev, MGMT_EV_DEVICE_FOUND, > - sizeof(ev) + eir_len + scan_rsp_len + 5); > + sizeof(*ev) + eir_len + scan_rsp_len + 5); > if (!skb) > return; > > -- > 2.34.1.173.g76aa8bc2d0-goog There is already a patch addressing this: https://patchwork.kernel.org/project/bluetooth/patch/20211213212650.2067066-1-luiz.dentz@gmail.com/ Please use that instead and if that works for you reply adding Tested-by.
diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c index c65247b5896c..5fd29bd399f1 100644 --- a/net/bluetooth/mgmt.c +++ b/net/bluetooth/mgmt.c @@ -9709,7 +9709,7 @@ void mgmt_device_found(struct hci_dev *hdev, bdaddr_t *bdaddr, u8 link_type, /* Allocate skb. The 5 extra bytes are for the potential CoD field */ skb = mgmt_alloc_skb(hdev, MGMT_EV_DEVICE_FOUND, - sizeof(ev) + eir_len + scan_rsp_len + 5); + sizeof(*ev) + eir_len + scan_rsp_len + 5); if (!skb) return;
Use correct sizeof() parameter while allocating skb. Signed-off-by: Manish Mandlik <mmandlik@google.com> --- (no changes since v8) Changes in v8: - New patch in the series. net/bluetooth/mgmt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)