diff options
author | Nico Huber <nico.h@gmx.de> | 2022-08-05 13:16:31 +0200 |
---|---|---|
committer | Felix Held <felix-coreboot@felixheld.de> | 2022-08-17 19:09:05 +0000 |
commit | b511804169038ba3bff5b8b9b81978234cc9b58b (patch) | |
tree | aae1b215a1512e275afabfd5d4bf8afda69e7ba0 /src/drivers/sil | |
parent | bba97354b05cc8770d6d913bfe5777b46450bbb8 (diff) |
pciexp_device: Drop quirk handling in pciexp_get_ext_cap_offset()
Keeping these checks in generic code seems rather dangerous.
In theory, it could lead to endless loops even for compliant
devices, if we accidentally detect arbitrary register contents
as capability and use them as a pointer to another one. Not
to forget that the register reads can have side effects.
All users of this `cafe` have been converted to use
pciexp_find_ext_vendor_cap().
Change-Id: I70d21534e04282a4156572a290b83c46be085e0c
Signed-off-by: Nico Huber <nico.h@gmx.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/66456
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/drivers/sil')
0 files changed, 0 insertions, 0 deletions