From e22af784cd44cd407e268307ede01f6e1b621393 Mon Sep 17 00:00:00 2001 From: Joel Sherrill Date: Thu, 27 Nov 2014 09:30:12 -0600 Subject: sync.c: Add asserts to document and check assumptions --- cpukit/libcsupport/src/sync.c | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/cpukit/libcsupport/src/sync.c b/cpukit/libcsupport/src/sync.c index 653a177c9b..f27349cf5b 100644 --- a/cpukit/libcsupport/src/sync.c +++ b/cpukit/libcsupport/src/sync.c @@ -29,6 +29,7 @@ int fdatasync(int); /* still not always prototyped */ #include #include +#include /* XXX check standards -- Linux version appears to be void */ void _fwalk(struct _reent *, void *); @@ -37,14 +38,19 @@ 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 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. + * 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. */ - fsync(fn); - fdatasync(fn); + rc = fsync(fn); + _Assert( rc == 0 ); + + rc = fdatasync(fn); + _Assert( rc == 0 ); } /* iterate over all FILE *'s for this thread */ -- cgit v1.2.3