From 7ccba49a7c2372cdfff6e2947e417d4d4f5436c2 Mon Sep 17 00:00:00 2001 From: Julius Werner Date: Mon, 10 Nov 2014 13:16:25 -0800 Subject: [PATCH] cbfstool: Add missing break statement This patch adds a missing break statement in cbfstool's option parser. This should reduce the chance of your bootblock file name suddenly being a number after you swapped the order of some flags, and might save you an hour of debugging and growing insanity. Also removing a nearby empty line that looks out of place BRANCH=None BUG=None TEST=Manual Change-Id: I9beebdf29e4fc4aa645581146fdc61c659de72df Signed-off-by: Julius Werner Reviewed-on: https://chromium-review.googlesource.com/229973 Reviewed-by: Aaron Durbin Reviewed-by: Patrick Georgi Reviewed-by: Hung-Te Lin Reviewed-by: Stefan Reinauer --- util/cbfstool/cbfstool.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/util/cbfstool/cbfstool.c b/util/cbfstool/cbfstool.c index 71f07af416..d25bb5feb2 100644 --- a/util/cbfstool/cbfstool.c +++ b/util/cbfstool/cbfstool.c @@ -607,7 +607,6 @@ int main(int argc, char **argv) break; case 'l': param.loadaddress = strtoul(optarg, NULL, 0); - break; case 'e': param.entrypoint = strtoul(optarg, NULL, 0); @@ -620,6 +619,7 @@ int main(int argc, char **argv) if (tolower(suffix[0])=='m') { param.size *= 1024 * 1024; } + break; case 'B': param.bootblock = optarg; break;