Question

这是我编写的shell程序，用于磨练我对C指针和数组的理解。这个shell程序具有读入命令的功能（在execvp（）中执行），历史记录功能可以存储10个命令（类似于Linux终端），其中“!!”返回最新命令并执行，！Nth返回第N个命令并执行，否则读取命令并将它们存储在历史数组中。当请求的历史命令不存在时，打印相关的错误消息。有关完整的代码参考，请参阅Shell_Program_C。

initialize()方法用字母'e'初始化每个历史字符串数组（'e'表示空或字符串尚未分配），以便稍后我可以检查历史字符串是否有值。 readHist()方法将每个单独的命令分配给历史字符串数组char** hist，其大小为10.

我的问题是：strcpy(hist[histC], tokens[count]) readHist()在此处返回“错误访问”。

预期行为：应将每个命令作为字符串字符串复制到char** hist，以便可以检索和执行以后请求的命令。

char** initialize() {
char** hist = malloc(10 * sizeof(char *));
for (int i = 0; i < 10; ++i) {
        hist[i] = (char *)malloc((MAX_LINE / 2) + 1);
    for(int j = 0; j < (MAX_LINE / 2); j++) {
        hist[i][j] = 'e';
    }
}
return hist;
}


void readHist(char**hist, int histC ,char** tokens, int count) {
histC = (histC - 1) % 10;
for(int i = 0; i < count; i++) {
    size_t strlenth = strlen(tokens[i]);
    //strcat(hist[histC], tokens[count]);
    if(count > 1) {
       strcat(hist[histC], tokens[count]);
       hist[histC][strlenth] = ' ';
    } else {
        printf("histC%d", histC);
       strcpy(hist[histC], tokens[count]); // bad access or segmentation fault
       hist[histC][strlenth] = '\0';
    }
}
}

Answer 1

您的SegFault违反了strcat要求＆＃34; 字符串可能不重叠＆＃34;这看起来像一个粗心的疏忽，会产生重大影响。具体来说，请查看以下cmdargv：

histCheck(cmdargv, &histCount, cmdargv, count, &isAmp);

您将相同的指针数组 cmdargv作为传递给hist和tokens的参数传递给：

void histCheck(char**hist, int* histCount, char**tokens, int tcount, int* amp)

在histCheck中，hist和tokens都指向可在调试器中看到的相同内存，例如

histCheck (hist=0x7fffffffd9b0, histCount=0x7fffffffd9a8, tokens=0x7fffffffd9b0

注意 hist和tokens的地址。

然后将同样的错误传递给readHist，例如

readHist (hist=0x7fffffffd9b0, histC=1, tokens=0x7fffffffd9b0

会导致您致电strcat尝试连接重叠字符串（相同的字符串），例如：

strcat (hist[histC], tokens[count]);

（注意：您确定要上面count吗？）

导致您熟悉：

Program received signal SIGSEGV, Segmentation fault.
__strcat_sse2_unaligned () at ../sysdeps/x86_64/multiarch/strcpy-sse2-unaligned.S:296
296     ../sysdeps/x86_64/multiarch/strcpy-sse2-unaligned.S: No such file or directory.

现在我并不是说你的问题仅限于此（请注意我在count中更新为checkAmp的评论），所以你还有更多工作要做。但是你的 SegFault 不再是一个谜。

看起来你打算：

histCheck (hist, &histCount, cmdargv, count, &isAmp);

您还需要重新访问：

strcat (hist[histC], tokens[count]);

这将导致您的下一个SegFault。在这里你似乎想要：

strcat (hist[histC], tokens[i]);

此外，在您的代码返工时，请谨慎注意重置所有值，并在do中main循环周围进行每次迭代，以防止将错误值留在{ {1}}等等......你还需要一些方法来cmdargv你分配的内存，以防止在各种函数中对free的后续调用中泄漏内存。

您希望与调试器结交朋友。没有办法绕过它，特别是当你将流程切入一些小的相互关联的函数来划分代码时。这没有什么不妥，它只是让调试器变得更加重要，以确定车轮何处脱落 - 以及为什么......

修改您的代码并发布您遇到的任何其他问题。要在malloc中绑定count问题，您可以考虑以下内容：

checkAmp

（并将呼叫更新为void checkAmp (int *count, char** tokens, int *Amp) { size_t strlenth = strlen(tokens[*count - 1]); if((strlenth == 1) && tokens[*count - 1][0] == '&') { *Amp = 1; tokens[*count - 1] = NULL; *count = *count -1; } else { tokens[*count] = NULL; } }）

最后，虽然不是错误，但C的标准编码样式避免使用checkAmp或caMelCase变量名来支持所有小写，同时保留<用于宏和常量的em>大写名称。这是一个风格问题 - 所以完全取决于你...

strcpy导致访问不良或故障分段

1 个答案: