Код работает, но сжирает всю доступную память

[calypso]

Программа в режиме реального времени отображает скорость и трафик на сетевых картах. В каком месте идёт утечка?

private: System::Void timer1_Tick(System::Object^  sender, System::EventArgs^  e) {
			 //получение текущей даты и времени в реальном режиме
				 SYSTEMTIME currentdate;
				 GetSystemTime(&currentdate);
				 String ^datanow=Convert::ToString(currentdate.wDay)+"."+Convert::ToString(currentdate.wMonth)+"."+Convert::ToString(currentdate.wYear);
				 String ^vremyanow=Convert::ToString(currentdate.wHour+4)+":"+Convert::ToString(currentdate.wMinute)+":"+Convert::ToString(currentdate.wSecond);
				label9->Text=datanow+" , "+vremyanow;
			//получение скоростей сетевых адаптеров				
				 time1=GetTickCount();
				 MIB_IF_TABLE2 *pIfTable2;
				 MIB_IF_ROW2 *pIfRow2;
				 pIfTable2=(MIB_IF_TABLE2*) MALLOC(sizeof(MIB_IF_TABLE2));
				 if (pIfTable2==NULL) listBox2->Items->Add("Ошибка при выделении памяти под GetIfTable2!");
				 //инициализация GetIfTable2Ex чтобы получить размер dwSize
				 dwSize=sizeof(MIB_IF_TABLE2);

				 if (GetIfTable2Ex(MibIfTableNormal,&pIfTable2)==ERROR_NOT_ENOUGH_MEMORY)
					 {
						 FREE(pIfTable2);
						 pIfTable2=(MIB_IF_TABLE2*) MALLOC(dwSize);
						 if (pIfTable2==NULL)	 listBox2->Items->Add("Ошибка выделения памяти MIB_IF_TABLE2");
				 }
					 //второй вызов GetIfTable2Ex который должен завершиться без ошибок
				 	 if (GetIfTable2Ex(MibIfTableNormal,&pIfTable2)==NO_ERROR) 
						 {
							 if (pIfTable2->NumEntries>0)
							 {
								pIfRow2=(MIB_IF_ROW2*) MALLOC(sizeof(MIB_IF_ROW2));
								if (pIfRow2==NULL)
								{
									listBox2->Items->Add("Ошибка выделения памяти MIB_IF_ROW2");
									if (pIfRow2!=NULL)
									{
										FREE(pIfTable2);
										pIfTable2=NULL;
									}
								}
							 }
					   }
					 else 
						 {
							 oshibka=GetLastError();
							 listBox2->Items->Add("Ошибка при вызове функции GetIfTable2Ex: "+oshibka.ToString());
					     }
					 //перебираем все записи в таблице
					 for (i=0;i<(int)pIfTable2->NumEntries;i++)
					 {
						 pIfRow2->InterfaceIndex=pIfTable2->Table[i].InterfaceIndex;
						 SecureZeroMemory((PVOID)pIfRow2, sizeof(MIB_IF_ROW2));
			pIfRow2->InterfaceLuid = pIfTable2->Table[i].InterfaceLuid;
			if ((dwRetVal = GetIfEntry2(pIfRow2)) == NO_ERROR)
			{
				if ((pIfRow2->Alias != NULL) && (pIfRow2->Type == IF_TYPE_ETHERNET_CSMACD) && (pIfRow2->PhysicalMediumType == NdisPhysicalMedium802_3) && (pIfRow2->OperStatus == IfOperStatusUp) && (pIfRow2->InterfaceAndOperStatusFlags.HardwareInterface == 1))
				{
					double down1=Convert::ToDouble((pIfRow2->InOctets)/1024.0/1024.0);
					double up1=Convert::ToDouble((pIfRow2->OutOctets)/1024.0/1024.0);
					//интерфейс маршрутизатора NETGEAR WNR3500L
					if (pIfRow2->InterfaceIndex==router1number)
					{
					label16->Text= msclr::interop::marshal_as<String^>(pIfRow2->Alias);
					label22->Text= msclr::interop::marshal_as<String^>(pIfRow2->Description);
					label11->Text="Down: " + (pIfRow2->InOctets / 1024/1024).ToString() + " МБ";
					label12->Text="Up: " + (pIfRow2->OutOctets / 1024/1024).ToString() + " МБ";
					if (counter>0)
							{
								if (time1==time2) time1=time1+1;
								label18->Text=("DownSpeed: "+(((pIfRow2->InOctets-download1)/(time1-time2)).ToString()));
							    label19->Text=("UpSpeed: "+(((pIfRow2->OutOctets-upload1)/(time1-time2)).ToString()));
							}
							download1=pIfRow2->InOctets;
							upload1=pIfRow2->OutOctets;
							time2=GetTickCount();
							counter++;
					}
					//интерфейс маршрутизатора TP-LINK TL-1043ND
					if (pIfRow2->InterfaceIndex==router2number)
					{
					label17->Text= msclr::interop::marshal_as<String^>(pIfRow2->Alias);
					label23->Text= msclr::interop::marshal_as<String^>(pIfRow2->Description);
					label14->Text="Down: " + (pIfRow2->InOctets / 1024/1024).ToString() + " МБ";
					label15->Text="Up: " + (pIfRow2->OutOctets / 1024/1024).ToString() + " МБ";
					if (counter>0)
							{
								if (time1==time2) time1=time1+1;
								label20->Text=("DownSpeed: "+(((pIfRow2->InOctets-download2)/(time1-time2)).ToString()));
							    label21->Text=("UpSpeed: "+(((pIfRow2->OutOctets-upload2)/(time1-time2)).ToString()));
							}
							download2=pIfRow2->InOctets;
							upload2=pIfRow2->OutOctets;
							time2=GetTickCount();
							counter++;
					}

				}
		
			 }
	}
			}

[Pavia]

У вас слишком длинный и страшный код. Не уверено, что кто-то захочет в нем копаться.
А так на вскидку у вас MALLOC 3 штуки, а FREE только 2 штуки.
С учётом того что эти строчки выполняются только в случае ошибки.

									if (pIfRow2!=NULL)
									{
										FREE(pIfTable2);
										pIfTable2=NULL;
									}

То можно сказать что утечка заключается в отсутствии освобождении памяти.
Всю память которую вы выделяете надо освобождать.

[calypso]

А глобальной функции нет, чтобы всё очистить после каждого цикла таймера? По сути дела ведь он например каждые 10 мс обновляется, старые то данные уже никому не нужны.

[Пепел Феникса]

С/С++ не будет за вами подтирать.
что взяли то обратно и положите.

[calypso]

Короче добавление

FREE(pIfTable2);
	 FREE(pIfRow2);

ничего не дает, ни после MALLOC, ни в конце обработчика Timer.
Пробовал ещё удалять

SecureZeroMemory((PVOID)pIfRow2, sizeof(MIB_IF_ROW2));

, без неё все работает. Но память как жрал, так и продолжает жрать. Расход примерно 1-2 МБ в секунду.

[waleri]

Вас не настораживает, что GetIfTable2Ex НЕ требует указать размер буфера?
Гляньте в документацию, насчет того, что делать с буфером, когда он больше не нужен.

[calypso]

Добавил

FreeMibTable(pIfTable2);
	FreeMibTable(pIfRow2);

в конце обработчика Timer1 , ничего не изменилось.