diff options
| author | Vadim Sukhomlinov <sukhomlinov@google.com> | 2021-06-02 18:12:05 -0700 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2021-06-03 04:36:15 +0000 |
| commit | 47e7451821083879349df71d1c3a3d0f6c30979f (patch) | |
| tree | fc20d14fa528f0e8fe78e510f76cb3f77a67363d /chip/host/system.c | |
| parent | d06a3ef222f36d6c0e09cec7206d6b8ab6bebf80 (diff) | |
| download | chrome-ec-47e7451821083879349df71d1c3a3d0f6c30979f.tar.gz | |
host: fixing make runtests
Changes in compiler resulted in multiple warnings treated as errors,
failing host test builds.
Addresing warnings by checking return values.
BUG=none
TEST=make runtests
Change-Id: Idb2686370bf041791099b3e332ff25173338e994
Signed-off-by: Vadim Sukhomlinov <sukhomlinov@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2936000
Reviewed-by: Vadim Sukhomlinov <sukhomlinov@chromium.org>
Reviewed-by: Vadim Bendebury <vbendeb@chromium.org>
Commit-Queue: Vadim Sukhomlinov <sukhomlinov@chromium.org>
Commit-Queue: Vadim Bendebury <vbendeb@chromium.org>
Tested-by: Vadim Sukhomlinov <sukhomlinov@chromium.org>
Auto-Submit: Vadim Sukhomlinov <sukhomlinov@chromium.org>
Diffstat (limited to 'chip/host/system.c')
| -rw-r--r-- | chip/host/system.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/chip/host/system.c b/chip/host/system.c index b1654911ab..3c4225dda0 100644 --- a/chip/host/system.c +++ b/chip/host/system.c @@ -50,7 +50,7 @@ static void ramdata_get_persistent(void) return; } - fread(__ram_data, RAM_DATA_SIZE, 1, f); + ASSERT(fread(__ram_data, RAM_DATA_SIZE, 1, f) == 1); release_persistent_storage(f); @@ -78,7 +78,7 @@ static uint32_t get_image_copy(void) if (f == NULL) return SYSTEM_IMAGE_UNKNOWN; - fread(&ret, sizeof(ret), 1, f); + ASSERT(fread(&ret, sizeof(ret), 1, f) == 1); release_persistent_storage(f); remove_persistent_storage("image_copy"); @@ -102,7 +102,7 @@ static uint32_t load_reset_flags(void) if (f == NULL) return EC_RESET_FLAG_POWER_ON; - fread(&ret, sizeof(ret), 1, f); + ASSERT(fread(&ret, sizeof(ret), 1, f) == 1); release_persistent_storage(f); remove_persistent_storage("reset_flags"); @@ -125,7 +125,7 @@ static int load_time(timestamp_t *t) if (f == NULL) return 0; - fread(t, sizeof(*t), 1, f); + ASSERT(fread(t, sizeof(*t), 1, f) == 1); release_persistent_storage(f); remove_persistent_storage("time"); @@ -218,7 +218,7 @@ int system_set_scratchpad(uint32_t value) { FILE *f = get_persistent_storage("scratchpad", "w"); - fprintf(f, "%lu", value); + fprintf(f, "%u", value); release_persistent_storage(f); return EC_SUCCESS; |