关于定时器崩溃的问题

rainfish

我想记录定时器任务的指针，如下操作

// 创建任务对象
        CTimerTask<TYPE>* pTask = new CTimerTask<TYPE>( 0 );
        ACE_ASSERT( pTask );
        
        if ( pTask == NULL )
        {
            ACE_ERROR( (LM_ERROR,
                ACE_TEXT ("<CTimerManager::regist_period_cycle_task> new start CTimerTask failed!\n")) );
            
            return -1;
        }

        

         pTask->m_fnTask = pFun;
         pTask->m_sTime = ACE_Time_Value(startTime);
         pTask->m_interval = ACE_Time_Value(cycleSecond, cycleMillisecond);
        pTask->m_stData.data = data;
        
        // 注册任务
        pTask->m_lID = m_timer.schedule( pTask, (void*)&(pTask->m_stData),
            pTask->m_sTime, pTask->m_interval );
        
        if ( pTask->m_lID <= 0 )
        {
            ACE_ERROR( (LM_ERROR,
                ACE_TEXT ("<CTimerManager::regist_single_task> schedule start time task failed(ID=%l)!\n"), pTask->m_lID) );

            delete pTask;
            pTask = NULL;
            return 0;
        }
       m_mapTimer.insert(pair<long, void*>(pTask->m_lID, (void*)pTask));

有时执行m_mapTimer.insert(pair<long, void*>(pTask->m_lID, (void*)pTask));时程序崩溃，提示访问越界，很纳闷，我只是保存一个指针，怎么能访问越界呢？

wishel

m_mapTimer是什么类型？

rainfish

m_mapTime是map<long, void*> ,

rainfish

而且把m_mapTimer.insert(pair<long, void*>(pTask->m_lID, (void*)pTask));
放在
CTimerTask<TYPE>* pTask = new CTimerTask<TYPE>( 0 );
        ACE_ASSERT( pTask );
        
        if ( pTask == NULL )
        {
            ACE_ERROR( (LM_ERROR,
                ACE_TEXT ("<CTimerManager::regist_period_cycle_task> new start CTimerTask failed!\n")) );
            
            return -1;
        }

        m_mapTimer.insert(pair<long, void*>(pTask->m_lID, (void*)pTask));//放在这里没有问题

         pTask->m_fnTask = pFun;
         pTask->m_sTime = ACE_Time_Value(startTime);
         pTask->m_interval = ACE_Time_Value(cycleSecond, cycleMillisecond);
        pTask->m_stData.data = data;
         //放在这里也没有问题
        //m_mapTimer.insert(pair<long, void*>(pTask->m_lID, (void*)pTask));
        // 注册任务
        pTask->m_lID = m_timer.schedule( pTask, (void*)&(pTask->m_stData),
            pTask->m_sTime, pTask->m_interval );
        
所以推断是m_timer.schedule的问题，可是一个指针，怎么会崩溃呢？

winston

表现未必代表本质。我个人意见，可能跟线程同步有关，是线程同步导致。
因为没有更多代码，所以精确结论不好说。

rainfish

线程同步的问题？m_mapTimer.insert(pair<long, void*>(pTask->m_lID, (void*)pTask));只是保存一个地址啊，就算线程同步，保存一个地址也能引起崩溃？请问原因是什么？请达人释疑，感激不尽

winston

多追踪一下就可以了。这种问题很好调试。
你看看崩溃时候的数据是不是有问题。

newzai

if ( pTask->m_lID <= 0 )
        {
            ACE_ERROR( (LM_ERROR,
                ACE_TEXT ("<CTimerManager::regist_single_task> schedule start time task failed(ID=%l)!\n"), pTask->m_lID) );

            delete pTask; # 在这里你把pTask给删除了。
            pTask = NULL;
            return 0;
        }
       m_mapTimer.insert(pair<long, void*>(pTask->m_lID, (void*)pTask)); # 在这里你使用了pTask，难道不会有问题吗？？？不崩溃才怪

修改
    if ( pTask->m_lID <= 0 )
        {
            ACE_ERROR( (LM_ERROR,
                ACE_TEXT ("<CTimerManager::regist_single_task> schedule start time task failed(ID=%l)!\n"), pTask->m_lID) );

            delete pTask; # 在这里你把pTask给删除了。
            pTask = NULL;
            return 0;
       } else
            m_mapTimer.insert(pair<long, void*>(pTask->m_lID, (void*)pTask));

winston

看起来还真是如此。

wishel

原帖由 newzai 于 2008-11-29 21:01 发表
if ( pTask->m_lID m_lID) );

            delete pTask; # 在这里你把pTask给删除了。
            pTask = NULL;
            return 0;
        }
       m_mapTimer.insert(pair(pTask->m_lID, (void*)pTask)); ...

这个分支没走到，如果走到的话下面有个return0，不会往下走了