diff options
author | Jeffrey Hugo <jhugo@codeaurora.org> | 2021-03-10 22:30:55 +0100 |
---|---|---|
committer | Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> | 2021-04-08 08:27:38 +0200 |
commit | ec32332df7645e0ba463a08d483fe97665167071 (patch) | |
tree | b4e595d1681ab4560fc99b3dfb345bed0d88707e /scripts/headers_install.sh | |
parent | bus: mhi: pci_generic: Add FIREHOSE channels (diff) | |
download | linux-ec32332df7645e0ba463a08d483fe97665167071.tar.xz linux-ec32332df7645e0ba463a08d483fe97665167071.zip |
bus: mhi: core: Sanity check values from remote device before use
When parsing the structures in the shared memory, there are values which
come from the remote device. For example, a transfer completion event
will have a pointer to the tre in the relevant channel's transfer ring.
As another example, event ring elements may specify a channel in which
the event occurred, however the specified channel value may not be valid
as no channel is defined at that index even though the index may be less
than the maximum allowed index. Such values should be considered to be
untrusted, and validated before use. If we blindly use such values, we
may access invalid data or crash if the values are corrupted.
If validation fails, drop the relevant event.
Signed-off-by: Jeffrey Hugo <jhugo@codeaurora.org>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Reviewed-by: Hemant Kumar <hemantk@codeaurora.org>
Link: https://lore.kernel.org/r/1615411855-15053-1-git-send-email-jhugo@codeaurora.org
Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Diffstat (limited to 'scripts/headers_install.sh')
0 files changed, 0 insertions, 0 deletions