Question

这里有一段编译很好但在运行时给出分段错误的代码。任何人都可以告诉我如何解决这个问题。代码在C中实现rm系统调用。

#include<sys/stat.h>
#include<unistd.h>
#include<dirent.h>
#include<error.h>
#include <stdio.h>
#include <string.h>


int rmq(char*pth)
{
  char path[1000]; //hold the file name to be removed
  strcpy(path,pth);
  char *b; // stores the complete path of the file to be removed
  struct dirent *d;
  DIR *dir;

  char cwd[256]; //holds current working directory
  getcwd(cwd, sizeof(cwd));  
  dir = opendir(cwd);
  char path1[1000]; //for recursively moving through dir and subdir
  strcpy(path1,cwd);

  char newp[1000];
  struct stat buf;

  while((d = readdir(dir))) //if there are directories to be read 
  {  
     if(!strcmp(d->d_name,".") || !strcmp(d->d_name,"..")) // skip "." and ".."
     continue;

     //appends directory read to cwd

     strcpy(newp,path1); 
     strcat(newp,"/");
     strcat(newp,d->d_name); 
     //printf("%s>>",d->d_name);

     if(stat(newp,&buf)==-1) // puts file info in buf
        perror("stat");
     if(S_ISDIR(buf.st_mode))// if directory, then add a "/" to current path
     { 
     //if a directory then call function again(recursion)
        strcat(path1,"/");
        strcat(path1,d->d_name);
        rmq(path1);
     }
     else{ 
        //if directory current read is the one to be removed
        if((strcmp(path, d->d_name)) == 0){
        // append it with cwd & put it in b
        b = malloc(25 + strlen(d->d_name) + 1);
        sprintf(b, "%s/%s", "/home/urwa/Documents/OPS", d->d_name);
        remove(b); // remove that file
        free(b);
        }
     } 
 }
 return 0;
}


int main(){
   char cwd[256];
   getcwd(cwd, sizeof(cwd));

   char *argv[2];
   argv[1] = "dumbledore.txt";
   rmq(argv[1]); // file to be removed is passed as parameter
   return 0;
}

我尝试了malloc，但它没有解决问题

Answer 1

您应该使用-Wall -Wextra标志编译代码，以便更好地从编译器帮助中受益。在这里，使用您的代码，您将收到以下警告：

test.c: In function ‘rmq’:
test.c:9:4: warning: implicit declaration of function ‘strcpy’ [-Wimplicit-function-declaration]
test.c:9:4: warning: incompatible implicit declaration of built-in function ‘strcpy’ [enabled by default]
test.c:25:3: warning: suggest parentheses around assignment used as truth value [-Wparentheses]
test.c:26:3: warning: implicit declaration of function ‘strcmp’ [-Wimplicit-function-declaration]
test.c:30:5: warning: implicit declaration of function ‘strcat’ [-Wimplicit-function-declaration]
test.c:30:5: warning: incompatible implicit declaration of built-in function ‘strcat’ [enabled by default]
test.c:34:5: warning: implicit declaration of function ‘perror’ [-Wimplicit-function-declaration]
test.c:43:5: warning: implicit declaration of function ‘sprintf’ [-Wimplicit-function-declaration]
test.c:43:5: warning: incompatible implicit declaration of built-in function ‘sprintf’ [enabled by default]
test.c:43:5: warning: passing argument 1 of ‘sprintf’ makes pointer from integer without a cast [enabled by default]
test.c:43:5: note: expected ‘char *’ but argument is of type ‘char’
test.c:44:5: warning: implicit declaration of function ‘remove’ [-Wimplicit-function-declaration]
test.c: In function ‘main’:
test.c:58:2: warning: control reaches end of non-void function [-Wreturn-type]
test.c: In function ‘rmq’:
test.c:43:12: warning: ‘b’ may be used uninitialized in this function [-Wuninitialized]

因此，解决所有问题的最佳方法是执行以下操作：

首先，缺少包括：
```
#include <stdio.h>
#include <string.h>
```
在while测试中为作业添加额外的括号（编译器需要这些括号以确保你知道你做了什么）：
```
while((d = readdir(dir))){
```

您需要将char b;转换为char *b;并分配/取消分配必要的内存来存储字符串：

if((strcmp(path, d->d_name)) == 0){
  b = malloc(25 + strlen(d->d_name) + 1);
  sprintf(b, "%s/%s", "/home/urwa/Documents/OPS", d->d_name);
  remove(b);
  free(b);
}

然后，您需要将return 0添加到main函数，以匹配您定义的签名。

完成所有这些后，仍然存在一个错误，因为您错过了处理一些案例。尝试使用gdb和/或valgrind进行修复（在编译时考虑设置-g选项。）

在Linux中实现rm命令的C代码中的分段错误

1 个答案: