You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by xi...@apache.org on 2021/01/08 16:20:58 UTC

[incubator-nuttx] branch master updated: fs/inode/fs_files.c: fix DEBUGASSERT in romfs

This is an automated email from the ASF dual-hosted git repository.

xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git


The following commit(s) were added to refs/heads/master by this push:
     new e772be8  fs/inode/fs_files.c: fix DEBUGASSERT in romfs
e772be8 is described below

commit e772be8c8a019d4c9123bff9bc6037b4da15d946
Author: ligd <li...@xiaomi.com>
AuthorDate: Fri Jan 8 20:50:52 2021 +0800

    fs/inode/fs_files.c: fix DEBUGASSERT in romfs
    
    up_assert: Assertion failed at file:romfs/fs_romfs.c line: 643
    
    newp->f_priv should be NULL.
    
    Change-Id: I5f72b1329a8938ce39cf1ac5fbee561ca106f4f6
    Signed-off-by: ligd <li...@xiaomi.com>
---
 fs/inode/fs_files.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/inode/fs_files.c b/fs/inode/fs_files.c
index e28685f..31117cb 100644
--- a/fs/inode/fs_files.c
+++ b/fs/inode/fs_files.c
@@ -184,7 +184,7 @@ int file_dup2(FAR struct file *filep1, FAR struct file *filep2)
   temp.f_oflags = filep1->f_oflags;
   temp.f_pos    = filep1->f_pos;
   temp.f_inode  = inode;
-  temp.f_priv   = filep1->f_priv;
+  temp.f_priv   = NULL;
 
   /* Call the open method on the file, driver, mountpoint so that it
    * can maintain the correct open counts.