diff options
| author | Vadim Bendebury <vbendeb@chromium.org> | 2015-01-08 16:36:35 -0800 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2015-04-18 08:48:46 +0200 |
| commit | 1161473ecb65844ee24359b0b943a33a696eea24 (patch) | |
| tree | dd807b8c0e103daf43c6b3a0dada1fae8c08452f /util/cbfstool | |
| parent | 5e273a4577d0392156cb217848b14eab9daa31d6 (diff) | |
| download | coreboot-1161473ecb65844ee24359b0b943a33a696eea24.tar.xz | |
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>
Diffstat (limited to 'util/cbfstool')
| -rw-r--r-- | util/cbfstool/cbfstool.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/util/cbfstool/cbfstool.c b/util/cbfstool/cbfstool.c index 3ccc78610d..b60f199f55 100644 --- a/util/cbfstool/cbfstool.c +++ b/util/cbfstool/cbfstool.c @@ -758,6 +758,7 @@ int main(int argc, char **argv) case 'D': param.copyoffset = strtoul(optarg, NULL, 0); param.copyoffset_assigned = 1; + break; case 'a': param.alignment = strtoul(optarg, NULL, 0); break; |