summaryrefslogtreecommitdiff
path: root/futility
diff options
context:
space:
mode:
authorHung-Te Lin <hungte@chromium.org>2018-11-30 23:29:27 +0800
committerchrome-bot <chrome-bot@chromium.org>2018-12-01 05:19:30 -0800
commit73d3c0e3d2d170e9bd34dacdbbdaa5d52860dea2 (patch)
tree60dd21424ff66a06b9fb0bafea5a674c35d64f63 /futility
parent979f8b906cfda7af57a4a526d352216eeafe1746 (diff)
downloadvboot-73d3c0e3d2d170e9bd34dacdbbdaa5d52860dea2.tar.gz
futility: updater: Skip applying white label for local build
For developers running a local build on white label models, currently the chromeos-firmwareupdate will always fail if VPD `whitelabel_tag` is set because the `keyset/` folder does not exist (which was created by signer bot). Developers in this case usually don't really care about which key to use and will be happy with the default (DEV signed) keys, also the key compatibility will be still checked later, so we can skip the white label patching if no keyset folder, which would allow developers getting same experience on WL and non-WL devices. BUG=b:120268135 TEST=TEST=make futil; tests/futility/run_test_scripts.sh $(pwd)/build/futility BRANCH=None Change-Id: I3992301ff4c406096e11e1ae8129f2f68b2319b5 Signed-off-by: Hung-Te Lin <hungte@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/1356688 Reviewed-by: C Shapiro <shapiroc@chromium.org>
Diffstat (limited to 'futility')
-rw-r--r--futility/updater.c9
1 files changed, 8 insertions, 1 deletions
diff --git a/futility/updater.c b/futility/updater.c
index 2908d994..dcd37bf6 100644
--- a/futility/updater.c
+++ b/futility/updater.c
@@ -1850,7 +1850,14 @@ static int updater_setup_archive(
cfg, arg, model->image, model->ec_image,
model->pd_image);
- if (model->is_white_label) {
+ if (model->is_white_label && !manifest->has_keyset) {
+ /*
+ * Developers running unsigned updaters (usually local build)
+ * won't be able match any white label tags.
+ */
+ WARN("No keysets found - this is probably a local build of "
+ "unsigned firmware updater. Skip applying white label.");
+ } else if (model->is_white_label) {
/*
* It is fine to fail in updater_apply_white_label for factory
* mode so we are not checking the return value; instead we