diff options
| author | Tim Wawrzynczak <twawrzynczak@chromium.org> | 2019-08-07 20:54:01 -0600 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2019-08-09 01:28:04 +0000 |
| commit | 680027edf6dce0fca22b4e4b9525b1a88cd2ade9 (patch) | |
| tree | 3f2e8a163fd84935ea00b0584119638feb18b772 /src/northbridge | |
| parent | 98f43a1f757a89afa4e48d15ff48abdd7e62f46b (diff) | |
| download | coreboot-680027edf6dce0fca22b4e4b9525b1a88cd2ade9.tar.xz | |
soc/nvidia/tegra210: Fix potential NULL pointer dereference
Recent Coverity scan indicated potential NULL deference; if either
spi->dma_in or spi->dma_out are NULL, the fifo_error() check could
dereference a NULL pointer.
Also fixed what appears to be a logic bug for the spi->dma_out case,
where it was using the todo (count) from spi->dma_in.
Found-by: Coverity CID 1241838, 1241854
Change-Id: Icd1412f0956c0a4a75266d1873d5e9848aceee32
Signed-off-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/34787
Reviewed-by: Julius Werner <jwerner@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/northbridge')
0 files changed, 0 insertions, 0 deletions