Simplication

git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@794 42af7a65-404d-4744-a932-0658087f49c3

1 files changed, 19 insertions, 8 deletions

diff --git a/nuttx/fs/fs_files.c b/nuttx/fs/fs_files.c
index fd8351fa5..245ff99a0 100644
--- a/nuttx/fs/fs_files.c
+++ b/nuttx/fs/fs_files.c
@@ -399,18 +399,29 @@ int files_allocate(FAR struct inode *inode, int oflags, off_t pos)
 int files_close(int filedes)
 {
  FAR struct filelist *list;
- int ret = -EBADF;
+ int                  ret;
+
+   /* Get the thread-specific file list */
 
   list = sched_getfiles();
-  if (list)
+  if (!list)
     {
-      if (filedes >=0 && filedes < CONFIG_NFILE_DESCRIPTORS)
-        {
-          _files_semtake(list);
-          ret = _files_close(&list->fl_files[filedes]);
-          _files_semgive(list);
-        }
+      return -EMFILE;
     }
+
+  /* If the file was properly opened, there should be an inode assigned */
+
+  if (filedes < 0 || filedes >= CONFIG_NFILE_DESCRIPTORS || !list->fl_files[filedes].f_inode)
+   {
+     return -EBADF;
+   }
+
+   /* Perform the protected close operation */
+
+   _files_semtake(list);
+   ret = _files_close(&list->fl_files[filedes]);
+   _files_semgive(list);
+   return ret;
 }
 
 /****************************************************************************