From: Josh Oguin <josh.og...@oarcorp.com>

CodeSonar flagged this as a case where the return value from fat_sync()
was not used. Now it is used to return pass/fail to the caller.
---
 cpukit/libfs/src/dosfs/msdos_file.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/cpukit/libfs/src/dosfs/msdos_file.c 
b/cpukit/libfs/src/dosfs/msdos_file.c
index 7a09272..1d62886 100644
--- a/cpukit/libfs/src/dosfs/msdos_file.c
+++ b/cpukit/libfs/src/dosfs/msdos_file.c
@@ -244,5 +244,8 @@ msdos_file_sync(rtems_libio_t *iop)
     rc = fat_sync(&fs_info->fat);
 
     rtems_semaphore_release(fs_info->vol_sema);
-    return RC_OK;
+    if ( rc == 0 )
+      return RC_OK;
+
+    rtems_set_errno_and_return_minus_one(EIO);
 }
-- 
1.9.3

_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to