diff options
| author | David Oberhollenzer <david.oberhollenzer@sigma-star.at> | 2021-11-24 10:58:15 +0100 |
|---|---|---|
| committer | David Oberhollenzer <david.oberhollenzer@sigma-star.at> | 2021-11-24 12:33:57 +0100 |
| commit | 1d0651367a3b900ebe99356ddcc1fc19fc0800d6 (patch) | |
| tree | d47ace819ba5142ce7530cc3bb4547eb183b994e /lib/fstream/win32 | |
| parent | e5540b5a3a545ed4e3f3df97be131ec81a155032 (diff) | |
Fix: libfstream: Correctly handle FlushFileBuffers resturn status
The Windows port uses FlushFileBuffers in libfstream for the
implmentation of the file flush method. Unlike other winapi functions,
this function returns a boolean and not an error code.
Previously, the error code path was executed on success, printing a
rather confusing error message, that this file already exists.
Signed-off-by: David Oberhollenzer <david.oberhollenzer@sigma-star.at>
Diffstat (limited to 'lib/fstream/win32')
| -rw-r--r-- | lib/fstream/win32/ostream.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/fstream/win32/ostream.c b/lib/fstream/win32/ostream.c index e593f7e..7b5bd3e 100644 --- a/lib/fstream/win32/ostream.c +++ b/lib/fstream/win32/ostream.c @@ -35,7 +35,7 @@ static int w32_append(HANDLE hnd, const char *filename, static int w32_flush(HANDLE hnd, const char *filename) { - if (FlushFileBuffers(hnd) != 0) { + if (!FlushFileBuffers(hnd)) { w32_perror(filename); return -1; } |