C 当函数返回字符数组指针时，Valgrind会发出抱怨

我猜这是一个新手C问题，但我就是找不到答案。这是我的代码：

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

char *copy(char *in) {
    char *out = (char *) malloc(strlen(in)+1);
    strcpy(out,in);
    return out;
}

int main() {    
    char *orig = (char *) malloc(100);
    strcpy(orig,"TEST");
    printf("Original reads : %s\n",orig);
    printf("Copy reads     : %s\n",copy(orig));
}

---

有人能告诉我我做错了什么吗？提前谢谢

您正在

copy

函数中分配内存块，但从未释放它。因此，您的代码会产生内存泄漏，这是Valgrind报告的。您的代码的正确版本应该是

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

char *copy(char *in) {
    char *out = malloc(strlen(in)+1);
    //You need to check malloc result here!
    strcpy(out,in);
    return out;
}

int main() {    
    char *orig = (char *) malloc(100);
    strcpy(orig,"TEST");
    printf("Original reads : %s\n",orig);
    char* copy = copy(orig); 
    printf("Copy reads     : %s\n",copy);
    free(orig);
    free(copy);
    return 0;
}

#包括
#包括
#包括
字符*复制（字符*输入）{
char*out=malloc（strlen（in）+1）；
//你需要在这里检查malloc结果！
strcpy（out，in）；
返回；
}
int main（）{
char*orig=（char*）malloc（100）；
strcpy（原“测试”）；
printf（“原始读取：%s\n”，原始）；
字符*副本=副本（原件）；
printf（“复制读取：%s\n”，复制）；
免费（原版）；
免费（副本）；
返回0；
}

---

另外

您正在

复制

功能中分配内存块，但从未释放它。因此，您的代码会产生内存泄漏，这是Valgrind报告的。您的代码的正确版本应该是

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

char *copy(char *in) {
    char *out = malloc(strlen(in)+1);
    //You need to check malloc result here!
    strcpy(out,in);
    return out;
}

int main() {    
    char *orig = (char *) malloc(100);
    strcpy(orig,"TEST");
    printf("Original reads : %s\n",orig);
    char* copy = copy(orig); 
    printf("Copy reads     : %s\n",copy);
    free(orig);
    free(copy);
    return 0;
}

#包括
#包括
#包括
字符*复制（字符*输入）{
char*out=malloc（strlen（in）+1）；
//你需要在这里检查malloc结果！
strcpy（out，in）；
返回；
}
int main（）{
char*orig=（char*）malloc（100）；
strcpy（原“测试”）；
printf（“原始读取：%s\n”，原始）；
字符*副本=副本（原件）；
printf（“复制读取：%s\n”，复制）；
免费（原版）；
免费（副本）；
返回0；
}

另外，你要玩两次malloc（），但不要释放它们。因此，valgrind抱怨泄漏。 释放您分配的内存块，应该可以

   char *p = copy(orig);
    printf("Copy reads     : %s\n", p);
    free(orig);
    free(p);

您正在malloc（）两次，但不释放它们。因此，valgrind抱怨泄漏。 释放您分配的内存块，应该可以

   char *p = copy(orig);
    printf("Copy reads     : %s\n", p);
    free(orig);
    free(p);

---

谢谢好了，谢谢！修好了，修好了！还感谢您提供有关强制转换malloc（）结果的建议，以及注意到缺少的返回0。已修复！还感谢您提供有关强制转换malloc（）结果的建议，以及注意到缺少的返回0。