提问人： 提问时间：9/20/2008 最后编辑：3 revs, 2 users 90%Bee 更新时间：5/4/2013 访问量：19495
函数调用中的 Malloc 似乎在返回时被释放了？
Malloc inside a function call appears to be getting freed on return?
问：
我想我已经把它归结为最基本的情况：
int main(int argc, char ** argv) {
int * arr;
foo(arr);
printf("car[3]=%d\n",arr[3]);
free (arr);
return 1;
}
void foo(int * arr) {
arr = (int*) malloc( sizeof(int)*25 );
arr[3] = 69;
}
输出如下：
> ./a.out
car[3]=-1869558540
a.out(4100) malloc: *** error for object 0x8fe01037: Non-aligned pointer
being freed
*** set a breakpoint in malloc_error_break to debug
>
如果有人能阐明我的理解失败的地方，我们将不胜感激。
答：
6赞
terminus
9/20/2008
#1
您已在 foo 中分配了 arr，但该指针值存储在调用堆栈中。如果你想这样做，请像这样做：
void foo( int ** arr) {
*arr = (int *)malloc( sizeof(int) * 25 );
(*arr)[3] = 69;
}
在 main 中，只需传递指向 foo 的指针（如 foo（&arr））
47赞
Nils Pipenbrinck
9/20/2008
#2
你通过值传递指针，而不是通过引用传递指针，所以无论你在 foo 中用 arr 做什么，都不会在 foo 函数之外产生任何影响。 正如m_pGladiator所写的，一种方法是声明对指针的引用（顺便说一句C++只能这样。C 不知道引用）：
int main(int argc, char ** argv) {
int * arr;
foo(arr);
printf("car[3]=%d\n",arr[3]);
free (arr);
return 1;
}
void foo(int * &arr ) {
arr = (int*) malloc( sizeof(int)*25 );
arr[3] = 69;
}
另一种（更好的恕我直言）方法是不将指针作为参数传递，而是返回指针：
int main(int argc, char ** argv) {
int * arr;
arr = foo();
printf("car[3]=%d\n",arr[3]);
free (arr);
return 1;
}
int * foo(void ) {
int * arr;
arr = (int*) malloc( sizeof(int)*25 );
arr[3] = 69;
return arr;
}
您可以将指针传递给指针。这是通过引用传递的 C 方式。使语法有点复杂，但很好 - 这就是 C 的样子......
int main(int argc, char ** argv) {
int * arr;
foo(&arr);
printf("car[3]=%d\n",arr[3]);
free (arr);
return 1;
}
void foo(int ** arr ) {
(*arr) = (int*) malloc( sizeof(int)*25 );
(*arr)[3] = 69;
}
评论
0赞
freespace
9/20/2008
我可以说，请不要转换malloc的返回值吗？它不是必需的，可以隐藏错误。
0赞
Nils Pipenbrinck
9/21/2008
@freespace，这是我多年来养成的习惯。然后我必须通过静态代码分析工具发送我的代码，他们抱怨没有转换指针。
0赞
freespace
9/21/2008
很抱歉让你挂在没有回复的情况下：除非我轮询我所有的评论，否则没有好办法在评论中找到回复:)我已经在 stackoverflow.com/questions/108768/ 中留下了答案......
0赞
Nils Pipenbrinck
9/21/2008
@freespace，我知道 - 希望有一天能添加一个通过评论通知的功能。
0赞
user1876508
6/4/2013
前两种方法中哪一种是使用函数分配内存的最快方法？
0赞
Bill K
9/20/2008
#3
如果参数 （arr） 的值不是通过引用 （&） 传入的，则无法更改该值。通常，您需要返回指针，因此您的方法应为：
arr=foo（）;
尝试重新分配参数是不好的 juju;我不推荐 （&） 解决方案。
3赞
Andrew Stein
9/20/2008
#4
Foo 接收 int 指针的本地副本，为其分配内存，并在超出范围时泄漏该内存。
解决此问题的一种方法是让 foo 返回指针：
int * foo() {
return (int*) malloc( sizeof(int)*25 );
}
int main() {
int* arr = foo();
}
另一种方法是将 foo 指针传递给指针
void foo(int ** arr) {
*arr = malloc(...);
}
int main() {
foo(&arr);
}
在 C++ 中，修改 foo 以接受对指针的引用更简单。在 C++ 中，您唯一需要的更改是将 foo 更改为
void foo(int * & arr)
1赞
Alexander
9/20/2008
#5
由于 u 是按值传递指针，因此 main 内的 arr 指针不会指向分配的内存。这意味着两件事：你自己有内存泄漏（不，函数 foo 完成后内存没有释放），当你访问 main 内的 arr 指针时，你正在访问一些任意范围的内存，因此你没有得到 3 打印出来，因此 free（） 拒绝工作。你很幸运，在main中访问arr[3]时没有遇到分段错误。
上一个：是什么导致了堆栈溢出？
下一个：如何记录 mallocs
评论