2

我需要制作一个程序，它需要一些系统命令，如ls,date等，并检查是否存在包含此命令（文件）的路径。我有变量commandandparameters在最后一个循环中开始改变，while我不知道为什么。

我puts(commandandparameters);表明如果你想运行它，输出效果不好。

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
void main()
{
    char *arr[5];//for the command and his parameter
    char command[10];
    int i,j;
    char *path=NULL,*tempath,*finalpath,*commandandparameters;
    do
    {
        i=0;
        printf("enter new command:");
        gets(command);
        arr[i]=strtok(command," ");
        while(arr[i]!=NULL)//save the command and also the parametrs
        {
            i++;
            arr[i]=strtok(NULL," ");
        }
        strcpy(commandandparameters,arr[0]);//add first command
        for(j=1;j<i;j++)//add the parameters
            {
                strcat(commandandparameters," ");
                strcat(commandandparameters,arr[j]);
            }
        //now we check if the command in every path
        path = getenv("PATH");
        tempath = strtok(path,":");
        while (tempath != NULL)
        {
            strcpy(finalpath,tempath);//get the current path
            puts(commandandparameters);
            strcat(finalpath,"/");//we add '/'
            execl(finalpath,commandandparameters,NULL);
            tempath = strtok(NULL, ":");//get next path
        }
    }while(command!="leave");
}
4

2 回答 2

4

您尚未定义commandandparameters指向的空间：

char *path=NULL,*tempath,*finalpath,*commandandparameters;
...
    strcpy(commandandparameters,arr[0]);

你有一个指向随机空间的指针；你复制那个随机空间。你会得到有趣的结果。如果幸运的话，程序会崩溃。如果你不走运，它会行为不端。

你有一个类似的问题finalpath：

    path = getenv("PATH");
    tempath = strtok(path,":");
    while (tempath != NULL)
    {
        strcpy(finalpath,tempath);

更糟糕的是，你也在屠杀你的环境。除非您打算修改 PATH 的值，否则返回的字符串getenv()应被视为只读。:事实上，在循环之后你不会留下太多的 PATH （如果它的第一个元素是 PATH ，则没有 PATH ）。

确保您知道每个指针指向的位置。

您的代码有很多潜在的缓冲区溢出，这令人毛骨悚然。永远不要使用gets()；假设它会把你的电脑炸成碎片。

如果你解决了这些问题，do { ... } while (command != "leave");循环就等于无限循环。你不能像这样有用地比较字符串；你需要使用strcmp().

我尝试做一个简单的程序，但我发现它execl()不起作用；有人能告诉我为什么“ls”命令不起作用吗？

这是评论中代码的略微修改版本。我在打印中添加了两个标题和换行符，但关键的变化是execl()在行中。您的原件在评论中；工作版本不是评论。我无法确定这是您的主要问题还是评论中的错字。修改后的代码编译并运行。

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

int main(void)
{
    int pid, stat;

    if ((pid = fork()) == 0)
    {
        execl("/bin/ls", "ls", NULL);
        //execl("/bin/", "ls", NULL);
        printf("1\n");
        exit(1);
    }
    else
    {
        wait(&stat);
        printf("2\n");
    }
}

程序中的代码会破坏 PATH；如果您未能执行第一个程序，那么它只会在原始 PATH 上的第一个目录中查找。你试图用来execl()处理可变数量的参数；这是工作的错误工具。您必须使用execv()或其亲属之一（execvp()例如，在不弄乱 PATH 的情况下进行 PATH 搜索）。原因是这execl()需要一组由空指针终止的参数，但是只有在知道有多少个参数时才能编写它。你可以写：

execl("/bin/ls", "ls", "-l", (char *)0);

但除非您强制每个命令最多有 4 个参数（给定char *arr[5];）并使用模板，例如：

execl(finalpath, arr[0], arr[1], arr[2], arr[3], arr[4]);

你不能execl()用来执行命令。但这将用户限制为最多 N 个参数，这是不可接受的。（例如，shell 扩展*可以生成 30 个或更多参数的列表。

您的代码也没有将命令名称附加到路径组件；第一个参数execl()是程序可执行文件的路径。

所以，使用execv()（或execvp()或execve()什至，如果你有的话，execvpe()）。如果找到您执行的命令，这是对您的程序的或多或少的最小修改，它可以正常工作；如果不是这样，那将是一场小灾难。这是一个一次性的外壳；它不应该也不fork()喜欢execv()它应该 - 如果你在修改路径之前分叉，那么路径修改将不那么重要，尽管进程运行将具有最小到不存在的路径。

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

int main(void)
{
    char *arr[5];//for the command and his parameter
    char command[100];  // Not 10
    int i;
    int j;
    char *path=NULL;
    char *tempath;
    char finalpath[200];
    char commandandparameters[200];

    do
    {
        i = 0;
        printf("enter new command: ");
        if (fgets(command, sizeof(command), stdin) == 0)
        {
            fprintf(stderr, "EOF or error\n");
            break;
        }
        arr[i]=strtok(command, " ");
        while (i < (5-1) && arr[i]!=NULL)//save the command and also the parameters
        {
            printf("arr[%d] = %s\n", i, arr[i]);
            i++;
            arr[i]=strtok(NULL, " \n");
        }
        arr[4] = 0;

        strcpy(commandandparameters, arr[0]);//add first command
        for (j=1;j<i;j++)//add the parameters
            {
                strcat(commandandparameters, " ");
                strcat(commandandparameters, arr[j]);
            }
        printf("Cmd&Params: %s\n", commandandparameters);

        //now we check if the command in every path
        path = getenv("PATH");
        tempath = strtok(path, ":");
        while (tempath != NULL)
        {
            puts(commandandparameters);
            strcpy(finalpath, tempath);//get the current path
            strcat(finalpath, "/");//we add '/'
            strcat(finalpath, arr[0]);
            puts(finalpath);
            execv(finalpath, arr);
            tempath = strtok(NULL, ":");//get next path
        }
    } while (strcmp(command, "leave") != 0);
    return(0);
}
于 2012-11-28T13:56:56.693 回答
1

您只存储由返回的指针strtok()，它们将指向command. 循环的每次迭代都会覆盖command，从而导致“旧”指针指向的数据发生变化。

您必须将标记从 中复制出来command，或者复制到动态分配的内存中（查找malloc()），或者有一个静态大小的字符串数组。在这种情况下，请注意缓冲区溢出。

于 2012-11-28T13:56:36.177 回答