Документация полный отстой! Ничего не ясно :D

Ну да… нужно, кстати, посмотреть, как она правильно делается для проектов на Си.

cat Makefile
CFLAGS=-O3 -std=c99 -W -Wall -pedantic -D_GNU_SOURCE

Но почему?

	if (ipv4 == 1)
		res = inet_pton(AF_INET, address, &(sa.sin_addr));

годный валидатор! аж пельмешкой подавился

man gcc

Зато работает. Альтернативой было клепать регекспы, но имел я такое счастье в баш-скриптах на продакшене.

А почему нет?

слишком много ненужных if

res = inet_pton(ipv4 == 1? AF_INET : AF_INET6, ddress, &(sa.sin_addr));

итп

слишком много ненужных ==

res = inet_pton(ipv4 ? AF_INET : AF_INET6, address, &(sa.sin_addr));

Оп, а о тринарном забыл. Покручу, спасибо.

это религиозное :)

Альтернативой было клепать регекспы, но имел я такое счастье в баш-скриптах на продакшене.

а в чем проблема? главное - тесткейсы сделать

Ну я так подумал — есть же готовая функция, почему её не использовать.

Проапгрейдил.

Садись, Сидоров, 3

% : ping 127.1 -qc 1 && echo Alive
Alive
% ./yaiv -4 -a 127.1
127.1: Malformed IPv4 address

Может, и не велосипед

Но колеса треугольные. Открой для себя мир PCRE и сократи свой код в M раз.

PS Я вижу ты уже высказался насчет регекспов, нуууу... Что сказать? Клепай дальше в том же духе на здоровье.

Если жеват кактус долго-долго то он на вкус становится как текила :)

слишком много ненужных if

Откомпилируется одинаково же.

а, у тебя там кстати buffer overflows :)

вот тебе, на затравку, как оно должно было бы быть, если смотреть с моей колокольни: http://pastebin.com/iZZaPdEu

не глумись, братюнь

% : ping 127.0.127 -c 1 && echo 'Alive!!!'
Alive!!!

Названия констант режут слух.

Не работает.

$ ./yaiv -4 -a 0x7f.0x1
0x7f.0x1: Malformed IPv4 address

$ ping -c 1 0x7f.0x1
PING 0x7f.0x1 (127.0.0.1) 56(84) bytes of data.
64 bytes from 127.0.0.1: icmp_req=1 ttl=64 time=0.020 ms

--- 0x7f.0x1 ping statistics ---
1 packets transmitted, 1 received, 0% packet loss, time 0ms
rtt min/avg/max/mdev = 0.020/0.020/0.020/0.000 ms

И так не работает.

$ ./yaiv -4 -a 017700000001
017700000001: Malformed IPv4 address

$ ping -c 1 017700000001
PING 017700000001 (127.0.0.1) 56(84) bytes of data.
64 bytes from 127.0.0.1: icmp_req=1 ttl=64 time=0.016 ms

--- 017700000001 ping statistics ---
1 packets transmitted, 1 received, 0% packet loss, time 0ms
rtt min/avg/max/mdev = 0.016/0.016/0.016/0.000 ms

Откомпилируется одинаково же.

выглядит симпатичнее, читается лучше - имхо это тоже немаловажно

beastie

а, у тебя там кстати buffer overflows :)

Это там, где работа со строками? Можно детальнее?

Всегда кажется, что на аватаре плачет синий мишка.

у вас система не феншуйная. в линупсовом man-е чотко вписано:

       AF_INET
              src points to a character string containing an IPv4 network address in dotted-decimal format, «ddd.ddd.ddd.ddd», where ddd
              is  a decimal number of up to three digits in the range 0 to 255.

третий аргумент inet_pton:

для v4 — это struct in_addr → 32 бита

для v6 — это struct in6_addr → 128 бит

а у тебя и там и там in_addr → переполнение в случае v6

EBAD! предлагаю переименовать директивы.

в sockaddr_in после in_addr есть неиспользуемое место.

Спасибо, поправлю.

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

Мне, собсно, в другом формате валидация и не нужна. Хотя интересно, почему оно не хавает.

Хотя интересно, почему оно не хавает.

в смысле не хавает? по моему наоборот хавает все подряд.

Ну вот всякие 0x7f.0x1 и не хавает.

тогда для ipv4 пользуй inet_aton

[pf@eternity]:[~/work/devel/yaiv][0]% ./yaiv -4 -a 0x7f.0x1
0x7f.0x1 is valid IPv4 address

Хе, пашет. Merci.

Какой же это валидатор, когда нигде даже не описаны критерии валидности.

Почитай RFC или как.

ок, Сидоров, неси зачотку

От Сидорова слышу.

Почитай RFC или как. Найдешь там про форматы, расскажешь.

а чем оно лучше inet_pton ?

«Код не читай — в тред отвечай» ©.

https://www.ietf.org/rfc/rfc2373.txt

А теперь такую же для ipv4.

как скажешь, братиш
Каковы ваши дальнейшие планы? валидатор сетей? Поделись RFC если найдешь, а то был недавно тред на эту тему, но rfc там не нашли

Перечитай внимательно исходный пост треда, чтобы увидеть, что это было just for fun, поэтому какие тут планы.

А нема!

Я зачем этот hello world под GPL публиковать и по поводу него создавать тред на лоре? Я то думал, что тут мегаалгоритм, который уделывает по скорости inet_pton в N раз, а тут это ....

В исходном сообщении всё написано.