diff options
| author | Ziyang Xuan <william.xuanziyang@huawei.com> | 2023-08-07 04:32:06 +0200 |
|---|---|---|
| committer | Luiz Augusto von Dentz <luiz.von.dentz@intel.com> | 2023-08-11 20:56:54 +0200 |
| commit | 3cd43dd15f9dbfb67a60889992bf03b92370b202 (patch) | |
| tree | ff27a992ae228a5b82d5d608ad5c2a48a170c59f /net | |
| parent | Bluetooth: hci_sync: Avoid use-after-free in dbg for hci_add_adv_monitor() (diff) | |
| download | linux-3cd43dd15f9dbfb67a60889992bf03b92370b202.tar.xz linux-3cd43dd15f9dbfb67a60889992bf03b92370b202.zip | |
Bluetooth: Remove unnecessary NULL check before vfree()
Remove unnecessary NULL check which causes coccinelle warning:
net/bluetooth/coredump.c:104:2-7: WARNING: NULL check before some
freeing functions is not needed.
Signed-off-by: Ziyang Xuan <william.xuanziyang@huawei.com>
Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
Diffstat (limited to 'net')
| -rw-r--r-- | net/bluetooth/coredump.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/net/bluetooth/coredump.c b/net/bluetooth/coredump.c index d2d2624ec708..ec97a4bab1c9 100644 --- a/net/bluetooth/coredump.c +++ b/net/bluetooth/coredump.c @@ -100,8 +100,7 @@ void hci_devcd_reset(struct hci_dev *hdev) /* Call with hci_dev_lock only. */ static void hci_devcd_free(struct hci_dev *hdev) { - if (hdev->dump.head) - vfree(hdev->dump.head); + vfree(hdev->dump.head); hci_devcd_reset(hdev); } |