关于一个备份功能的小程序的问题

jerrymy

源码：

1. 
   
2. #include<stdio.h>
   
3. #include<sys/types.h>
   
4. #include<sys/stat.h>
   
5. #include<unistd.h>
   
6. #include<fcntl.h>
   
7. int main(int argc,char *argv[])
   
8. {
   
9.         int fd;
   
10.         int stat,pid;
    
11.         struct stat stbuf;
    
12.         time_t old_time = 0;
    
13.         if( (fd=open(argv[1],O_WRONLY))==-1 )
    
14.         {
    
15. 
    
16.                 printf("can't ooen watchfile\n");
    
17.                 return 2;
    
18.         }
    
19. 
    
20.        
    
21. 
    
22.         fstat(fd,&stbuf);
    
23.         old_time = stbuf.st_mtime;
    
24.         for(;;)
    
25.         {
    
26.                 fstat(fd,&stbuf);
    
27.                 if(old_time != stbuf.st_mtime)
    
28.                 {
    
29.                         while(pid = vfork()== -1 );
    
30.                        
    
31.                         if(pid == 0)
    
32.                         {
    
33.                                 execl("/bin/cp","/bin/cp",argv[1],argv[2],0);
    
34.                                 return 3;
    
35.                         }
    
36.                         wait(&stat);
    
37.                         old_time = stbuf.st_mtime;
    
38.                 }
    
39.                 else
    
40.                         sleep(20);
    
41.         }
    
42. }
    

复制代码

程序的功能：循环检测原文件的mtime,如果mtime改变则在子程序执行备份。

这是一个群里的朋友问的，这个程序在我的Cent OS5.6上能执行，但有个诡异的现象：创建的原文件必须是空文件/

并且只有用touch flie 才能执行备份。

如果用编辑器更改原文件的mtime则程序永远都在无限循环，不会执行备份。

但我加了一句之后变为下面,即在for循环里加上fd=open(argv[1],O_WRONLY)；这句：

1. 
   
2. #include<stdio.h>
   
3. #include<sys/types.h>
   
4. #include<sys/stat.h>
   
5. #include<unistd.h>
   
6. #include<fcntl.h>
   
7. int main(int argc,char *argv[])
   
8. {
   
9.         int fd;
   
10.         int stat,pid;
    
11.         struct stat stbuf;
    
12.         time_t old_time = 0;
    
13.         if( (fd=open(argv[1],O_WRONLY))==-1 )
    
14.         {
    
15. 
    
16.                 printf("can't ooen watchfile\n");
    
17.                 return 2;
    
18.         }
    
19. 
    
20.        
    
21. 
    
22.         fstat(fd,&stbuf);
    
23.         old_time = stbuf.st_mtime;
    
24.         for(;;)
    
25.         {
    
26.                 fd=open(argv[1],O_WRONLY)；
    
27.                 fstat(fd,&stbuf);
    
28.                 if(old_time != stbuf.st_mtime)
    
29.                 {
    
30.                         while(pid = vfork()== -1 );
    
31.                        
    
32.                         if(pid == 0)
    
33.                         {
    
34.                                 execl("/bin/cp","/bin/cp",argv[1],argv[2],0);
    
35.                                 return 3;
    
36.                         }
    
37.                         wait(&stat);
    
38.                         old_time = stbuf.st_mtime;
    
39.                 }
    
40.                 else
    
41.                         sleep(20);
    
42.         }
    
43. }
    

复制代码

更改后的程序用touch和编辑器编辑之后都能执行备份。我觉得很纳闷，烦劳前辈们解释一下/

循环前面的open打开的fd跟循环的open打开的fd的stat结构体有什么不一样。

另外为什么成功执行备份后都程序都会退出，但从源码来看，程序的意图是无限循环，一旦文件mtime改变就执行备份，but。。。。

这是清华大学某某某写的一本linux 编程书籍上的例子。

jerrymy

程序虽然看起来有点点长，但功能很简单，请路过的认真看一下。

snriyt

1. [code]

复制代码

[/code]

jerrymy

回复 3# snriyt


    你发的什么啊，没东西啊。

snriyt

回复 4# jerrymy


    代码括起来吧

jerrymy

顶一下。

yug1129

估计原来程序“如果用编辑器更改原文件的mtime则程序永远都在无限循环，不会执行备份”是由于系统对打开文件属性(*fd)的缓存，使得每次fstat(fd,&stbuf)内容不变。

更新的代码中没有fclose()，运行时间长了会出问题。

jerrymy

回复 7# yug1129


    第一点有同感，但不确定。

加入那个语句仅仅是希望程序能执行成功。

即便是执行成功的情形，程序的行为跟作者的本意也不一致，执行一次之后父进程也嗝屁了，所以都不用fclose()了

如果程序按照预期的执行，fclose()是必要的。

yug1129

为什么不直接使用system()?为什么要vfork()?是为了检验系统调用还是为了实现备份功能？

jerrymy

为什么不直接使用system()?为什么要vfork()?是为了检验系统调用还是为了实现备份功能？
yug1129 发表于 2011-05-04 17:59

    目的是根据文件修改时间的变化来备份文件。

只要mtime更改了就copy.