c - 使用execvp不会出现此类文件或目录错误

我编写这个程序是为了使用execvp命令，这是我的代码：

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>

int main(int argc, const char * argv[]) {

    char **cmd;
    int i;

    if (argc == 1){

        fprintf(stdout, "No command inserted!\n");
        exit(EXIT_SUCCESS);

    }

    cmd = (char **) malloc( (argc + 1) * sizeof(char *));
    cmd[0] = (char *) malloc( strlen(argv[1] + 1 ) * sizeof(char) );
    strcpy(cmd[i], argv[i+1]);

    if (argc > 2){

        for (i = 1 ; i < argc - 1  ; i++ ){

            cmd[i] = (char *) malloc( strlen(argv[i+1] + 1 ) * sizeof(char) );
            strcpy(cmd[i], argv[i+1]);

        }

        cmd[argc] = NULL;
        execvp(cmd[0], cmd);

        fprintf(stderr, "Failed Execution or not existing command!!\n");
        exit(EXIT_FAILURE);

    }

    cmd[1] = NULL;

    execvp(cmd[0], cmd);

    fprintf(stderr, "Failed Execution or not existing command!!\n");
    exit(EXIT_FAILURE);

    return 0;
}

我用这些命令在终端上运行这个程序
./a.out ls
./a.out who
./a.out ls -l
./a.out mkdir newdir
前两个命令和其他没有参数的命令一样工作良好。最后两个我在终端中得到一个No such File or Directory错误，但是实际上mkdir命令创建了目录newdir。。。
有人能帮我吗

最佳答案

您正在设置cmd[argc] = NULL;，但这太多了1（argc是包含命令的参数计数）。因此cmd[argc - 1]没有初始化（注意上面的循环在argc - 2停止
您需要将cmd[argc - 1]设置为NULL，否则会将另一个垃圾参数传递给execvp，例如，这就解释了mkdir对第一个参数有效，但在处理垃圾参数时失败。由于这个移位，所有命令都会收到一个垃圾参数。
您需要一个大小为argc而不是argc + 1的指针数组，因为不考虑a.out命令。
另外，如注释中所述，您使用的i一开始并没有初始化，这是正常的（因为您的mkdir命令可以工作），但是运气不好（未定义的行为）
一个简单得多的方法是复制argv指针数组（忽略第一个是您的注释），而不复制字符串内容的内存（因为您没有修改它们）。

   cmd = malloc( argc * sizeof(char *)); // argc because we need 1 slot for NULL termination
   for (i=0;i<argc-1;i++)
   {
     cmd[i] = argv[i+1];
   }
   cmd[i] = NULL;  // i is argc-1 now

关于c - 使用execvp不会出现此类文件或目录错误，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/46135340/