diff options
| author | Tim Wawrzynczak <twawrzynczak@chromium.org> | 2021-05-04 10:08:10 -0600 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2021-05-05 20:26:38 +0000 |
| commit | 54c3662a571a6c95bffbaf117a5c5c9255b18917 (patch) | |
| tree | 40f2e2074d9444f7f58805cd2b8cbf2efcf70fce /src/mainboard/google/cherry | |
| parent | cdae2d9cdf928ba27ae1afa3b91ed1207f043de1 (diff) | |
util/sconfig: Fix null pointer dereferences
Should use `name` instead of `field->name`, because `field is supposed
to be NULL at this point.
TEST=add new field from bits 29-64 to volteer, ensure sconfig prints an
error instead of segfaulting.
Change-Id: I933330494e0b10e8494a92e93d6beb58fbec0bc1
Found-by: Coverity CID 1452916
Signed-off-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/52888
Reviewed-by: Duncan Laurie
Reviewed-by: Furquan Shaikh <furquan@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/mainboard/google/cherry')
0 files changed, 0 insertions, 0 deletions