由于realloc而在执行时出错

时间:2015-12-14 09:58:48

标签: c arrays pointers memory-management realloc

我写了一个小程序控制台程序,用于存储数字中的单词,由char** test_tab表示,然后打印出来。

只要不通过条件realloc()
(例如,如果我将size增加到1000),该程序就可以正常工作。 /> 但如果realloc()被调用,程序在阵列打印期间崩溃,可能是因为内存混乱了。

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

char* get_word();

int main(int argc, char* argv[])
{
    size_t size = 100;
    size_t nb_pointer = 0;
    char** test_tab = malloc(size * sizeof *test_tab);
    char** temp_tab;

    while((*(test_tab + nb_pointer) = get_word()) != NULL)
    {
        nb_pointer++;
        if(nb_pointer >= size)
        {
            size += 100;
            temp_tab = realloc(test_tab, size);

            if(temp_tab != NULL)
                test_tab = temp_tab;
            else
            {
                free(test_tab);
                exit(1);
            }
        }
    }

    for(nb_pointer = 0; *(test_tab + nb_pointer) != NULL; nb_pointer++)
        printf("%s\n", *(test_tab + nb_pointer));

    free(test_tab);

    return 0;
}

有人能解释我在这里做错了什么吗?感谢。

2 个答案:

答案 0 :(得分:2)

realloc中的内存量未正确计算。

       temp_tab = realloc(test_tab, size);

应该是

       temp_tab = realloc(test_tab, size * sizeof *test_tab);

答案 1 :(得分:0)

每当您尝试按下一个字符串并同时将所有先前推送的字符串随身携带。现在字符串表示char *&amp;因此你需要使用sizeof(char*) * size&amp;然后你需要再次将内存分配给字符串以存储实际字符.. 但是你也可以这样接近

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

static int size = 0;  // static global means no risk while including this file


char** push(char** memptr, char* data) {
  size++;
  if (size == 1)
    memptr = (char**)malloc(size * sizeof(char*));
  else
    memptr = (char**)realloc(memptr, size* sizeof(char*));

  memptr[size - 1] = (char*)malloc(sizeof(char) * strlen(data) + 1);
  strncpy(memptr[size - 1], data, strlen(data));
  memptr[size - 1][strlen(data) -1] = '\0'; // over writing the `\n` from `fgets`
  return memptr;
}


int main() {
  char buf[1024];
  int i;
  static char** memptr = NULL;
  for (i = 0; i < 5; i++){
    fgets(buf, 1024, stdin);
    memptr = push(memptr, buf);
  }

  for (i = 0; i < size; i++)
    printf("%s\n", memptr[i]);

  return 0;
}