1

これは、正常にコンパイルされますが、実行時にセグメンテーション違反が発生するコードの一部です。この問題を回避する方法を誰か教えてください。コードは 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を試しましたが、問題は解決しませんでした

4

1 に答える 1

6

-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

これをすべて行っても、いくつかのケースを処理できなかったため、まだバグがあります。gdband/orで修正してみてください(コンパイル時にオプションvalgrindを設定することを考えてください)。-g

于 2013-04-18T21:13:29.577 に答える