64,644
社区成员
发帖
与我相关
我的任务
分享【求助】C++子线程访问自己的局部指针变量无效
请教各位大神一个C++多线程编程的问题。
有一个线程函数，这个函数是一个类里面的，我在这个线程函数里面申请了buffer这个局部指针变量，这个函数中间还要调用这个类的其他两个子函数，这个buffer进入第一个子函数的时候，buffer正常，但是进入第二个的时候，就报错。
代码如下：
在这里DepthToMat函数的时候，buffer正常，能够被这个子函数处理
到这里Cov这个函数的时候，buffer就好像失效了，每次一访问就出错
这是两个子函数，和这个线程同属于一个类，depthbuffer和colorbuffer名字无所谓，因为两个都会用到，两处都会报错。
还请大神们不吝赐教，谢谢！
有一个线程函数，这个函数是一个类里面的，我在这个线程函数里面申请了buffer这个局部指针变量，这个函数中间还要调用这个类的其他两个子函数，这个buffer进入第一个子函数的时候，buffer正常，但是进入第二个的时候，就报错。
代码如下：
在这里DepthToMat函数的时候，buffer正常，能够被这个子函数处理
到这里Cov这个函数的时候，buffer就好像失效了，每次一访问就出错
这是两个子函数，和这个线程同属于一个类，depthbuffer和colorbuffer名字无所谓，因为两个都会用到，两处都会报错。
还请大神们不吝赐教，谢谢！
...全文
请发表友善的回复…
发表回复
有点贪玩 2018-02-28
- 打赏
- 举报
你的两个buffer指针初始值是nullptr 后续赋值是在哪？
apaulette 2018-02-28
- 打赏
- 举报
谢谢赵老师指点迷津。我自己先学者调试看看。
赵4老师 2018-02-27
- 打赏
- 举报
http://blog.csdn.net/zhao4zhong1/article/details/53078924 老司机找bug的十年心路历程
赵4老师 2018-02-27
- 打赏
- 举报
崩溃的时候在弹出的对话框按相应按钮进入调试，按Alt+7键查看Call Stack即“调用堆栈”里面从上到下列出的对应从里层到外层的函数调用历史。双击某一行可将光标定位到此次调用的源代码或汇编指令处，看不懂时双击下一行，直到能看懂为止。
赵4老师 2018-02-27
- 打赏
- 举报
有时不将“调用函数名字＋各参数值，进入函数后各参数值，中间变量值，退出函数前准备返回的值，返回函数到调用处后函数名字＋各参数值＋返回值”这些信息写日志到文件中是无论如何也发现不了问题在哪里的，包括捕获各种异常、写日志到屏幕、单步或设断点或生成core或dmp文件、……这些方法都不行！ 写日志到文件参考下面：
//循环向a函数每次发送200个字节长度（这个是固定的）的buffer,
//a函数中需要将循环传进来的buffer，组成240字节（也是固定的）的新buffer进行处理，
//在处理的时候每次从新buffer中取两个字节打印
#ifdef _MSC_VER
#pragma warning(disable:4996)
#endif
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#ifdef _MSC_VER
#include <windows.h>
#include <process.h>
#include <io.h>
#define MYVOID void
#define vsnprintf _vsnprintf
#else
#include <unistd.h>
#include <sys/time.h>
#include <pthread.h>
#define CRITICAL_SECTION pthread_mutex_t
#define MYVOID void *
#endif
//Log{
#define MAXLOGSIZE 20000000
#define MAXLINSIZE 16000
#include <time.h>
#include <sys/timeb.h>
#include <stdarg.h>
char logfilename1[]="MyLog1.log";
char logfilename2[]="MyLog2.log";
static char logstr[MAXLINSIZE+1];
char datestr[16];
char timestr[16];
char mss[4];
CRITICAL_SECTION cs_log;
FILE *flog;
#ifdef _MSC_VER
void Lock(CRITICAL_SECTION *l) {
EnterCriticalSection(l);
}
void Unlock(CRITICAL_SECTION *l) {
LeaveCriticalSection(l);
}
void sleep_ms(int ms) {
Sleep(ms);
}
#else
void Lock(CRITICAL_SECTION *l) {
pthread_mutex_lock(l);
}
void Unlock(CRITICAL_SECTION *l) {
pthread_mutex_unlock(l);
}
void sleep_ms(int ms) {
usleep(ms*1000);
}
#endif
void LogV(const char *pszFmt,va_list argp) {
struct tm *now;
struct timeb tb;
if (NULL==pszFmt||0==pszFmt[0]) return;
vsnprintf(logstr,MAXLINSIZE,pszFmt,argp);
ftime(&tb);
now=localtime(&tb.time);
sprintf(datestr,"%04d-%02d-%02d",now->tm_year+1900,now->tm_mon+1,now->tm_mday);
sprintf(timestr,"%02d:%02d:%02d",now->tm_hour ,now->tm_min ,now->tm_sec );
sprintf(mss,"%03d",tb.millitm);
printf("%s %s.%s %s",datestr,timestr,mss,logstr);
flog=fopen(logfilename1,"a");
if (NULL!=flog) {
fprintf(flog,"%s %s.%s %s",datestr,timestr,mss,logstr);
if (ftell(flog)>MAXLOGSIZE) {
fclose(flog);
if (rename(logfilename1,logfilename2)) {
remove(logfilename2);
rename(logfilename1,logfilename2);
}
} else {
fclose(flog);
}
}
}
void Log(const char *pszFmt,...) {
va_list argp;
Lock(&cs_log);
va_start(argp,pszFmt);
LogV(pszFmt,argp);
va_end(argp);
Unlock(&cs_log);
}
//Log}
#define ASIZE 200
#define BSIZE 240
#define CSIZE 2
char Abuf[ASIZE];
char Cbuf[CSIZE];
CRITICAL_SECTION cs_HEX;
CRITICAL_SECTION cs_BBB;
struct FIFO_BUFFER {
int head;
int tail;
int size;
char data[BSIZE];
} BBB;
int No_Loop=0;
void HexDump(int cn,char *buf,int len) {
int i,j,k;
char binstr[80];
Lock(&cs_HEX);
for (i=0;i<len;i++) {
if (0==(i%16)) {
sprintf(binstr,"%03d %04x -",cn,i);
sprintf(binstr,"%s %02x",binstr,(unsigned char)buf[i]);
} else if (15==(i%16)) {
sprintf(binstr,"%s %02x",binstr,(unsigned char)buf[i]);
sprintf(binstr,"%s ",binstr);
for (j=i-15;j<=i;j++) {
sprintf(binstr,"%s%c",binstr,('!'<buf[j]&&buf[j]<='~')?buf[j]:'.');
}
Log("%s\n",binstr);
} else {
sprintf(binstr,"%s %02x",binstr,(unsigned char)buf[i]);
}
}
if (0!=(i%16)) {
k=16-(i%16);
for (j=0;j<k;j++) {
sprintf(binstr,"%s ",binstr);
}
sprintf(binstr,"%s ",binstr);
k=16-k;
for (j=i-k;j<i;j++) {
sprintf(binstr,"%s%c",binstr,('!'<buf[j]&&buf[j]<='~')?buf[j]:'.');
}
Log("%s\n",binstr);
}
Unlock(&cs_HEX);
}
int GetFromRBuf(int cn,CRITICAL_SECTION *cs,struct FIFO_BUFFER *fbuf,char *buf,int len) {
int lent,len1,len2;
lent=0;
Lock(cs);
if (fbuf->size>=len) {
lent=len;
if (fbuf->head+lent>BSIZE) {
len1=BSIZE-fbuf->head;
memcpy(buf ,fbuf->data+fbuf->head,len1);
len2=lent-len1;
memcpy(buf+len1,fbuf->data ,len2);
fbuf->head=len2;
} else {
memcpy(buf ,fbuf->data+fbuf->head,lent);
fbuf->head+=lent;
}
fbuf->size-=lent;
}
Unlock(cs);
return lent;
}
MYVOID thdB(void *pcn) {
char *recv_buf;
int recv_nbytes;
int cn;
int wc;
int pb;
cn=(int)pcn;
Log("%03d thdB thread begin...\n",cn);
while (1) {
sleep_ms(10);
recv_buf=(char *)Cbuf;
recv_nbytes=CSIZE;
wc=0;
while (1) {
pb=GetFromRBuf(cn,&cs_BBB,&BBB,recv_buf,recv_nbytes);
if (pb) {
Log("%03d recv %d bytes\n",cn,pb);
HexDump(cn,recv_buf,pb);
sleep_ms(1);
} else {
sleep_ms(1000);
}
if (No_Loop) break;//
wc++;
if (wc>3600) Log("%03d %d==wc>3600!\n",cn,wc);
}
if (No_Loop) break;//
}
#ifndef _MSC_VER
pthread_exit(NULL);
#endif
}
int PutToRBuf(int cn,CRITICAL_SECTION *cs,struct FIFO_BUFFER *fbuf,char *buf,int len) {
int lent,len1,len2;
Lock(cs);
lent=len;
if (fbuf->size+lent>BSIZE) {
lent=BSIZE-fbuf->size;
}
if (fbuf->tail+lent>BSIZE) {
len1=BSIZE-fbuf->tail;
memcpy(fbuf->data+fbuf->tail,buf ,len1);
len2=lent-len1;
memcpy(fbuf->data ,buf+len1,len2);
fbuf->tail=len2;
} else {
memcpy(fbuf->data+fbuf->tail,buf ,lent);
fbuf->tail+=lent;
}
fbuf->size+=lent;
Unlock(cs);
return lent;
}
MYVOID thdA(void *pcn) {
char *send_buf;
int send_nbytes;
int cn;
int wc;
int a;
int pa;
cn=(int)pcn;
Log("%03d thdA thread begin...\n",cn);
a=0;
while (1) {
sleep_ms(100);
memset(Abuf,a,ASIZE);
a=(a+1)%256;
if (16==a) {No_Loop=1;break;}//去掉这句可以让程序一直循环直到按Ctrl+C或Ctrl+Break或当前目录下存在文件No_Loop
send_buf=(char *)Abuf;
send_nbytes=ASIZE;
Log("%03d sending %d bytes\n",cn,send_nbytes);
HexDump(cn,send_buf,send_nbytes);
wc=0;
while (1) {
pa=PutToRBuf(cn,&cs_BBB,&BBB,send_buf,send_nbytes);
Log("%03d sent %d bytes\n",cn,pa);
HexDump(cn,send_buf,pa);
send_buf+=pa;
send_nbytes-=pa;
if (send_nbytes<=0) break;//
sleep_ms(1000);
if (No_Loop) break;//
wc++;
if (wc>3600) Log("%03d %d==wc>3600!\n",cn,wc);
}
if (No_Loop) break;//
}
#ifndef _MSC_VER
pthread_exit(NULL);
#endif
}
int main() {
#ifdef _MSC_VER
InitializeCriticalSection(&cs_log);
InitializeCriticalSection(&cs_HEX);
InitializeCriticalSection(&cs_BBB);
#else
pthread_t threads[2];
int threadsN;
int rc;
pthread_mutex_init(&cs_log,NULL);
pthread_mutex_init(&cs_HEX,NULL);
pthread_mutex_init(&cs_BBB,NULL);
#endif
Log("Start===========================================================\n");
BBB.head=0;
BBB.tail=0;
BBB.size=0;
#ifdef _MSC_VER
_beginthread((void(__cdecl *)(void *))thdA,0,(void *)1);
_beginthread((void(__cdecl *)(void *))thdB,0,(void *)2);
#else
threadsN=0;
rc=pthread_create(&(threads[threadsN++]),NULL,thdA,(void *)1);if (rc) Log("%d=pthread_create %d error!\n",rc,threadsN-1);
rc=pthread_create(&(threads[threadsN++]),NULL,thdB,(void *)2);if (rc) Log("%d=pthread_create %d error!\n",rc,threadsN-1);
#endif
if (!access("No_Loop",0)) {
remove("No_Loop");
if (!access("No_Loop",0)) {
No_Loop=1;
}
}
while (1) {
sleep_ms(1000);
if (No_Loop) break;//
if (!access("No_Loop",0)) {
No_Loop=1;
}
}
sleep_ms(3000);
Log("End=============================================================\n");
#ifdef _MSC_VER
DeleteCriticalSection(&cs_BBB);
DeleteCriticalSection(&cs_HEX);
DeleteCriticalSection(&cs_log);
#else
pthread_mutex_destroy(&cs_BBB);
pthread_mutex_destroy(&cs_HEX);
pthread_mutex_destroy(&cs_log);
#endif
return 0;
}
apaulette 2018-02-27
- 打赏
- 举报
求大神答疑解惑。。。。5555555
apaulette 2018-02-19
- 打赏
- 举报
有没有大神回答一下啊??
apaulette 2018-02-14
- 打赏
- 举报
指针越界没有，x和y都是0的时候就报错了，根本到不了后面。
释放的话，我看了一下也没有。而且如果是提前释放的话，和main函数放在一起执行就啥问题没有，偏偏是用CreateThread放到一个子线程里面执行，就一定报错。看报错感觉是不让访问。指针在，buffer里面的数据也在。
释放的话，我看了一下也没有。而且如果是提前释放的话，和main函数放在一起执行就啥问题没有，偏偏是用CreateThread放到一个子线程里面执行，就一定报错。看报错感觉是不让访问。指针在，buffer里面的数据也在。
apaulette 2018-02-13
- 打赏
- 举报
地址我单步跟过了，虽然初始化是nullptr，但始终都不是nullptr，进出子函数，地址也没有变。而且这段代码在主线程运行，是一点问题也没有的，但是把这段代码用CreateThread放到单独的子线程上运行，就会报错。就算我nullptr去掉了，也不行。nullptr这个初始化的写法也是人家微软kinect sdk里面这么写的，人家sdk sample在主线程运行也是对的。我现在就是把这段代码从主线程挪到子线程上面运行，就不行了。也不涉及同步死锁什么的，就开了一个子线程。
zilaishuichina 2018-02-13
- 打赏
- 举报
否个地方 -》某个地方
zilaishuichina 2018-02-13
- 打赏
- 举报
从你截图的这几段代码看， 只能：
1，查你报错的那个地方的 y * mDepthWidth + x 看算出来是多少，有没有越界。
2，查你报错的那个地方的pDepthBuffter是否是个空指针或者野指针，也就是在此行代码之前pDepthBuffter是不是被否个地方释放掉了
apaulette 2018-02-13
- 打赏
- 举报
老师，我现在这个只涉及一个子线程。主线程的main函数里面，只是一个CreateThread()，然后后面就while空循环，主线程不参与任何的处理。我只开了这唯一的一个子线程，这个子线程函数是GetPointCloudFrame()。现在的问题是，这个GetPointCloudFrame()函数，如果在主线程里面调用执行，一切OK，啥错误没有，但是放到子线程里面执行，就报错。GetPointCloudFrame()函数里面我又调用了两个同Class下的子函数。如果是指针是nullptr或者指针异常的话，第一个子函数DepthToMat()，是正常的，而第二个子函数CovTo()就异常了。我试过如果只调用一个子函数，OK，只要调用超过2个子函数，不管这个函数是不是我贴出来的这个CovTo()还是其他的函数，例如我紧接着再调用一次DepthToMat()，就会报这个错。我不知道是不是这个buffer指针在子线程里面只能用一次，第二次必定失效，还是我建立的这个子线程有问题。
赵4老师 2018-02-13
- 打赏
- 举报
《Windows核心编程》
《深入解析Windows操作系统-Windows Internals》
赵4老师 2018-02-13
- 打赏
- 举报
多线程访问相同内存，需要加锁。
真相重于对错 2018-02-12
- 打赏
- 举报
把值变量传入函数，函数结束后，那个变量依然是传入前的值，不会因为在函数里面的改变而改变。
同理
把指针传入函数，函数结束后，那个指针依然是传入前指向的地址，不会因为在函数里面改变的地址而改变。
比如一个函数
void fun(int *input)
{
input =(int*)malloc(sizeof(int)*10);
}
你这么调用的话
int* a=nullptr;
fun(a)；
fun结束后
a 依然是nullptr;
apaulette 2018-02-12
- 打赏
- 举报
不是空指针，先经过kinect的AccessUbderlyingBuffer，把kinect的图像帧填充到buffer里面。然后buffer就有数据了，经过DepthToMat函数，把帧转换为cv::mat，再往后和另外一个图像帧的buffer一起融合。问题就是目前我这个buffer不知道为什么好像只能被我的子函数调用一次，第二次再调用就会失效。但是不用CreateThread建立子线程运行，而直接写在main函数里在主线程运行，就没有这个问题。我也没有在子函数里把buffe释放，delete的代码。所以很奇怪。只能被 和这个线程属于用一个类(kinectGesture)的其他子函数 调用一次，再调用第二次就不行。
百合杰 2018-02-10
- 打赏
- 举报
要不把这些图换成代码文本我们再来看？
apaulette 2018-02-10
- 打赏
- 举报
这个是错误信息，每次必出现。这个和多线程同步还没有太大关系，因为我只是把这个处理函数开到子线程上面运行，不干扰主线程，子线程只开了这一个，主线程也不会访问到子线程的这个变量。
walkereklaw 2018-02-10
- 打赏
- 举报
你的意思是把空指针传进去？然后再传到别的函数？
