这几天重构以块代码,其中有一个功能是将新创建的对象发回,遇到一个奇葩的问题,实例代码如下:
GsStreamingPad* //A.c
gs_encoder_create_streaming_pad(Encoder* enc)
{
GsStreamingPad* spad;
gchar name[32];
sprintf(name, "spad_%02d", g_slist_length(enc->streaming_pads));
spad = gs_streamingpad_new(name);
//这里能正常访问s_name变量
printf("Create StreamingPad %p->name=%s", spad, spad->s_name);
return spad;
}
int main( ...) //B.c注意这个main调用在另一个文件中
{
... ...
GsStreamingPad* spad = gs_encoder_create_streaming_pad(encoder);
/*
* 问题在这个地方,程序前几次能正常访问,但是偶尔程序就奔溃了
*/
printf("Create StreamingPad %p->name=%s", spad, spad->s_name);
... ...
return 0;
}
看到这块代码我当时就无语了,心里那个叫奇怪啊!根本不知道怎么回事,明明好好的逻辑怎么就不能用了呢,“见鬼了”?
后面继续追追了好几天,无果。那个头顿时感觉大了一圈,为什么呢?
后面加打印发现如下问题
[A.c::gs_encoder_create_streaming_pad::11]create StreamingPad 0x7f3bdc001990->name=spad_02
[B.c::main::8] Created streaming_pad=0xffffffffdc001990 0xffffffffdc001990
看到没有64位的指针函数返回后尽然截断了:从 0x7f3bdc001990 变为 0xffffffffdc001990, 指针前32位被重置了。
好吧,总算是找到点眉目了,于是继续编译,耐心的看编译的log,居然又发现如下一句话:
......
warning: initialization makes pointer from integer without a cast
......
看到了,编译器居然说我的函数返回的是一个 integer,“天哪”编译器果真瞎了吗?我不是明明指的是指针吗?
后经检查,才发现原来我在头文件中没有声明这个函数,只是在C文件中实现了,编译器生成了默认声明,并默认返回值为 integer。
修改A.h,添加如下声明:
GsStreamingPad* gs_encoder_create_streaming_pad(Encoder* enc);
问题搞定!!!
告诫自己和我的盆友们:不要忽视编译器的任何抱怨,否则会死的很惨。