From 9c80d9416306720acdaf5d8d79553b37ed45e089 Mon Sep 17 00:00:00 2001
From: patacongo <patacongo@42af7a65-404d-4744-a932-0658087f49c3>
Date: Tue, 4 Oct 2011 22:07:45 +0000
Subject: [PATCH] Fix check for open files when overwriting an existing file

git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@4019 42af7a65-404d-4744-a932-0658087f49c3
---
 fs/nxffs/nxffs_open.c | 20 ++++++++++++++++++--
 1 file changed, 18 insertions(+), 2 deletions(-)

diff --git a/fs/nxffs/nxffs_open.c b/fs/nxffs/nxffs_open.c
index 99998350b8..c00e2ea6df 100644
--- a/fs/nxffs/nxffs_open.c
+++ b/fs/nxffs/nxffs_open.c
@@ -423,11 +423,27 @@ static inline int nxffs_wropen(FAR struct nxffs_volume_s *volume,
   ret = nxffs_findinode(volume, name, &entry);
   if (ret == OK)
     {
-      /* It exists.  It would be an error if we are asked to create it
+      FAR struct nxffs_ofile_s *ofile;
+
+      /* It exists.  Is the file already open for reading? */
+
+      ofile = nxffs_findofile(volume, name);
+      if (ofile)
+        {
+          /* The file is already open.
+           * Limitation:  Files cannot be open both for reading and writing.
+           */
+
+          fdbg("File is open for reading\n");
+          ret = -ENOSYS;
+          goto errout_with_exclsem;
+        }
+
+      /* It would be an error if we are asked to create the file
        * exclusively.
        */
 
-      if ((oflags & (O_CREAT|O_EXCL)) == (O_CREAT|O_EXCL))
+      else if ((oflags & (O_CREAT|O_EXCL)) == (O_CREAT|O_EXCL))
         {
           fdbg("File exists, can't create O_EXCL\n");
           ret = -EEXIST;