这边博客用于记录项目开发过程中遇到的一个bug。样例代码如下(按场景还原的代码,只是展示代码关键部分,报错无法还原),项目中会报“段错误”。
【bug|一次由于传递局部变量指针产生的bug】下面的代码主要是新建一个执行线程,这个线程会一直运行到进程结束。线程运行时会根据传递的参数执行相应的操作(结构体中的cmd字符串)。这个线程是一个ts流处理工具,是同事开发的一个模块,他封装了一个执行函数供我调用,而我根据程序运行状况组装cmd。
在启动该线程后,程序运行过程中会突然报错,而且报错的时机不固定。由于整个程序逻辑比较复杂,模块较多,无法确定发生“段错误”的位置。之后我将程序中各插件模块分离后,单独测试,可各模块处理数据都很正常。
所以,我将问题定位在创建线程这块,也就是如下代码这部分。发现是我创建线程时传递的指针参数为局部变量的指针( -.- ,直接copy同事提供的调用demo),这就很容易出现问题。因为局部变量是在栈上分配的内存,在函数执行完毕后会自行释放,此时充满了不确定性。而同事的代码中并没有将我传递的参数拷贝一份,而是一直保存着这个栈上的指针,所以发生了上述的问题。我在将需要传递的参数使用malloc分配内存空间传递给他后,这个问题就解决了,如代码中new_pthread2 函数所示。
#include
#include
#include
#include
#include typedef struct _info{
int channel_no;
int stream_no;
char cmd[1024];
}info;
char *cmd_pack()
{
char *cmd = (char *)malloc(1024);
memset(cmd, 0, 1024);
/*
cmd pack
*/
return cmd;
}void * process(void *arg)
{
pthread_detach(pthread_self());
info *params = (info *)arg;
/*
some operations with params
*/
return NULL;
}void new_pthread1()
{
pthread_t tid1;
info tmp;
tmp.channel_no = 111;
tmp.stream_no = 222;
char *cmd = cmd_pack();
strcpy(tmp.cmd, cmd);
pthread_create(&tid1, NULL, process, (void *)&tmp);
free(cmd);
}void new_pthread2()
{
pthread_t tid1;
info *tmp = (info *)malloc(sizeof(info));
tmp->channel_no = 111;
tmp->stream_no = 222;
char *cmd = cmd_pack();
strcpy(tmp->cmd, cmd);
pthread_create(&tid1, NULL, process, (void *)&tmp);
free(cmd);
}int main(void)
{
/*
some operations
*/
if (condition)
{
new_pthread1();
//new_pthread2();
}
/*
a lot of operations
*/
return 0;
}
推荐阅读
- 十万个为什么|口诀快速记住C语言中的运算符优先级
- linux|linux进程、线程常用的几个基础知识点
- C|C语言——知识点汇总
- 数组与指针进阶|指针进阶-ONE(包含对数组的加深理解)
- 数组与指针进阶|指针进阶-TWO
- 数组与指针进阶|指针进阶-THREE(有对指针和数组的加深理解)
- c/c++技术|小白 C++ 入门到大神发疯学习路线
- C++|C++应用程序列表(来自Bjarne Stroustrup)
- C|梦回童年——基于C语言实现三子棋小游戏