复制并释放 malloc 指针

Question

我正在尝试寻找内存泄漏并找到了一个来源。我在一个函数中对指针进行 malloc 并在另一个函数中释放它，但我错过了了解如何复制指针指向的值，同时也能够释放指针。

Current implementation (with memory leak):

// This code has been greatly simplified
// and as such does not appear to have any purpose
int foo(){
  int bestval = 0;
  char *best;
  char *match;
  for (int i=0;i<3;i++) {
      int returnValue = bar(&best);
      if (returnValue > 10) {
        (1)
         match = best;
      }
  }

  printf("%s", match); 
  (2)    
  return 0;
}


int bar(char ** best) {
  char*s = "Hello!";
  *best = malloc(strlen(s) + 1);
  strcpy(*best,s);
  return 0;
}

两个问题

1. 如果我必须在（1）而不是（2）处释放内存，我该怎么做才能使匹配仍然具有最佳中包含的内容？

2. 我应该做 strcpy 来复制最佳匹配吗？如果是这样，我是否必须在 foo 中执行另一个 malloc？

Answer 1

假设 Foo 中有一个循环，有点在黑暗中刺伤……

int foo()
{   
    int bestval = 0;   
    char *best;   
    char *match = 0;    // initialize to null

    // start some loop
    for (int i=0;i<3;i++) {       

        // fetch the next best value...
        int returnValue = bar(&best);       
        // some check (if best is really best!)
        if (returnValue > 10) {
            // if match has previously been populated, free it
            if(match) {
                free(match);
            }
            // save the new best value
            match = best;   
        }
        else {
           // not saving best in match, so free it!
           free(best);
        }
    }
    // end some loop

    // only do this if match was successful?!?
    if(match) {
        printf("%s", match);    
        // clean up once the best of the best has been used...
        free(match);
    }
    return 0; 
} 

Answer 2

在功能栏中，strcpy 应该读作

strcpy(*best,s);

在主函数中，您可以通过

strcpy(match, best);
free(best);

匹配之前需要指向的有效内存块来复制最佳点的值。如果您进行

match = best;
free(best);

匹配也将无效，因为它指向最佳指向的同一释放内存。

Answer 3

您需要知道字符串的大小。
在 (1) 处，您将分配已释放的内存地址的地址，您必须执行另一个 malloc 来 match*=malloc(sizestr) ，然后使用 memmove 或 strcpy 复制它，如果你想要自由最好。

如果我理解正确的话，你想将字符串复制到 best 中，然后释放 bests 内存并分配 ptr 来匹配？如果在 memmoving 或 strcpying 到另一个位置之前释放最佳内存，您会丢失其内容，并且如果您想首先将其复制到另一个位置，则需要在要将其复制到的位置分配内存，因此该代码上需要 2 个 malloc 。

Answer 4

如果我必须在 (1) 而不是 (2) 处释放内存，我该怎么做才能使匹配仍然保留 best 中包含的内容？

如果您在位置 (1) 处free，则无法执行此操作，因此 match 仍将包含 中包含的内容最好的。

我应该做 strcpy 来复制最佳匹配吗？如果是这样，我是否必须在 foo 中再进行一次 malloc？

  match = best;

通过上面的语句，两者都指向同一个位置。因此，根本不需要strcpy。为此，请为 match 分配内存以指向其长度为 best+1 的位置，然后执行 strcpy。

Answer 5

复制指针的值不会复制底层内存。因此，在完成match之前不要free(best)，否则您将需要malloc一个新的缓冲区，例如memcpy() 将内容从一个缓冲区传输到另一个缓冲区。

Answer 6

是的，您可以 malloc 和 strcpy：

match = malloc(strlen(best) + 1);
strcpy(match, best);

但是，如果您的实现提供了它，您可以使用 strdup() 函数，这要容易得多：

match = strdup(best);

如果您还没有 strdup()，最好自己创建一个。

Answer 7

您当前的分配只是将指针分配给同一缓冲区。如果您随后 free() 这个缓冲区，您就删除了此处包含的内容（因此取消引用它是一个坏主意）。
您不需要使用 strcpy() 复制最佳匹配 - 您最好在 printf() （或它的最后一点）之后释放它。是需要的）。没有必要使用一个或六个额外的函数调用来使事情变得过于复杂，只需记住在每个函数末尾分配的free()内存即可！