多线程死锁了，给下gdb日志，帮我分析下

jd808

happy_fish100 发表于 2016-01-11 11:40
lock住之后判断宿主指针是否为空，总感觉这样的用法不科学！（lock之前已经判断过了）

如果其他线程会释 ...

哥，没用过智能指针，现在改起来工程量有点大哟

jd808

happy_fish100 发表于 2016-01-11 11:40
lock住之后判断宿主指针是否为空，总感觉这样的用法不科学！（lock之前已经判断过了）

如果其他线程会释 ...

已经是延迟了1秒钟删除的啦，socket断开的时候，并不会真正删除掉conn数据的，只会把他表示成is_down=1，另外一个线程检查is_down==1时彻底删除数据和释放内存

happy_fish100

回复 12# jd808

如何实现的延时删除呢？
不要告诉我你在锁里延时的。
如果的确是延时时间不够，多延时一会儿呗，比如1分钟！

jd808

happy_fish100 发表于 2016-01-11 15:56
回复 12# jd808

如何实现的延时删除呢？

是这样的，程序分成主线程，检测线程，执行线程池，主线程负责socket的接入读取数据，检测线程负责检测句柄是否正常，如果用户下线，主线程能够知道，并触发关闭程序

1. void TServer::CloseEvent(Conn *conn, short events)
   
2. {
   
3.         pthread_mutex_lock(&conn->UserCMDLock);
   
4.         conn->is_down=1;
   
5.         pthread_mutex_unlock(&conn->UserCMDLock);
   
6. }

复制代码

这个主线程关闭不关别的，就只是吧句柄状态改为可关闭状态（conn->is_down=1;），检测线程每一秒钟检测句柄列表，是否存在is_down==1的，如果存在，则真正将他关闭，并且清除内存。

1. 
   
2. void TServer::AutoCloseEvent(Conn *conn)
   
3. {
   
4.         //通知模块，将这个用户下线
   
5.         if(conn==NULL)return;
   
6.         //调试
   
7.         pthread_mutex_lock(&conn->UserCMDLock);
   
8.         if(conn==NULL)return;
   
9. 
   
10.         ConnList *plist = conn->GetList();
    
11.         pthread_rwlock_wrlock(&plist->ConnListLock);
    
12.         if(conn==NULL){
    
13.                 pthread_rwlock_unlock(&plist->ConnListLock);
    
14.                 return;
    
15.         }
    
16. 
    
17.         _free_prophase(conn);//清除相关内存
    
18.         conn->GetList()->Delete(conn);//从句柄列表中删除此句柄，并且将conn置为NULL
    
19.         pthread_rwlock_unlock(&plist->ConnListLock);
    
20. 
    
21. }
    

复制代码

happy_fish100

void TServer::AutoCloseEvent(Conn *conn)中，有个问题请注意下：
只看到了 pthread_mutex_lock(&conn->UserCMDLock); 没有看到对应的unlock。

另外一个关键问题，检测线程每一秒钟检测执行，这个不能保证至少延迟1s释放，可能刚设置完is_down，检测线程就将其释放了。

zylthinking

谁能告诉我这个函数里面有谁在 if(send_conn==NULL)return -1;   通过后，能够将局部变量 send_conn 置为 NULL

int TEngine::Base_Client_Lock(Conn *send_conn,const char* command,int size, int is_zip)
{
        if(send_conn==NULL)return -1;
        pthread_mutex_lock(&send_conn->UserCMDLock);
        if(send_conn==NULL)
                return -1;

        if(send_conn->is_down==1){
                pthread_mutex_unlock(&send_conn->UserCMDLock);
                return -1;
        }

        int mid=proto->MODLUE_ID<0?proto->MODLUE_ID:9999999;
        if (Client_BuildPact(mid,m_sbuff, command,size, is_zip) == -1)
        {
                pthread_mutex_unlock(&send_conn->UserCMDLock);
                return -1;
        }
        socket_send(send_conn, m_sbuff->buff,m_sbuff->data_size);
        pthread_mutex_unlock(&send_conn->UserCMDLock);
        return 0;
}

jd808

happy_fish100 发表于 2016-01-11 18:39
void TServer::AutoCloseEvent(Conn *conn)中，有个问题请注意下：
只看到了 pthread_mutex_lock(&conn- ...

确实是不能保证1秒钟，AutoCloseEvent中的unlock没有是没问题的，因为先锁住句柄，然后删除，此时锁已经没了～～，conn已经是NULL啦，这个时候再用pthread_mutex_unlock(&conn->...就会出现段错误呀。