| Message ID | 20220307155338.140860-1-colin.i.king@gmail.com |
|---|---|
| State | Accepted |
| Commit | 7ffc9daa9ddbb48b2ed9de05657075c90397787b |
| Headers | show |
| Series | Bluetooth: mgmt: remove redundant assignment to variable cur_len | expand |
Hi Colin, > Variable cur_len is being ininitialized with a value in the start of > a for-loop but this is never read, it is being re-assigned a new value > on the first statement in the for-loop. The initialization is redundant > and can be removed. > > Cleans up clang scan build warning: > net/bluetooth/mgmt.c:7958:14: warning: Although the value stored to 'cur_len' > is used in the enclosing expression, the value is never actually read > from 'cur_len' [deadcode.DeadStores] > > Signed-off-by: Colin Ian King <colin.i.king@gmail.com> > --- > net/bluetooth/mgmt.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) patch has been applied to bluetooth-next tree. Regards Marcel
diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c index 8101a6a31841..e0137bc1080e 100644 --- a/net/bluetooth/mgmt.c +++ b/net/bluetooth/mgmt.c @@ -7955,7 +7955,7 @@ static bool tlv_data_is_valid(struct hci_dev *hdev, u32 adv_flags, u8 *data, return false; /* Make sure that the data is correctly formatted. */ - for (i = 0, cur_len = 0; i < len; i += (cur_len + 1)) { + for (i = 0; i < len; i += (cur_len + 1)) { cur_len = data[i]; if (!cur_len)
Variable cur_len is being ininitialized with a value in the start of a for-loop but this is never read, it is being re-assigned a new value on the first statement in the for-loop. The initialization is redundant and can be removed. Cleans up clang scan build warning: net/bluetooth/mgmt.c:7958:14: warning: Although the value stored to 'cur_len' is used in the enclosing expression, the value is never actually read from 'cur_len' [deadcode.DeadStores] Signed-off-by: Colin Ian King <colin.i.king@gmail.com> --- net/bluetooth/mgmt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)