diff options
| author | George Burgess IV <gbiv@google.com> | 2022-02-17 17:55:49 -0800 |
|---|---|---|
| committer | Chris Manton <cmanton@google.com> | 2022-02-18 20:03:55 +0000 |
| commit | 3dc0fb608bf31e35190eed06455837f1cac8154d (patch) | |
| tree | be08c07810983a600b948cdd713c9cb03b80c56c /framework/java/android/bluetooth/BluetoothLeAudioCodecConfig.java | |
| parent | 4ff2eae57f104c0dd20b83089cc1a88e23ec7b73 (diff) | |
l2c_csm: delete nullptr check
The code after this unconditionally dereferences `p_ccb->p_lcb`, and
`alarm_cancel` does nothing to somehow set this field. Hence, either
this ~entire switch block needs to be conditional on `p_ccb->p_lcb`, or
this NULL check isn't helpful.
Caught by the static analyzer:
> packages/modules/Bluetooth/system/stack/l2cap/l2c_csm.cc:889:27:
warning: Access to field 'pending_ecoc_conn_cnt' results in a
dereference of a null pointer (loaded from field 'p_lcb')
[clang-analyzer-core.NullDereference]
Tags: #stability
Bug: 206470603
Test: TreeHugger
Change-Id: I85394d3e9d0235f8b5ccc2bce5ca5283bc90f7ed
Diffstat (limited to 'framework/java/android/bluetooth/BluetoothLeAudioCodecConfig.java')
0 files changed, 0 insertions, 0 deletions