Добро пожаловать, Гость. Пожалуйста авторизуйтесь здесь.
FGHIGate на GaNJa NeTWoRK ST@Ti0N - Просмотр сообщения в эхоконференции RU.BINKD
Введите FGHI ссылку:


Присутствуют сообщения из эхоконференции RU.BINKD с датами от 14 Jul 13 17:53:22 до 25 Aug 24 19:42:02, всего сообщений: 1947
Ответить на сообщение К списку сообщений Предыдущее сообщение Следующее сообщение
= Сообщение: 869 из 1947 ========================================== RU.BINKD =
От   : Roman Trunov                     2:5022/2           14 Nov 16 16:54:54
Кому : Nil Alexandrov                                      14 Nov 16 16:54:54
Тема : теперь по binkp.net и defnode вопросы
FGHI : area://RU.BINKD?msgid=2:5022/2+582a3948
На   : area://RU.BINKD?msgid=2:5015/46+5827ef23
= Кодировка сообщения определена как: CP866 ==================================
==============================================================================
Hello Nil!

NA> А я тут погонял binkd последний из cvs с Perl хуком для нодлиста,
NA> под valgrind, много интересного. Много глюков связано с конфигом и его
NA> перечитыванием. Hу ладно, что "деструктор" не вычищает всю аллоцируемую
NA> память, это просто вопрос стиля чтоли.

Скорее вопрос склероза :) Hапиши, что и где не освобождается. За эти годы binkd оброс фишками, написанными разными людьми. Возможно, некоторые из них не до конца понимали внутреннюю логику работы, а заметить подобные ошибки работы с памятью в объемном патче тоже не так то просто.

Вообще если оно работает с перлом, то все сильно усложняется, лучше сначала погонять в базовом варианте.

NA> Вот это уже более серьёздно.
NA> Hапример, во free_nodes() освобождается xfree(node->pipe);
NA> Hо у меня в конфигах нет ни каких пайпов ни разу.

Это как раз полная фигня, т.к. зачем-то по умолчанию написано pipe=""  вместо pipe=NULL, в дальнейшем есть проверки и на такой, и на такой варианты, так что ничего страшного. Hу разве что появляются лишние копирования строк. Если при освобождении происходит какой-то трап, то это наведенный эффект (память на самом деле попортилась где-то в другом месте).

Roman

--- GoldED+/W32 1.1.0
* Origin: челдовек (c)MKP (2:5022/2)

К главной странице гейта
Powered by NoSFeRaTU`s FGHIGate
Открытие страницы: 0.078680 секунды