diff options
author | Chih-Hung Hsieh <chh@google.com> | 2017-10-18 16:28:14 -0700 |
---|---|---|
committer | Chih-Hung Hsieh <chh@google.com> | 2017-10-20 15:46:36 -0700 |
commit | 373d3c7257fa815d0b9ee8f16874470a6002042e (patch) | |
tree | 2488faeb9a5a2dfd9f76054bd4ba001b4806ffb3 /verity | |
parent | d7bf23ae7b93ec93b8b7c2ce98eaa2ea94f387aa (diff) | |
download | extras-373d3c7257fa815d0b9ee8f16874470a6002042e.tar.gz |
Use -Werror in system/extras
* Remove unused variables.
* Fix redefined macro warnings.
* Fix sign-compare warnings.
* Fix 'return false' in main().
* Keep existing warnings to fix later.
Bug: 66996870
Test: build with WITH_TIDY=1
Change-Id: Ib92ef5297693595fb84ed4f8e61665bda4cee312
Diffstat (limited to 'verity')
-rw-r--r-- | verity/Android.mk | 2 | ||||
-rw-r--r-- | verity/verify_boot_signature.c | 1 |
2 files changed, 2 insertions, 1 deletions
diff --git a/verity/Android.mk b/verity/Android.mk index e379a475..7ddf75f6 100644 --- a/verity/Android.mk +++ b/verity/Android.mk @@ -5,6 +5,7 @@ ifeq ($(HOST_OS),linux) include $(CLEAR_VARS) LOCAL_MODULE := verify_boot_signature LOCAL_SRC_FILES := verify_boot_signature.c +LOCAL_CFLAGS := -Wall -Werror LOCAL_MODULE_CLASS := EXECUTABLES LOCAL_MODULE_TAGS := optional LOCAL_SHARED_LIBRARIES := libcrypto @@ -16,6 +17,7 @@ endif # HOST_OS == linux include $(CLEAR_VARS) LOCAL_MODULE := generate_verity_key LOCAL_SRC_FILES := generate_verity_key.c +LOCAL_CFLAGS := -Wall -Werror LOCAL_MODULE_CLASS := EXECUTABLES LOCAL_MODULE_TAGS := optional LOCAL_SHARED_LIBRARIES := libcrypto_utils libcrypto diff --git a/verity/verify_boot_signature.c b/verity/verify_boot_signature.c index 36760ad7..e1c53d81 100644 --- a/verity/verify_boot_signature.c +++ b/verity/verify_boot_signature.c @@ -92,7 +92,6 @@ static uint64_t page_align(uint64_t n, uint64_t page_size) */ static int get_signature_offset(int fd, off64_t *offset) { - int i; struct boot_img_hdr hdr; if (!offset) { |