diff options
| author | Vadim Bendebury <vbendeb@chromium.org> | 2022-10-13 16:40:37 -0700 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-10-22 03:47:28 +0000 |
| commit | 9a1be550f92c15b0b0f2fb058493470346b48753 (patch) | |
| tree | 91fec30c8fbcaf0ebd828488e76f9eafbeb43931 | |
| parent | d0f7fdf670407af5ef6e08ee28019d44244af13a (diff) | |
| download | vboot-9a1be550f92c15b0b0f2fb058493470346b48753.tar.gz | |
cmd_update: avoid variable name aliasing
This is a noop change making reading the code a little easier, avoid
naming variable the same as the function name.
BRANCH=none
BUG=none
TEST='make futil' still succeeds
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Change-Id: I5b5d742aa5463160207f05f6c19c20754e538813
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3954469
Reviewed-by: Hung-Te Lin <hungte@chromium.org>
| -rw-r--r-- | futility/cmd_update.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/futility/cmd_update.c b/futility/cmd_update.c index b16a2b74..57c54029 100644 --- a/futility/cmd_update.c +++ b/futility/cmd_update.c @@ -160,7 +160,7 @@ static int do_update(int argc, char *argv[]) { struct updater_config *cfg; struct updater_config_arguments args = {0}; - int i, errorcnt = 0, do_update = 1; + int i, errorcnt = 0, update_needed = 1; int detect_servo = 0; const char *prepare_ctrl_name = NULL; char *servo_programmer = NULL; @@ -323,8 +323,8 @@ static int do_update(int argc, char *argv[]) prepare_servo_control(prepare_ctrl_name, 1); if (!errorcnt) - errorcnt += updater_setup_config(cfg, &args, &do_update); - if (!errorcnt && do_update) { + errorcnt += updater_setup_config(cfg, &args, &update_needed); + if (!errorcnt && update_needed) { int r; STATUS("Starting firmware updater.\n"); r = update_firmware(cfg); |