handle_close里加锁导致内存出错

lordoffox

问题是这样的,我在使用\ACE_wrappers\examples\APG\Reactor下的例子HAStatus时在

2. int
3. ClientService::handle_close (ACE_HANDLE, ACE_Reactor_Mask mask)
4. {
5.   if (mask == ACE_Event_Handler::WRITE_MASK)
6.     return 0;
7.   lock_.acquire(); //增加这里
8.   mask = ACE_Event_Handler::ALL_EVENTS_MASK |
9.          ACE_Event_Handler::DONT_CALL;
10.   this->reactor ()->remove_handler (this, mask);
11.   this->sock_.close ();
12.   this->output_queue_.flush ();
13.   delete this;
14.   lock_.release();//在增加这里
15.   return 0;
16. }

复制代码

本来是想改成TP_Reactor防止多次删除,但是奇怪的问题发生了
在Client第二次连接的时候发生错误

HEAP[HAStatus.exe]: HEAP: Free Heap block 96c110 modified at 96c190 after it was freed
Unhandled exception at 0x7c94a3e1 in HAStatus.exe: User breakpoint.

跟踪进去也没发现错误,继续运行结果仍然正常,请大家为我解疑

我测试的平台
Windows XP SP2
Vc++ 2003
ACE 5.6.0

thks

peakzhang

delete this;
lock_.release();//在增加这里

改成
lock_.release();//在增加这里
delete this;
试试

lordoffox

谢谢，问题找到了，但是没办法解决呀，ACE能保证handle_close只被调用一次吗，因为只要delete this后，handle_close只要能重入就一定会出错呀

winston

先仔细研究Reactor的机制，想想看。