0

老实说，我不知道以下情况是如何发生的。这是代码：

while(1)
{
    char** line = read_command();
    char* command = line[0];
    char** parameters = malloc(100);
    int i;

    for(i = 0; i < pNum; i++) // pNum is a global var containing the number of words read from read_command()
    {
        parameters[i] = line[i];
        printf("%i: %s", i, parameters[i]);
    }

    printf("%s\n", parameters[0]);
    parameters[0] = "/usr/bin/";
    strcat(parameters[0], command);
    printf("%s\n", command);
    printf("%s\n", parameters[0]);

    if(fork() != 0)
        waitpid(1, &status, 0);
    else
        execve(parameters[0], parameters, NULL);
}

read_command() 返回一个 char**，它基本上是输入字符串的“数组”，每个 char* 包含一个单词。就像我输入“hello people of earth”一样，结果将是 [“hello”、“people”、“of”、“earth”]。此功能始终有效。

在第一次迭代时，一切都按预期工作。例如，当我输入“日期”时，输出如下：

0: date
date
date
/usr/bin/date
and then the date is displayed

但是在第二次迭代中，如果我再次使用“日期”作为输入，则输出如下：

0:date
edate 
/usr/bin/datedate 
and the date command is not issued

即使我打印像“hello”这样的常量字符串，第二个 printf 语句也总是在第一次迭代后打印“e”。然后参数 [0] 以某种方式在其中包含 2 个“日期”，即使命令指针只有 1 个“日期”。

在第三次迭代之后，程序不等待用户输入，它只是不停地循环并显示“PM：警告，进程表已满！”

什么可能导致这种情况？

我在 MINIX 3.1.0 中使用 C 的 cc 编译器工作

编辑：read_command()：

char* line = malloc(), * linep = line;
size_t lenmax = 100, len = lenmax;
int c;
int currPos = 0;
int currParam = 0;
int i;
char** parameters = malloc(100);

if(line == NULL)
    return NULL;

while(1)
{
    c = fgetc(stdin);

    if(c == EOF) || c == '\n')
        break;

    if(--len == 0)
    {
        char* linen = realloc(linep, lenmax *= 2);
        len = lenmax;

        if(linen == NULL)
        {
            free(linep);
            return NULL;
        }

        line = linen + (line - linep);
        linep = linen;
    }

    if((*line++ = c) == '\n')
        break;
}

*line = '\0'; // everything up to this point i got from this link: http://stackoverflow.com/a/314422/509914

 parameters[currentParam] = malloc(100);

for(i = 0; i < strlen(linep); i++);
{
    if(isspace(linep[i]) || line[i] == EOF)
    {
        parameters[currParam][currPos] = '\0;
        currPos = 0;
        parameters[++currParam] = malloc(100);
    }
    else
        parameters[currParam][currPos++] = line[i];
}

parameters[currParam][currPos] = '\0';
pNum = currParam + 1;
return parameters;
4

1 回答 1

1

有趣的是，那些通过阅读信誉良好的资源来学习的人，例如几十年来久经考验的 K&R，他们遇到这些问题的频率往往远低于那些不...

char** parameters = malloc(100);正在尝试分配 100个字节。realloc行为类似，所以我不会再提这个了。也许您的意思是分配 100 批char *? 分配pNum很多char *: char **parameters = malloc(pNum * sizeof *parameters);... char* linen = realloc(linep, (lenmax *= 2) * sizeof *linep);...

strcat不分配内存；malloc分配内存的唯一函数是calloc和realloc。

当您打电话时，strcat(foo, bar);您要求strcat将指向的字符串附加到指向的字符串bar的末尾foo。在您的代码中，您正在尝试修改字符串文字。未定义的行为，通常是段错误。

即使您修改后的尝试也是错误的。在parameters[0] = "/usr/bin/";中，您没有将字符串复制到parameters[0]; 您正在分配parameters[0]指向字符串（正如我之前提到的，它通常位于不可变内存中）。您确实需要通过创建 MCVE 来缩小未定义行为的来源...

在您的read_command()函数的第一行中char* line = malloc(), * linep = line;，您在malloc没有提供参数的情况下进行了调用。这是违反约束的。您的编译器应该向您发出错误。也许你忘记了#include <stdlib.h>，所以malloc错过了它的原型？请提供一个 MCVE，这样我们就不必做出这样的猜测。

在...处还有另一个约束违规，if(c == EOF) || c == '\n')即使我们要填写空白来为您生成 MCVE（我们不应该这样做，因为这是您的工作，而您正在向我们寻求帮助），这段代码也会不编译。也许这就是导致您崩溃的原因？始终检查编译器给您的消息。不要忽略警告......绝对不要忽略错误消息。

我比较了你声称来自这个答案的代码，它是完全不同的。该答案的代码可以编译，例如。尽管如此，这是你与试图帮助你的人建立信任的方式......通过说谎？

你不需要这么多的动态分配，通常我会不厌其烦地解释你如何以最好的方式做到这一点，但谎言让我失望了。我还有一点要说：确保parameters由一个空指针终止（类似于字符串如何由一个空字符终止），因为它是手册所要求的。

于 2015-08-11T09:27:08.213 回答