= Сообщение: 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у разве что появляются лишние копирования строк. Если при освобождении происходит какой-то трап, то это наведенный эффект (память на самом деле попортилась где-то в другом месте).