Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > C/C++: Общие вопросы > Есть ли утечка?


Автор: ksili 17.3.2009, 12:53
Ищу утечки в чужом приложении (MS VS 2008 SP1).

Суть работы приложения такова: оно ставит хук на все сообщения. Ловит все появляющиеся диалоги, сабклассит их, и при помощи сабклассинга дорисовывает на них кнопку. 

Я заметил, что конструктор класса этой кнопки трассируется столько раз, сколько было поймано таких диалогов (т.е. если таких окон было 10, то и бряк в конструкторе сработает 10 раз), а деструктор вообще не трассируется. 

Человек, который делал программу, сказал мне что это нормально. Конструктор выполняется в контексте основного процесса, т.е. того, который запустила студия. А деструктор выполняется в контексте хукнутого процесса. Так что утечки нет. Но меня что-то гложут сомнения. 
Прав ли тот программист? Или неправ? Или могут быть варианты?

Автор: xvr 17.3.2009, 15:34
Цитата(ksili @ 17.3.2009,  12:53)
Человек, который делал программу, сказал мне что это нормально. Конструктор выполняется в контексте основного процесса, т.е. того, который запустила студия. А деструктор выполняется в контексте хукнутого процесса. Так что утечки нет. 

Ой!  smile Значит память берем у одного приложения, а отдаем ее другому?  smile 
Цитата

Но меня что-то гложут сомнения. 
Правильно гложут
Цитата

Или могут быть варианты?
Могут, если основная цель приложения откачать себе памяти от других  smile 
 smile 

Автор: Earnest 17.3.2009, 22:04
А кнопка-то динамически создается, как объект? Или просто hWnd = CreateWindow(...)? Хотя, раз ты написал про конструктор...
Очевидно, что хукнутое приложение деструктор вызывать не будет, т.к. ничего про данный код не знает. При разрушении диалога просто вызовется DestroyWindow. А что мешает сабклассить диалог так, чтобы при закрытии он кнопку таки разрушал?

Автор: ksili 18.3.2009, 06:24
Всем спасибо, вы подтвердили мои опасения. 
Будем доделывать сабклассинг

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)