JFFS2文件系统open完偶尔flag不正确的BUG

发布于 2013-09-26 14:55:07 浏览：3212 订阅该版

在调用VFS的open后通过dfs_jffs2_open打开JFFS2文件系统，VFS为了兼容各种文件系统，将data指向`cyg_file * jffs2_file`。
`result = jffs2_opendir(mte, mte->root, name, jffs2_file);`时，会初始化一些变量
```c
file->f_type = CYG_FILE_TYPE_FILE;
file->f_ops = &jffs2_dirops;
file->f_offset = 0;
file->f_data = (CYG_ADDRWORD) ds.node;
file->f_xops = 0;
```
此时这里并没有初始化f_flag变量，也就是说jffs2_file结构其实是指向一块没有初始化过的内存区域，里面的值是随机的。

jffs2_opendir函数结束后，此时访问文件的flag才开始确定，代码如下：
```c
	/* regular file operations */
	mode = 0;
	if (oflag & DFS_O_RDONLY) mode |= JFFS2_O_RDONLY;
	if (oflag & DFS_O_WRONLY) mode |= JFFS2_O_WRONLY;
	if (oflag & DFS_O_RDWR)   mode |= JFFS2_O_RDWR;
	/* Opens the file, if it is existing. If not, a new file is created. */
	if (oflag & DFS_O_CREAT) mode |= JFFS2_O_CREAT;
	/* Creates a new file. If the file is existing, it is truncated and overwritten. */
	if (oflag & DFS_O_TRUNC) mode |= JFFS2_O_TRUNC;
	/* Creates a new file. The function fails if the file is already existing. */
	if (oflag & DFS_O_EXCL) mode |= JFFS2_O_EXCL;
//	if (oflag & DFS_O_APPEND) mode |= JFFS2_O_APPEND;
	rt_mutex_take(&jffs2_lock, RT_WAITING_FOREVER);
    result = jffs2_open(mte, 0, name, mode, jffs2_file);
```

由以上代码可以看出mode将在调用jffs2_open时，传递给jffs2_file结构体下的flag，代码如下：
```c
	// Initialise the file object
	file->f_flag |= mode & CYG_FILE_MODE_MASK;
	file->f_type = CYG_FILE_TYPE_FILE;
	file->f_ops = &jffs2_fileops;
	file->f_offset = (mode & O_APPEND) ? node->i_size : 0;
	file->f_data = (CYG_ADDRWORD) node;
	file->f_xops = 0;
```
此处`file->f_flag |= mode & CYG_FILE_MODE_MASK; `这里是或等于啊！就是说`file->f_flag`的值并不确定，又或上一个正确的值。
有木有啊！！！

8 个回答

bernard 2013-09-26

这家伙很懒，什么也没写！

haitao52198 2013-09-26

这家伙很懒，什么也没写！

bernard 2013-09-26

这家伙很懒，什么也没写！

prife 2013-09-28

这家伙很懒，什么也没写！

谢谢反馈，测试发现，这个地方确实是个问题。稍后把代码提交上。

jffs2_opendir里与jffs2_open里是两个地方，修改方法如下。

```
diff --git a/components/dfs/filesystems/jffs2/src/fs-ecos.c b/components/dfs/filesystems/jffs2/src/fs-ecos.c
index 5f3b5d6..814d2e3 100644
--- a/components/dfs/filesystems/jffs2/src/fs-ecos.c
+++ b/components/dfs/filesystems/jffs2/src/fs-ecos.c
@@ -747,7 +747,7 @@ int jffs2_open(cyg_mtab_entry * mte, cyg_dir dir, const char *name,
         }

// Initialise the file object
-       file->f_flag |= mode & CYG_FILE_MODE_MASK;
+       file->f_flag = mode & CYG_FILE_MODE_MASK;
        file->f_type = CYG_FILE_TYPE_FILE;
        file->f_ops = &jffs2_fileops;
        file->f_offset = (mode & O_APPEND) ? node->i_size : 0;
@@ -1065,6 +1065,7 @@ static int jffs2_opendir(cyg_mtab_entry * mte, cyg_dir dir, const char *name,
        // Initialize the file object, setting the f_ops field to a
        // special set of file ops.

+       file->f_flag = 0;
        file->f_type = CYG_FILE_TYPE_FILE;
        file->f_ops = &jffs2_dirops;
        file->f_offset = 0;

```