futility: updater: Platform name check should be case insensitive
Some platforms (e.g., Zork) may decide to change platform name from
lower_case to CamelCase after early builds, and platform name check will
be an AU blocker in that case.
Considering there should be no devices having same name (and using
incompatible firmware), it should be fine to change the strncmp to
strncasecmp so the platform name check will be case insensitive.
BUG=b:156119908
TEST=make runtests
BRANCH=None
Change-Id: I5014c136976454dd0965a607e5a98bbe4543a41e
Signed-off-by: Hung-Te Lin <hungte@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2192537
Commit-Queue: Rob Barnes <robbarnes@google.com>
Tested-by: Rob Barnes <robbarnes@google.com>
Reviewed-by: Rob Barnes <robbarnes@google.com>
diff --git a/futility/updater.c b/futility/updater.c
index 8f2db61..336ec6d 100644
--- a/futility/updater.c
+++ b/futility/updater.c
@@ -654,7 +654,7 @@
}
len = from_dot - image_from->ro_version + 1;
VB2_DEBUG("Platform: %*.*s\n", len, len, image_from->ro_version);
- return strncmp(image_from->ro_version, image_to->ro_version, len);
+ return strncasecmp(image_from->ro_version, image_to->ro_version, len);
}
/*
