Question

char **add_string(char **existing, const char *string){
  size_t size = 0;
  while (NULL != existing[size]) 
    {
      ++size; 
    }
  char **arr = realloc(existing, (size + 2) * sizeof *arr);
  arr[size] = malloc(strlen(string) + 1);
  strcpy(arr[size], string);  
  arr[size+1] = '\0';
  return arr;
}

void free_strings(char **strings)
{
  size_t size = 0;
  while (NULL != strings[size]) 
  {
    free(strings[size]);
    ++size;
  }
}

我在第

行有内存泄漏

char **arr = realloc(existing, (size + 2) * sizeof *arr);

我认为在使用realloc时，现有内存被认为是免费的？如何修复此内存泄漏？

编辑：添加了我的free_string函数和我用来运行程序的主要文件。

Answer 1

您没有使用空指针附加指针existing指向的数组。因此在这个循环中

while（NULL！=现有[size]） { ++大小; }

该函数具有未定义的行为。

看来你的意思是以下

char ** add_string( char **existing, const char *string )
{
    size_t size = 0;

    while ( NULL != existing[size] ) ++size; 

    char **arr = realloc( existing, ( size + 2 ) * sizeof *arr );

    if ( arr != NULL )
    {
        arr[size] = malloc( strlen( string ) + 1 );
        strcpy( arr[size], string );  
        arr[size+1] = NULL;
    }

    return arr;
}

函数free_strings也不正确。它应该看起来像

void free_strings( char **strings )
{
    size_t size` = 0;

    do
    {
        free( strings[size] );
    } while ( strings[size++] != NULL );

    free( strings );
}

c中字符串数组的内存泄漏

1 个答案: