[PW_SID:1070881] Bluetooth: btintel_pcie: validate RX buffer tags#3390
[PW_SID:1070881] Bluetooth: btintel_pcie: validate RX buffer tags#3390BluezTestBot wants to merge 2 commits intoworkflowfrom
Conversation
This patch adds workflow files for ci: [sync.yml] - The workflow file for scheduled work - Sync the repo with upstream repo and rebase the workflow branch - Review the patches in the patchwork and creates the PR if needed [ci.yml] - The workflow file for CI tasks - Run CI tests when PR is created Signed-off-by: Tedd Ho-Jeong An <tedd.an@intel.com>
The RX completion path trusts urbd1->frbd_tag as a direct index into the fixed rxq->bufs[] table. The table only has rxq->count entries, while frbd_tag is a wider firmware-provided field and is not range-checked before use. Validate the completion tag before indexing the RX buffer table so a malformed completion cannot walk past the descriptor-backed buffer array. Signed-off-by: Pengpeng Hou <pengpeng@iscas.ac.cn>
|
CheckPatch |
|
GitLint |
|
SubjectPrefix |
|
BuildKernel |
|
CheckAllWarning |
|
CheckSparse |
|
BuildKernel32 |
|
TestRunnerSetup |
|
TestRunner_l2cap-tester |
|
TestRunner_iso-tester |
|
TestRunner_bnep-tester |
|
TestRunner_mgmt-tester |
|
TestRunner_rfcomm-tester |
|
TestRunner_sco-tester |
|
TestRunner_ioctl-tester |
|
TestRunner_mesh-tester |
|
TestRunner_smp-tester |
|
TestRunner_userchan-tester |
|
IncrementalBuild |
9f31ee4 to
19dcf1f
Compare
9dddb3f to
c6e7062
Compare
f07ea67 to
9a108c6
Compare
The RX completion path trusts urbd1->frbd_tag as a direct index into the
fixed rxq->bufs[] table. The table only has rxq->count entries, while
frbd_tag is a wider firmware-provided field and is not range-checked
before use.
Validate the completion tag before indexing the RX buffer table so a
malformed completion cannot walk past the descriptor-backed buffer
array.
Signed-off-by: Pengpeng Hou pengpeng@iscas.ac.cn
drivers/bluetooth/btintel_pcie.c | 6 ++++++
1 file changed, 6 insertions(+)