теперь по binkp.net и defnode вопросы

Roman Trunov написал(а) к Vitaliy Aksyonov в Nov 16 14:10:18 по местному времени:

Нello Vitaliy!

RT>> ПРЭЛЭЭСТНО! Теперь понятно, почему оно регулярно валится. Дело в том,
RT>> что раньше в gethost_andport было в параметрах unsigned short *port,
RT>> и *port = find_port() возвращал число - номер порта, никто ни на кого
RT>> не ссылался и все освобождалось где надо. Потом кто-то полез
RT>> переделывать и gethost_andport за каким-то хреном стал возвращать
RT>> порт в виде строки. Подозреваю, что это произошло или в момент
RT>> вкорячивания пайпов, или ipv6.

VA> Да уж... Бывают досадные ошибки... Патч пришлешь? :)

Я не знаю, кто это сломал, а также зачем и почему там вообще потребовалась возвращать строку. Разбираться надо.

Roman

--- GoldED+/W32 1.1.0

Roman Trunov написал(а) к Nil Alexandrov в Nov 16 16:54:54 по местному времени:

Нello Nil!

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

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

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

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

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

Roman

--- GoldED+/W32 1.1.0

Vitaliy Aksyonov написал(а) к Roman Trunov в Nov 16 00:03:22 по местному времени:

Привет, Roman!

14 ноя 16 14:10, Roman Trunov -> Vitaliy Aksyonov:

RT> Нello Vitaliy!

RT>>> ПРЭЛЭЭСТНО! Теперь понятно, почему оно регулярно валится. Дело в
RT>>> том, что раньше в gethost_andport было в параметрах unsigned
RT>>> short port, и port = find_port() возвращал число - номер
RT>>> порта, никто ни на кого не ссылался и все освобождалось где
RT>>> надо. Потом кто-то полез переделывать и gethost_andport за
RT>>> каким-то хреном стал возвращать порт в виде строки. Подозреваю,
RT>>> что это произошло или в момент вкорячивания пайпов, или ipv6.

VA>> Да уж... Бывают досадные ошибки... Патч пришлешь? :)

RT> Я не знаю, кто это сломал, а также зачем и почему там вообще
RT> потребовалась возвращать строку. Разбираться надо.

Судя по истории:
- Port information is stored as string now, i.e. may be service name

С наилучшими пожеланиями, Vitaliy.

... 10.0 times 0.10 is hardly ever 1.00.
--- GoldED+/LNX 1.1.5-b20160201

Roman Trunov написал(а) к Vitaliy Aksyonov в Nov 16 11:21:10 по местному времени:

Нello Vitaliy!

RT>> Я не знаю, кто это сломал, а также зачем и почему там вообще
RT>> потребовалась возвращать строку. Разбираться надо.

VA> Судя по истории:
VA> - Port information is stored as string now, i.e. may be service name

Т.е. до сих пор тянутся хвосты от ipv6 и поддержки SRV, которое было вкорячено туда молотком, если мне не изменяет память, в общем то сторонним разработчиком и далеко не безболезненно...

Roman

--- GoldED+/W32 1.1.0

Vitaliy Aksyonov написал(а) к Roman Trunov в Nov 16 12:35:40 по местному времени:

Привет, Roman!

15 ноя 16 11:21, Roman Trunov -> Vitaliy Aksyonov:

RT>>> Я не знаю, кто это сломал, а также зачем и почему там вообще
RT>>> потребовалась возвращать строку. Разбираться надо.

VA>> Судя по истории:
VA>> - Port information is stored as string now, i.e. may be service
VA>> name

RT> Т.е. до сих пор тянутся хвосты от ipv6 и поддержки SRV, которое было
RT> вкорячено туда молотком, если мне не изменяет память, в общем то
RT> сторонним разработчиком и далеко не безболезненно...

А в чем конкретно там проблема? Я посмотрел, быстро не нашел, от чего оно может
падать при перечитке конфига.

С наилучшими пожеланиями, Vitaliy.

... 10.0 times 0.10 is hardly ever 1.00.
--- GoldED+/LNX 1.1.5-b20160201

Roman Trunov написал(а) к Vitaliy Aksyonov в Nov 16 14:50:34 по местному времени:

Нello Vitaliy!

VA>>> Судя по истории:
VA>>> - Port information is stored as string now, i.e. may be service
VA>>> name

RT>> Т.е. до сих пор тянутся хвосты от ipv6 и поддержки SRV, которое было
RT>> вкорячено туда молотком, если мне не изменяет память, в общем то
RT>> сторонним разработчиком и далеко не безболезненно...

VA> А в чем конкретно там проблема? Я посмотрел, быстро не нашел, от чего оно
VA> может падать при перечитке конфига.

Порча памяти. Как минимум одно место уже нашли. Это уже надо срочно править. И как вариант - при перечитке конфига как раз происходит много выделений/освобождений, вот оно и валится. Может, конечно, и какой-то элемент конфига некорректно обрабатывается, но сначала надо убрать эту бяку.

Сделать что-ли форк от точки, где еще нет ipv6, с cherrypick багфиксами...

Roman

--- GoldED+/W32 1.1.0

Vitaliy Aksyonov написал(а) к Roman Trunov в Nov 16 14:19:04 по местному времени:

Привет, Roman!

15 ноя 16 14:50, Roman Trunov -> Vitaliy Aksyonov:

VA>>>> Судя по истории:
VA>>>> - Port information is stored as string now, i.e. may be service
VA>>>> name

RT>>> Т.е. до сих пор тянутся хвосты от ipv6 и поддержки SRV, которое
RT>>> было вкорячено туда молотком, если мне не изменяет память, в
RT>>> общем то сторонним разработчиком и далеко не безболезненно...

VA>> А в чем конкретно там проблема? Я посмотрел, быстро не нашел, от
VA>> чего оно может падать при перечитке конфига.

RT> Порча памяти. Как минимум одно место уже нашли. Это уже надо срочно
RT> править. И как вариант - при перечитке конфига как раз происходит
RT> много выделений/освобождений, вот оно и валится. Может, конечно, и
RT> какой-то элемент конфига некорректно обрабатывается, но сначала надо
RT> убрать эту бяку.

RT> Сделать что-ли форк от точки, где еще нет ipv6, с cherrypick
RT> багфиксами...

Лучше починить проблемы с IPv6, если они есть, чем выпиливать его совсем... Все-таки востребовано может быть.

С наилучшими пожеланиями, Vitaliy.

... 10.0 times 0.10 is hardly ever 1.00.
--- GoldED+/LNX 1.1.5-b20160201

Alexandr Kruglikov написал(а) к Vitaliy Aksyonov в Nov 16 15:35:02 по местному времени:

Привет, Vitaliy!

Ответ на сообщение Vitaliy Aksyonov (2:463/3232) к Roman Trunov, написанное 15 ноя 16 в 14:19:

VA> Лучше починить проблемы с IPv6, если они есть, чем выпиливать его
VA> совсем... Все-таки востребовано может быть.

Востребовано =)

[root@srv-msk ~]# host fido.kruglikov.info
fido.kruglikov.info is an alias for kruglikov.info.
kruglikov.info has address 37.46.135.107
kruglikov.info has IPv6 address 2a01:230:2:6::ae3

С уважением - Alexandr
--- GoldED+/W64-MSVC 1.1.5-b20160322

Vladimir Donskoy написал(а) к Alexandr Kruglikov в Nov 16 16:14:16 по местному времени:

Нello Alexandr!

15 ноя 16 15:35, Alexandr Kruglikov wrote to Vitaliy Aksyonov:

VA>> Лучше починить проблемы с IPv6, если они есть, чем выпиливать его
VA>> совсем... Все-таки востребовано может быть.

AK> Востребовано =)

AK> [root@srv-msk ~]# host fido.kruglikov.info
AK> fido.kruglikov.info is an alias for kruglikov.info.
AK> kruglikov.info has address 37.46.135.107
AK> kruglikov.info has IPv6 address 2a01:230:2:6::ae3

А вот и нет: раз есть адрес IPv4 - то IPv6 становится не нужен, связь можно и по-старому установить.

Есть ли у кого-то нода, отвечающая только по IPv6? Если нет - то и в самом деле стоит форкнуть, чтобы не плодить баги в неиспользуемом софте.

С уважением, Vladimir Donskoy.

--- GoldED+/W32-MSVC 1.1.5-b20160322

Peter Khanin написал(а) к Alexandr Kruglikov в Nov 16 21:32:28 по местному времени:

Здpавствуй, Alexandr!

Вторник 15 Ноября 2016 15:35, ты писал(а) Vitaliy Aksyonov, в сообщении по ссылке area://ru.binkd?msgid=2:5053/58.2+582b0178:

VA>> Лучше починить проблемы с IPv6, если они есть, чем выпиливать его
VA>> совсем... Все-таки востребовано может быть.

AK> Востребовано =)

AK> [root@srv-msk ~]# host fido.kruglikov.info
AK> fido.kruglikov.info is an alias for kruglikov.info.
AK> kruglikov.info has address 37.46.135.107
AK> kruglikov.info has IPv6 address 2a01:230:2:6::ae3

Оно даже работает :)

+ 12 Nov 12:06:02 [27045] incoming session with mail.homelabs.org [2607:5300:100:200::d88]

С уважением - Peter
--- GoldED-NSF/W32-MINGW 1.1.5-20090710