WaveFile: Return GET_BUFFER_ERROR if wrong amount read

jepler · jepler · commit 82427612d15a · 2019-09-08T21:59:07.000-05:00
Closes: #2128
diff --git a/shared-module/audiocore/WaveFile.c b/shared-module/audiocore/WaveFile.c
@@ -216,7 +216,7 @@ audioio_get_buffer_result_t audioio_wavefile_get_buffer(audioio_wavefile_obj_t*
         } else {
             *buffer = self->buffer;
         }
-        if (f_read(&self->file->fp, *buffer, num_bytes_to_load, &length_read) != FR_OK) {
+        if (f_read(&self->file->fp, *buffer, num_bytes_to_load, &length_read) != FR_OK || length_read != num_bytes_to_load) {
             return GET_BUFFER_ERROR;
         }
         self->bytes_remaining -= length_read;

Original file line number	Diff line number	Diff line change
`@@ -216,7 +216,7 @@ audioio_get_buffer_result_t audioio_wavefile_get_buffer(audioio_wavefile_obj_t*`
`216`	`216`	`} else {`
`217`	`217`	`*buffer = self->buffer;`
`218`	`218`	`}`
`219`		`- if (f_read(&self->file->fp, *buffer, num_bytes_to_load, &length_read) != FR_OK) {`
	`219`	`+ if (f_read(&self->file->fp, *buffer, num_bytes_to_load, &length_read) != FR_OK \|\| length_read != num_bytes_to_load) {`
`220`	`220`	`return GET_BUFFER_ERROR;`
`221`	`221`	`}`
`222`	`222`	`self->bytes_remaining -= length_read;`