diff options
| author | Tarun Tuli <taruntuli@google.com> | 2022-05-13 15:19:40 +0000 |
|---|---|---|
| committer | Arthur Heymans <arthur@aheymans.xyz> | 2022-05-14 14:09:58 +0000 |
| commit | ec3c41a6eecd20141431d9e1103b20395bc70594 (patch) | |
| tree | aed153891ba0bad082e85c4a73c576456ef6f779 /src/soc/intel/alderlake | |
| parent | 29bc79fddb62c30caa33474ac773ae6a6ec1c4f0 (diff) | |
soc/intel/alderlake: Fix Coverity CID 1488814
CID 1488814: Uninitialized variables (UNINIT)
Commit c66ea98 introduced an issue after static analysis on merge.
Because every APIC is associated with a CPU, this did not result in
any issues at runtime but should be fixed/cleaned up. Now, the path
name is initialized to null.
Fixes: Coverity CID 1488814, commit c66ea98
TEST=Built on brya
Change-Id: I0cfc8fd7a0c39e6610a9361630e3755293084f3d
Signed-off-by: Tarun Tuli <taruntuli@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/64336
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Elyes Haouas <ehaouas@noos.fr>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Diffstat (limited to 'src/soc/intel/alderlake')
| -rw-r--r-- | src/soc/intel/alderlake/acpi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/soc/intel/alderlake/acpi.c b/src/soc/intel/alderlake/acpi.c index b6c5699d78..e588f803c7 100644 --- a/src/soc/intel/alderlake/acpi.c +++ b/src/soc/intel/alderlake/acpi.c @@ -293,7 +293,7 @@ void soc_lpi_get_constraints(void *unused) break; case DEVICE_PATH_APIC: - char path[32]; + char path[32] = {0}; /* Lookup CPU id */ for (size_t i = 0; i < CONFIG_MAX_CPUS; i++) { |