shithub: aubio

Info  •  Files  •  Log  •  Branches

[sink_wavwrite] call fflush in open

This ensures the file header was actually written correctly, and fails
otherwise, for instance when the target disk-system is full.

--- a/src/io/sink_wavwrite.c

+++ b/src/io/sink_wavwrite.c

@@ -211,9 +211,11 @@

   written += fwrite(write_little_endian(0, buf, 4), 4, 1, s->fid);

   // fwrite(*, *, 1, s->fid) was called 13 times, check success

-  if (written != 13) {

-    AUBIO_STRERR("sink_wavwrite: writing header to %s failed, expected %d"

-        " write but got only %d (%s)\n", s->path, 13, written, errorstr);

+  if (written != 13 || fflush(s->fid)) {

+    AUBIO_STRERR("sink_wavwrite: writing header to %s failed"

+        " (wrote %d/%d, %s)\n", s->path, written, 13, errorstr);

+    fclose(s->fid);

+    s->fid = NULL;

     return AUBIO_FAIL;

   }