diff options
author | Hung-Te Lin <hungte@chromium.org> | 2018-09-17 17:47:25 +0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-09-18 10:31:02 -0700 |
commit | 0828bde7b1f0e932c2db0884cb6894c422672666 (patch) | |
tree | 972c9c7e32ba9a1686cafb09552d7d887a8f877d /futility | |
parent | b6cd64b799f94ef2fcc1f585939c4fe8c74a4d02 (diff) | |
download | vboot-0828bde7b1f0e932c2db0884cb6894c422672666.tar.gz |
futility: update: Fix update failure in factory mode due to wrong WP logic
In --mode=factory, we should fail if write protection is enabled.
Also added an unit test for the case.
BRANCH=None
BUG=b:115764295
TEST=make futil; tests/futility/run_test_scripts.sh $(pwd)/build/futility
Change-Id: I6c693e470a034554b1f4a3cb16cfbf974ae6f54b
Signed-off-by: Hung-Te Lin <hungte@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1226587
Reviewed-by: Marco Chen <marcochen@chromium.org>
Diffstat (limited to 'futility')
-rw-r--r-- | futility/cmd_update.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/futility/cmd_update.c b/futility/cmd_update.c index 7e57b97d..47a734aa 100644 --- a/futility/cmd_update.c +++ b/futility/cmd_update.c @@ -1894,7 +1894,7 @@ static int do_update(int argc, char *argv[]) } else if (strcmp(optarg, "factory") == 0 || strcmp(optarg, "factory_install") == 0) { cfg.try_update = 0; - if (!is_write_protection_enabled(&cfg)) { + if (is_write_protection_enabled(&cfg)) { errorcnt++; Error("Mode %s needs WP disabled.\n", optarg); |