diff options
author | Mike Leach <mike.leach@linaro.org> | 2021-09-20 14:55:10 +0100 |
---|---|---|
committer | Mike Leach <mike.leach@linaro.org> | 2021-10-10 10:04:22 +0100 |
commit | 51aef80eddb7df9b606d3b99f7db852e4ce89e35 (patch) | |
tree | 3a2ec5526943b632b0ffc8b772088bf45931ae6d | |
parent | 1be557e437a08446177b78538f99dd2a3b40c0a7 (diff) |
build: tests: Fix build warnings in mem_buff_demo test
Signed-off-by: Mike Leach <mike.leach@linaro.org>
-rw-r--r-- | decoder/tests/source/mem_buff_demo.cpp | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/decoder/tests/source/mem_buff_demo.cpp b/decoder/tests/source/mem_buff_demo.cpp index cacc227e941f..052870fd3e56 100644 --- a/decoder/tests/source/mem_buff_demo.cpp +++ b/decoder/tests/source/mem_buff_demo.cpp @@ -126,6 +126,7 @@ static int initDataBuffers() FILE *fp; std::string filename; long size; + size_t bytes_read; /* the file names to create the data buffers */ #ifdef _WIN32 @@ -158,8 +159,10 @@ static int initDataBuffers() return OCSD_ERR_MEM; } rewind(fp); - fread(input_trace_data, 1, input_trace_data_size, fp); + bytes_read = fread(input_trace_data, 1, input_trace_data_size, fp); fclose(fp); + if (bytes_read < (size_t)input_trace_data_size) + return OCSD_ERR_FILE_ERROR; /* load up a memory image */ filename = default_base_snapshot_path; @@ -178,8 +181,10 @@ static int initDataBuffers() return OCSD_ERR_MEM; } rewind(fp); - fread(program_image_buffer, 1, program_image_size, fp); + bytes_read = fread(program_image_buffer, 1, program_image_size, fp); fclose(fp); + if (bytes_read < (size_t)program_image_size) + return OCSD_ERR_FILE_ERROR; program_image_address = mem_dump_address; return OCSD_OK; } |