GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

cbfstool: add the missing 'break' 

The previous patch introduced a bug where the new added case statement
was missing the break. There was no problem testing, because an
unrelated parameter structure field was being modified as a result.

BRANCH=storm
BUG=none
TEST=compiles and runs

Change-Id: Iaeb328048f61ffd57057ebce47f2ac8e00fc5aac
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Original-Commit-Id: 27ecc130569e4252e4627052f617130a2017c645
Original-Change-Id: Ib3e6c4c2b5c37588c612b8ab2672f6845c1b4ecb
Original-Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Original-Reviewed-on: https://chromium-review.googlesource.com/239598
Original-Reviewed-by: Julius Werner <jwerner@chromium.org>
Reviewed-on: http://review.coreboot.org/9743
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Reviewed-by: Edward O'Callaghan <edward.ocallaghan@koparo.com>
This commit is contained in:
Vadim Bendebury 2015-01-08 16:36:35 -08:00 committed by Patrick Georgi
parent 5e273a4577
commit 1161473ecb
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
util/cbfstool/cbfstool.c
View File

@ -758,6 +758,7 @@ int main(int argc, char **argv)
case 'D': case 'D':
param.copyoffset = strtoul(optarg, NULL, 0); param.copyoffset = strtoul(optarg, NULL, 0);
param.copyoffset_assigned = 1; param.copyoffset_assigned = 1;
break;
case 'a': case 'a':
param.alignment = strtoul(optarg, NULL, 0); param.alignment = strtoul(optarg, NULL, 0);
break; break;