c - strcpy 导致访问错误或段错误

Question

这个问题已经有答案了:

Correctly allocating multi-dimensional arrays (2 个回答)

已关闭 6 年前。

这是我为了加深对 C 指针和数组的理解而编写的 shell 程序。该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要求“字符串不得重叠”这看起来像是一个粗心的疏忽，有很大的影响结果。具体请看下面的cmdargv:

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

您正在传递相同的指针数组 cmdargv 作为传递给 hist 和 tokens 的参数在您的通话中:

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

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

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

记下hist和 token 的地址。

然后将相同的错误传递给 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.

现在我并不是说您的问题仅限于此(请注意我关于他在 checkAmp 中更新为 count 的评论)，因此您还有更多工作要做。但您的SegFault不再是一个谜。

看来您的意图是:

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

您还想重新访问:

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

这将导致您的下一个SegFault。这里出现了您想要的:

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

此外，在修改代码时，请特别注意在 main 中围绕 do 循环的每次迭代重置所有值，以防止在 cmdargv 等中留下杂散值...您还需要某种方法来释放您分配的内存，以防止在后续调用 malloc< 时泄漏内存 在你的各种函数中。

您会想与调试器交 friend 。没有办法解决这个问题，特别是当您将流程分割成许多小的相互关联的函数来划分代码时。这没什么问题，它只是让调试器变得更加重要，以找出轮子在哪里脱落 - 以及为什么......

修改您的代码并发布您遇到的任何其他问题。要解决 checkAmp 中的 count 问题，您可以考虑如下操作:

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;
    } 
}

(并更新对 checkAmp 的调用)

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