gelraen
14.12.2011 20:07 imax
Мать моя женщина, как только земля таких носит... http://code.google.com/p/linux-eoip/sour...
55 comments
recommend
bookmark
subscribeDo you really want to delete ?
- within threads
- within tags
- within users
что не так?
во-первых дебильнейшая модификация GRE, но это не к нему. во-вторых — очень многозначительный коментарий, явно демонстрирующий то, что автор даже не пытался ознакомится с RFC1701. в третьих — каша начиная с 340-й строчки вместо использования функций преобразования byte order.
не знаю ни что такое GRE, ни что за RFC1701, и не очень понял что там с 340
нахуя спрашивал тогда? %)
интересно стало :) То ты жалуешься что в магазине работать хуёво, а то вдруг теперь сишный код критикуешь. Вот я и заинтересовался.
ну я не в магазине работаю таки :) и таки пишу на сях почучуть :) вот сегодня как раз подсунули задачку реализовать mikrotik'овский EoIP для FreeBSD, потому и рассматриваю сабж, собсно
кстати, на 5 malloc ни единого free — это ок? В том числе тот самый ip. Или я что-то непонимаю.
зачем ты трогаешь труп? Брось бяку
кто труп?
FreeBSD
FreeBSD живее всех живых ;)
в консольных утилитах — да. Всё равно ОС прибьёт всё сожранное процессом после выхода. А так он почём зря будет свою кучу телепать.
FreeBSD труп. С такими её друзьями и недруги не нужны. Это я про менеджмент и разработчиков
а, тогда ок. Я думал это нечто долгоживущее.
только вот это не консольная утилита, а демон перебрасывающий данные между raw ip socket и tun-интерфейсом
на деле похуй. Память аллоцируется нечасто и вся нужна до окончания работы программы. А что, slice'ов ядро FreeBSD не знает?
а это кстати похуй. Главное, что память аллоцируется нечасто, и вся без исключения нужна, пока работает программа.
а у freebsd есть менеджмент? ок
Зая, у любого opensource проекта есть менеджер. Честно-честно.
slice — это что?
по-моему не освобождение памяти в функции, которая может выполнятся в цикле — это грех больший, чем дебильные комментарии и ковбойское присваивание IP адреса.
там не функция не в цикле, а в треде. и у неё внутри while(1){}
и там не IP адрес, а кусок заголовка GRE забивается побайтно
Это опечатка. Я про splice'ы спросить хотел http://en.wikipedia.org/wiki/Splice_(sys...
ага, вижу. Один хуй некузяво, но это во мне говорит опыт avrоебства.
по-моему похуй
ок
похуй когда рамы дохуя. :) после avr-а все тянет вылизать и дискомфортно от такого.
таки да, не знает
ребята, не мутите воду. Вся выделенная память нужна до конца работы программы. А потом её убивать смысла нету, ОС на завершение процессе всё уебёт
я ж про это ничо и не говорю, успокойся
да никто ничего не мутит. просто я на этот код смотрел меньше чем ты, и не знаю нафиг оно надо. Вот и всё. И я рад за современные ОС, что у них нету утечек памяти процессов при их смерти.
А потом получим лишний мусор в выводе детектора утечек, ну оооок.
Ctrl+F "malloc", пять мест, код тривиальный, чо тут думать?
пиздишь косой. Читай код!
я не про malloc. Забей.
ты про valgrind? :)
и не про валгринд.
Я теряюсь уже в догадках. Не томи
нет никаких догадках. Не зная архитектуры сего поделия и даже не представляя нафиг оно надо, я высказал мысль, что в общем случае память надо освобождать. Потому как функция, которая выделяет память может вызываться дохуя раз. @gelraen же, покурив этот код достаточно, знает, что функция одна на тред, поэтому пох. Вот и всё.
блять, тут кода всего ничего. Просмотреть основные ветки и логику вызовов тредов — это от силы две минуты.
Порождается n тредов, n зависит от cli options, в каждом треде выделяется память, не в цикле.
На всю программу один цикл, от 1 до n, где n константа.
Хули тут может быть непонятногО? Мы с тобьой общаемся дольше, чем я этот код смотрел.
в другие файлы не смотрел, да. с GRE тоже не имел дела, как eoip. Прости пожалуйста :D.
кстати с pthreads в C тоже.
там значимого один файл билять. @gelraen, потверди мои наблюдения что ле
я с GRE и eoip тоже делов не имел. Обычный сишный код, идиоматичный. Я такого читал уже много и часто.
это классно. Однако не посмотрев на другие файлы я был в этом не уверен. Ты посмотрел — и знал, какой файл главный. Че ты добиваешься?
а я в сырцы на сях редко гляжу последнее время. Тем более для сети и в linux. Для меня код относительно нов. Сетевое на сях писал последний раз лет 8 назад.
суууука, ты травишь что ле?
Там не системных заголовков два
#include "minIni.h"
#include "config.h"
ini файлы и конфиги.
а почему ты психуешь? Может уже спать пора, или чаю попить, отдохнуть?
я просто подозреваю, что меня троллят, а я ведусь. От того у меня и бугурт!
да успокойся. Не троллят. Тема закрыта, я всё понял, ты всё понял, все довольны, ок?
да конечно, мужик, какие вопросы :)
название поста ок — #ohhoee :)
Конечно, то, что менеджмент у фри адекватный и разработчики не плюсобыдло, а нормальные си-кодеры, это необычно для OSS, но нельзя это назвать минусом
пишет программы на C