diff options
| author | Joel Sherrill <joel.sherrill@oarcorp.com> | 2014-11-27 09:30:12 -0600 |
|---|---|---|
| committer | Joel Sherrill <joel.sherrill@oarcorp.com> | 2014-11-27 10:06:23 -0600 |
| commit | e22af784cd44cd407e268307ede01f6e1b621393 (patch) | |
| tree | 137c7c7dc6575b45385afa716cf608bf36ca0ab8 | |
| parent | msdos_file.c: Reverse return codes per Gedare (diff) | |
| download | rtems-e22af784cd44cd407e268307ede01f6e1b621393.tar.bz2 | |
sync.c: Add asserts to document and check assumptions
| -rw-r--r-- | cpukit/libcsupport/src/sync.c | 16 |
1 files 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 <stdio.h> #include <rtems.h> +#include <rtems/score/assert.h> /* 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 */ |