文件系统目录操作的严重问题（齐心协力已解决）

发布于 2012-10-30 21:36:28 浏览：6772 订阅该版

测试代码如下：

```c
	{
		int fd;
		DIR* dir;
		struct dirent* dirp;
		off_t off[6];
		int i;
		mkdir("/testdir",0777);
		fd = open("/testdir/file1",O_CREAT|O_RDWR,0777);
		close(fd);
		fd = open("/testdir/file2",O_CREAT|O_RDWR,0777);
		close(fd);
		fd = open("/testdir/file3",O_CREAT|O_RDWR,0777);
		close(fd);
		fd = open("/testdir/file4",O_CREAT|O_RDWR,0777);
		close(fd);
		fd = open("/testdir/file5",O_CREAT|O_RDWR,0777);
		close(fd);
		fd = open("/testdir/file6",O_CREAT|O_RDWR,0777);
		close(fd);

dir = opendir("/testdir");
		for(i=0;i<6;i++)
		{
			off* = telldir(dir); 
			dirp = readdir(dir);
			if(dirp)
				rt_kprintf("#%d NAME:%s
",i,dirp->d_name);
			else break;
		}
		for(i=0;i<6;i++)
		{
			seekdir(dir,off*);
			dirp = readdir(dir);
			if(dirp)
				rt_kprintf("#%d NAME:%s
",i,dirp->d_name);
			else break;
		}

rt_kprintf("unlink file2
");
		unlink("/testdir/file2");
		rewinddir(dir);
		for(i=0;i<6;i++)
		{
			off* = telldir(dir); 
			dirp = readdir(dir);
			if(dirp)
				rt_kprintf("#%d NAME:%s
",i,dirp->d_name);
			else break;
		}
		for(i=0;i<6;i++)
		{
			seekdir(dir,off*);
			dirp = readdir(dir);
			if(dirp)
				rt_kprintf("#%d NAME:%s
",i,dirp->d_name);
			else break;
		}

rt_kprintf("unlink file4
");
		unlink("/testdir/file4");
		rewinddir(dir);
		for(i=0;i<6;i++)
		{
			off* = telldir(dir); 
			dirp = readdir(dir);
			if(dirp)
				rt_kprintf("#%d NAME:%s
",i,dirp->d_name);
			else break;
		}
		for(i=0;i<6;i++)
		{
			seekdir(dir,off*);
			dirp = readdir(dir);
			if(dirp)
				rt_kprintf("#%d NAME:%s
",i,dirp->d_name);
			else break;
		}

unlink("/testdir/file1");
		unlink("/testdir/file3");
		unlink("/testdir/file5");
		unlink("/testdir/file6");

closedir(dir);
	}
```

运行结果：
```
#0 NAME:FILE1
#1 NAME:FILE2
#2 NAME:FILE3
#3 NAME:FILE4
#4 NAME:FILE5
#5 NAME:FILE6
#0 NAME:FILE1
#1 NAME:FILE2
#2 NAME:FILE3
#3 NAME:FILE4
#4 NAME:FILE5
#5 NAME:FILE6
unlink file2
#0 NAME:FILE1
#1 NAME:FILE3
#2 NAME:FILE4
#3 NAME:FILE5
#4 NAME:FILE6
#0 NAME:FILE1
#1 NAME:FILE1
#2 NAME:FILE1
#3 NAME:FILE3
#4 NAME:FILE3
#5 NAME:FILE4
unlink file4
#0 NAME:FILE1
#1 NAME:FILE3
#2 NAME:FILE5
#3 NAME:FILE6
#0 NAME:FILE1
#1 NAME:FILE1
#2 NAME:FILE1
#3 NAME:FILE3
#4 NAME:FILE3
#5 NAME:FILE5
```

从运行结果可以看到删除文件对目录操作的影响。
![psb1.jpg](https://oss-club.rt-thread.org/uploads/3948_d547e0db1dce91018dcbbd53df1f5ef7.jpg)

![psb2.jpg](https://oss-club.rt-thread.org/uploads/3948_c88a66d1050016442ef1231861255a08.jpg)

![psb3.jpg](https://oss-club.rt-thread.org/uploads/3948_f2d8086cb5a535adf8a8b8a5a19524b2.jpg)

![psb5.jpg](https://oss-club.rt-thread.org/uploads/3948_ddffa4155579a867ef54d8b0a7313b16.jpg)

18 个回答

prife 2012-10-31

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

amsl 2012-11-01

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

rtt-fans 2012-11-03

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

rtt-fans 2012-11-13

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

rtt-fans 2012-11-13

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

prife 2012-11-14

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

prife 2012-11-14

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

prife 2012-11-14

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

用simulator模拟器测试，
此贴中的测试代码我已经放在application.c里，并导出至finsh中。
test_fs()函数。

原帖中给出的解决方案是不行的。我发现问题出在
dfs_posix.c里

```
515 struct dirent *readdir(DIR *d)
516 {
517     int result;
518     struct dfs_fd *fd;
519
520     fd = fd_get(d->fd);
521     if (fd == RT_NULL)
522     {
523         rt_set_errno(-DFS_STATUS_EBADF);
524
525         return RT_NULL;
526     }
527
528     if (!d->num || (d->cur += ((struct dirent*)(d->buf + d->cur))->d_reclen) >=
529     {
530         /* get a new entry */
531         result = dfs_file_getdents(fd, (struct dirent*)d->buf, sizeof(d->buf) -
532         if (result <= 0)
533         {
534             fd_put(fd);
535             rt_set_errno(result);
536
537             return RT_NULL;
538         }
539
540         d->num = result; //此值一般是 sizeof(struct dirent)=260 字节
541         d->cur = 0; /* current entry index */
542     }
543
544     fd_put(fd);
545
546     return (struct dirent *)(d->buf+d->cur);
547 }
548 RTM_EXPORT(readdir);
```

问题就出在540行，d->num = result; d->num这个域的语义不是很明确。

```

558 long telldir(DIR *d)
559 {
560     struct dfs_fd *fd;
561     long result;
562
563     fd = fd_get(d->fd);
564     if (fd == RT_NULL)
565     {
566         rt_set_errno(-DFS_STATUS_EBADF);
567
568         return 0;
569     }
570
571     result = fd->pos - d->num + d->cur;
572     fd_put(fd);
573
574     return result;
575 }
576 RTM_EXPORT(telldir);   
```

d->num和d->cur这两个域看起来是用于dfs_file_getdents 这个函数。当这个函数可以一次读取多个 struct dirent结构时，num = n*sizeof(struct dirent), n是dfs_file_getdents实际读取到的struct direct结构。
但是当前DFS的实现中， dfs_file_getdents每次都是只读一个 struct dirent结构，即n为1，所以num和cur并没有太用处。
如果要使得dfs_file_getdents可以一次读取多个struct dirent结构，那么上面的readdir还需要仔细修改。至少528行的判断就需要改的。

不知道我对num和cur的理解是否正确啊？

PS：简单点的修改办法是把571行修改为
571     result = fd->pos; 
这样就似乎就没什么问题了。