diff options
author | Jacob Garber <jgarber1@ualberta.ca> | 2019-04-10 11:51:58 -0600 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2019-04-11 11:27:11 +0000 |
commit | 94d61ecab035fbef7e93db8cfbdbabd7c41a264d (patch) | |
tree | bdce3cd388d73c1398efaf8d6f223cc033cf7702 /src/northbridge/intel/i945/ram_calc.c | |
parent | e05fe3166e69f8b23eb5a448052cea79e9e6a38a (diff) |
util/inteltool: Swap conditions to prevent uninit reads
Both values in each array are only initialized if
`two_channels` is true, so we need to check that first.
Found-by: Coverity Scan #1370{584,585,588,589,590-596,600}
Signed-off-by: Jacob Garber <jgarber1@ualberta.ca>
Change-Id: I592bc6ae00f834f74a61668d7a3919014ec635f3
Reviewed-on: https://review.coreboot.org/c/coreboot/+/32269
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-by: Patrick Rudolph <siro@das-labor.org>
Diffstat (limited to 'src/northbridge/intel/i945/ram_calc.c')
0 files changed, 0 insertions, 0 deletions