From 03532b5f4120cbc79e68cb26b00a4aa999862202 Mon Sep 17 00:00:00 2001 From: "mallinath@webrtc.org" Date: Wed, 21 Dec 2011 15:36:44 +0000 Subject: [PATCH] Fixing the double delete problem in UdpSocket2ManagerWindow. PopFront deletes the items, to there is no need to delete item explicitly. Review URL: http://webrtc-codereview.appspot.com/333014 git-svn-id: http://webrtc.googlecode.com/svn/trunk@1268 4adac7df-926f-26a2-2b94-8c16560cd09d --- .../udp_transport/source/udp_socket2_manager_windows.cc | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/modules/udp_transport/source/udp_socket2_manager_windows.cc b/src/modules/udp_transport/source/udp_socket2_manager_windows.cc index de6e47e06..4003b54a4 100644 --- a/src/modules/udp_transport/source/udp_socket2_manager_windows.cc +++ b/src/modules/udp_transport/source/udp_socket2_manager_windows.cc @@ -68,8 +68,6 @@ UdpSocket2ManagerWindows::~UdpSocket2ManagerWindows() UdpSocket2WorkerWindows* pWorker; while((pItem = _workerThreadsList.First()) != NULL) { - pWorker = (UdpSocket2WorkerWindows*)pItem->GetItem(); - delete pWorker; _workerThreadsList.PopFront(); } @@ -217,8 +215,6 @@ bool UdpSocket2ManagerWindows::StartWorkerThreads() UdpSocket2WorkerWindows* pWorker; while((pItem = _workerThreadsList.First()) != NULL) { - pWorker = (UdpSocket2WorkerWindows*)pItem->GetItem(); - delete pWorker; _workerThreadsList.PopFront(); } _pCrit->Leave();