1

Добрый день.

Есть СМТП-виндовс сервис, который слушает 25ый порт на сервере. Я исследую его на утечки памяти, и вот что обнаружил - клиентский поток не закрывается вообще.

Код ниже:

  /// <summary>
  /// Запуск сервиса
  /// </summary>
        protected override void OnStart(string[] args)
        {
            EventLog.WriteEntry( "[ " + DateTime.Now.ToString() +  "] Сервис запущен: " + ServiceName);
            WriteLog("Сервис запущен: " + ServiceName);

            try
            {
                new Thread(() =>
                {
                    Listen();
                    WriteLog(">>Listen thread is finished! <<<");
                }).Start();

            }
            catch (Exception ex)
            {
                EventLog.WriteEntry( ServiceName + " error: \n" + ex.ToString());
            }        
        }

/// <summary>
/// Основной цикл сервиса - подключение клиента и работа с ним
/// </summary>
/// <param name="temp"></param>
public void Listen(object temp = null)
{
    LogHelper _logHelper = new LogHelper();                    

    try
    {
        WriteLog("***************START*******************");
        WriteLog("SMTP server started " + DateTime.Now.ToString());

        SMTP_Listener = new TcpListener(IPAddress.Any, 25);
        SMTP_Listener.Start();

        while (true)
        {
           // Этот Клиентский поток не закрывается !!!!
            new Thread(() =>
            {
                testA();
                WriteLog(">> Client thread is finished! <<<");
            }).Start();

        }
    }
    catch (ThreadInterruptedException e)
    {
    }
    catch (Exception ex)
    {
    }
}

//содержимое клиентского потока:

  void testA()
        {
            int[] arr = new int[999999999];
            for (int i = 0; i < arr.Length; i++)
            {
                arr[i] = 99999999;
            }
            WriteLog(">> array is filled <<<");
        }

Я всего-лишь в потоке заполняю массив, и после того как он заполнится, поток должен записать в файл 'array is filled', после чего должна записаться запись в файл ">> Client thread is finished! <<<", но поток не закрывается и память (по диспетчеру) не освобождается.

Что я делаю неправильно? Подскажите пожалуйста, почему поток не закрывается?

Спасибо

UPD Оказывается, если в потоке закоментировать цикл:

/*int[] arr = new int[999999999];
                for (int i = 0; i < arr.Length; i++)
                {
                    arr[i] = 99999999;
                }
*/

то поток закрывается, т.к. в логи пишется ">> array is filled <<<" и " Client thread is finished! <<<"

1 ответ 1

2

while (true) new Thread(...).Start(); - бесконечно запускаешь потоки.


Потому что чтобы 1 поток даже из 2000 (а их там больше может быть) потоков досчитал до миллиарда, надо более 5 часов. И это я ещё время на переключение между потоками не учёл. Можно это на что-то ещё умножить. В общем, запусти на недельку, если ни один поток не завершится, можно будет подумать.

А ещё не факт, что там создаётся массив нужного размера - может и памяти не хватить.

9
  • В этом коде я жду клиентские подключения 22 мар 2016 в 9:25
  • 1
    @LeonardBertone, вы спросили ПОЧЕМУ, вам ответили почему
    – tCode
    22 мар 2016 в 9:33
  • 1
    @LeonardBertone: Вы в цикле запускаете бесконечно много потоков без паузы. Код new Thread(...).Start() не дожидается окончания потока, только запускает его. В итоге у системы не хватает ресурсов, и, думаю, программа падает.
    – VladD
    22 мар 2016 в 10:15
  • 1
    @LeonardBertone, потому что чтобы 1 поток даже из 2000 (а их там больше может быть) потоков досчитали до миллиарда, надо более 5 часов. И это я ещё время на переключение между потоками не учёл. Можно это на что-то ещё умножить. В общем, запусти на недельку, если ни один поток не завершится, можно будет подумать.
    – Qwertiy
    22 мар 2016 в 10:17
  • 1
    @Qwertiy: И поскольку Exception глотается, то об этом никто и не узнает.
    – VladD
    22 мар 2016 в 10:20

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.