diff options
| author | Simon Glass <sjg@chromium.org> | 2013-08-15 21:32:08 -0600 |
|---|---|---|
| committer | ChromeBot <chrome-bot@google.com> | 2013-08-25 16:57:28 -0700 |
| commit | 3401fdcd4125beea1a8cb1cc59ee27df89d4d88a (patch) | |
| tree | 5fd2c191d0f98ab276bf11f87c88d3c7fbfe222a /host/arch/arm/lib | |
| parent | 47779880b28f2c549dd3349d8f28d68a0f784eb4 (diff) | |
| download | vboot-3401fdcd4125beea1a8cb1cc59ee27df89d4d88a.tar.gz | |
Correct some minor compiler warnings
A few places in the code through up warnings when building with strict
compiler flags. Correct these.
BUG=chrome-os-partner:21115
BRANCH=pit
TEST=manual
Build with:
FEATURES=test emerge-peach_pit vboot_reference
and see that iot now succeeds. Warnings include:
host/arch/arm/lib/crossystem_arch.c: In function 'ReadFdtValue':
host/arch/arm/lib/crossystem_arch.c:93:8: error: ignoring return value of 'fread', declared with attribute warn_unused_result [-Werror=unused-result]
Change-Id: I765723636e5f8979b794925c7b610081b2849026
Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/66174
Diffstat (limited to 'host/arch/arm/lib')
| -rw-r--r-- | host/arch/arm/lib/crossystem_arch.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/host/arch/arm/lib/crossystem_arch.c b/host/arch/arm/lib/crossystem_arch.c index 033632e4..99b86897 100644 --- a/host/arch/arm/lib/crossystem_arch.c +++ b/host/arch/arm/lib/crossystem_arch.c @@ -90,7 +90,10 @@ static int ReadFdtValue(const char *property, int *value) { return E_FILEOP; } - fread(&data, 1, sizeof(data), file); + if (fread(&data, 1, sizeof(data), file) != sizeof(data)) { + fprintf(stderr, "Unable to read FDT property %s\n", property); + return E_FILEOP; + } fclose(file); if (value) |