无法使用“execve()”成功地

Question

该程序的目的是分叉一个新的子进程并执行一个也具有命令行参数的进程。如果我输入 /bin/ls --help，我会收到错误：

shadyabhi@shadyabhi-desktop:~/lab/200801076_lab3$ ./a.out
Enter the name of the executable(with full path)/bin/ls --help
Starting the executable as a new child process...
Binary file to be executed: /bin/ls
/bin/ls: unrecognized option '--help
'
Try `/bin/ls --help' for more information.
Status returned by Child process: 2
shadyabhi@shadyabhi-desktop:~/lab/200801076_lab3$

What would be the right argument to execve()?

#include<stdio.h>
#include<string.h>      //strcpy() used
#include<malloc.h>      //malloc() used
#include<unistd.h>      //fork() used
#include<stdlib.h>      //exit() function used
#include<sys/wait.h>    //waitpid() used

int main(int argc, char **argv)
{
    char command[256];
    char **args=NULL;
    char *arg;
    int count=0;
    char *binary;
    pid_t pid;
    printf("Enter the name of the executable(with full path)");
    fgets(command,256,stdin);
    binary=strtok(command," ");
    args=malloc(sizeof(char*)*10);
    args[0]=malloc(strlen(binary)+1);
    strcpy(args[0],binary);
    while ((arg=strtok(NULL," "))!=NULL)
    {
        if ( count%10 == 0) args=realloc(args,sizeof(char*)*10);
        count++;
        args[count]=malloc(strlen(arg));
        strcpy(args[count],arg);
    }
    args[++count]=NULL;
    if ((pid = fork()) == -1)
    {
        perror("Error forking...\n");
        exit(1);
    }
    if (pid == 0)
    {
        printf("Starting the executable as a new child process...\n");
        printf("Binary file to be executed: %s\n",binary);
        execve(args[0],args,NULL);
    }
    else
    {
        int status;
        waitpid(-1, &status, 0);
        printf("Status returned by Child process: %d\n",WEXITSTATUS(status));
    }
    return 0;
}

Answer 1

args 数组中的第一个条目应该再次是程序名称。您的代码以 --help 作为进程名称来调用 /bin/ls 。

Answer 2

请检查以确保 args 没有被 realloc 调用破坏。请参阅此处关于 realloc

编辑：
而且循环看起来很有趣......
您像这样调用 strtok ：

binary=strtok(command," ");

将循环构造更改为使用 binary 代替，如图所示...

char *tmpPtr;
while (binary != NULL){
   if ( count%10 == 0) tmpPtr=realloc(args,sizeof(char)*10);
   if (tmpPtr != NULL) args = tmpPtr;
   count++;
   args[count-1]=malloc(strlen(binary)+1);
   strcpy(args[count-1],binary);  
   binary = strtok(command, " ");
}

并使用 binary 复制字符串。 ..

希望这有帮助，
此致，
汤姆.

Answer 3

你的程序有一些明显的错误。例如，声明 char **args=NULL; 然后声明 args=realloc(args,sizeof(char)*10); （因为它是 char**，你应该alloc-ing到char*，不是吗？..）。

由于 sizeof(char*) 通常为 4，而 sizeof(char) 通常为 1，因此您最终会遇到一些严重的内存管理问题（分配的内存少于使用的内存） ，你最终会写到不应该写的地方）。从那时起，一切都乱了套，你不能指望你的程序的行为有任何意义。

我建议您通过 Valgrind 之类的实用程序运行程序，以找出内存泄漏并适当纠正程序。一旦内存问题得到纠正，您的 execve 问题可能就会消失。