Emulation/switch-coreboot
1
0
Fork 0
mirror of https://github.com/fail0verflow/switch-coreboot.git synced 2025-05-04 01:39:18 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

UPSTREAM: util/romcc: Move access after NULL-check

Browse source 
Change-Id: I7f9c38fd6e75b32fe1ed8a60c7054f4dd1fcd5c0
Original-Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Original-Found-by: Coverity Scan #1129104
Original-Reviewed-on: https://review.coreboot.org/17884
Original-Tested-by: build bot (Jenkins)
Original-Reviewed-by: Martin Roth <martinroth@google.com>
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/421218
Reviewed-by: Patrick Georgi <pgeorgi@chromium.org>
This commit is contained in:
Patrick Georgi 2016-12-15 15:14:59 +01:00 • committed by chrome-bot
parent 7c655b3316
commit 7cd937c15c
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
util/romcc/romcc.c
View file

@ -6287,10 +6287,10 @@ static size_t field_reg_offset(struct compile_state *state,
internal_error(state, 0, "field_reg_offset only works on structures and unions");
}
size += reg_needed_padding(state, member, size);
if (!member || (member->field_ident != field)) {
error(state, 0, "member %s not present", field->name);
}
size += reg_needed_padding(state, member, size);
return size;
}