malloc()：在一个奇怪的地方内存损坏答案

【问题标题】：malloc(): memory corruption in a weird placemalloc()：在一个奇怪的地方内存损坏
【发布时间】：2016-03-02 19:34:36
【问题描述】：

几个小时以来，我一直在处理烦人的内存损坏错误。我已经检查了这里的每个相关线程，但我无法修复它。

首先，我正在用 C 语言编写一个简单的终端。我正在解析管道 (|) 和重定向 (>、

这是队列的数据结构；

struct command_node {
    int index;
    char *next_symbol;
    struct command_node *nextCommand;
    int count;
    char **args;
};

当我在 **args 指针中存储小字符串时，一切正常。但是，当其中一个参数很长时，我得到 malloc(): memory corruption 错误。例如，以下第 1 个命令可以正常工作，第 2 个命令会导致错误

   1st:  ls -laF some_folder | wc -l

   2nd:  ls -laF /home/enesanbar/Application | wc -l

我运行调试器，它显示对队列中新节点的 malloc() 调用导致错误。

newPtr = malloc( sizeof(CommandNode) );

我正在仔细分配字符串数组并在完成后释放它们，如下所示：

    char **temp = NULL;
    temp = malloc(sizeof(char*) * number_of_args);

    /* loop through the argument array */
    for (i = 0; i < number_of_args; i++) {
        /* ignore the remaining commands after ampersand */
        if (strcmp(args[i], "&") == 0) return;

        /* split commands by the redirection or pipe symbol */
        if (!isSymbol(args[i])) {
            temp[count] = malloc(sizeof(strlen(args[i])) + 1);
            strcpy(temp[count], args[i]);
            count++;

            /* if it is the last argument, assign NULL to the symbol in the data structure */
            if (i + 1 == number_of_args) {
                insertIntoCommands(&headCommand, &tailCommand, temp, NULL, count);
                for (j = 0; j < count; j++) free(temp[j]);
                count = 0;  // reset the counter
            }

        }
        else {
            insertIntoCommands(&headCommand, &tailCommand, temp, args[i], count);
            for (j = 0; j < count; j++) free(temp[j]);
            count = 0;  // reset the counter
        }
    }

我一定错过了什么，或者我不知道 **args 字段和新节点的分配，尽管这不是我以前没有做过的事情。

【问题讨论】：

temp[count] count？
它是为了跟踪下一个参数的放置位置，并且在打印用于调试的内容时，我也会将其存储起来以方便使用。
sizeof(strlen(... 可能不是您想要的。删除大小
谢谢，这解决了这个问题，但是在 sizeof 周围包裹一个数字怎么会导致节点分配错误呢？出于好奇，我只是想了解一下。
您尝试在strlen 函数中获取指针的大小，而不是通过函数提供的长度。

标签： c pointers memory malloc corruption

【解决方案1】：

但是在 sizeof 周围包裹一个数字怎么会导致节点分配错误呢？出于好奇，我只是想了解一下。

就像我在评论中所说的那样，您尝试在 strlen 函数中获取指针的大小，而不是通过函数提供的长度。

请看以下内容：

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


int main(void){
    char *name = "Michi";
    size_t length1, length2;

    length1 = strlen(name);
    length2 =  sizeof strlen(name);

    printf("Length1 = %zu\n",length1);
    printf("Length2 = %zu\n",length2);
    return 0;
}

输出：

Length1 = 5
Length2 = 8

还有一件事，在你free(temp[j]) 之后别忘了也free(temp)。

类似这样的：

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


int main(void){
    long unsigned int size = 2,i;
    char **array;

    array = malloc(sizeof(char*) * size * size);

    if (array == NULL){
        printf("Error, Fix it!\n");
        exit(2);
    }

    for (i = 0; i < size; i++){
         array[i] = malloc(sizeof(char*) * 100);
    }

    /* do code here */

    for (i = 0; i < size; i++){
         free(array[i]);
    }
    free(array);

    return 0;
}

【讨论】：