From 25e5f616fe155d8201f0cd373e78eadbc9a9410c Mon Sep 17 00:00:00 2001 From: Sebastian Huber Date: Fri, 12 Dec 2014 11:03:42 +0100 Subject: libcsupport: Do not use _Assert() for sync() Use a best-effort approach. --- cpukit/libcsupport/src/sync.c | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/cpukit/libcsupport/src/sync.c b/cpukit/libcsupport/src/sync.c index f27349cf5b..9f6e31ba0c 100644 --- a/cpukit/libcsupport/src/sync.c +++ b/cpukit/libcsupport/src/sync.c @@ -29,7 +29,6 @@ int fdatasync(int); /* still not always prototyped */ #include #include -#include /* XXX check standards -- Linux version appears to be void */ void _fwalk(struct _reent *, void *); @@ -38,19 +37,12 @@ void _fwalk(struct _reent *, void *); static void sync_wrapper(FILE *f) { int fn = fileno(f); - int rc; /* - * We are explicitly NOT checking the return values in non-debug builds - * as it does not matter if they succeed. We are just making a best - * faith attempt at both and trusting that we were passed a good - * FILE pointer. + * There is no way to report errors here. So this is a best-effort approach. */ - rc = fsync(fn); - _Assert( rc == 0 ); - - rc = fdatasync(fn); - _Assert( rc == 0 ); + (void) fsync(fn); + (void) fdatasync(fn); } /* iterate over all FILE *'s for this thread */ -- cgit v1.2.3