Не пойму, dooman, 20-Янв-04, 01:32 [смотреть все]Поскажите пожалуйста почему не работает эта маленькая функция???!!!: int inet_pton_loose(const char *addr,int af) {struct in_addr k; int dst; if(af==AF_INET){
if( inet_pton(AF_INET,addr,&dst) ==0) { return(inet_aton(addr,&k)); } } if(af==AF_INET6){ if( inet_pton(AF_INET6,addr,&dst) ==0) { return(inet_aton(addr,&k)); } } return(inet_pton(af,addr,&dst)); } |
- Не пойму, asso, 18:24 , 20-Янв-04 (1)
Она не работает потому что ты man inet_pton не прочитал. Помимо того что эта функция в таком виде не имеет смысла, у тебя еще переполнение буфера есть.В случае ошибки inet_pton возвращает 0. В случае неправильно заданного протокола - отрицательное число. В случае успеха - положительное число. В зависимости от протокола параметром dst должен быть указатель на структуру struct in_addr или struct in6_addr. Подробности читай в мане.
- Не пойму, dooman, 20:04 , 20-Янв-04 (2)
>Она не работает потому что ты man inet_pton не прочитал. Помимо >того что эта функция в таком виде не имеет смысла, у >тебя еще переполнение буфера есть. > >В случае ошибки inet_pton возвращает 0. В случае неправильно заданного протокола >- отрицательное число. В случае успеха - положительное число. В >зависимости от протокола параметром dst должен быть указатель на структуру struct >in_addr или struct in6_addr. Подробности читай в мане. Ой блин...видимо я просто пересидел, если честно то только проснулся, но dst не обязательно должен быть указатель на структуру например return(dst) и потом вывод его приводят к корректному результату а вот inet_aton там да там только указатель на саму структуру
- Не пойму, asso, 05:51 , 21-Янв-04 (3)
>Ой блин...видимо я просто пересидел, если честно то только проснулся, но dst >не обязательно должен быть указатель на структуру например return(dst) и потом >вывод его приводят к корректному результату а вот inet_aton там да >там только указатель на саму структуру В функции inet_pton параметр dst - указатель на структуру. Обязательно. Прочитай man inet_pton раз мне не веришь.
- Не пойму, dooman, 17:26 , 21-Янв-04 (4)
>>Ой блин...видимо я просто пересидел, если честно то только проснулся, но dst >>не обязательно должен быть указатель на структуру например return(dst) и потом >>вывод его приводят к корректному результату а вот inet_aton там да >>там только указатель на саму структуру > >В функции inet_pton параметр dst - указатель на структуру. Обязательно. >Прочитай man inet_pton раз мне не веришь. inet_pton(3) Linux Programmer's Manual inet_pton(3)NAME inet_pton - Create a network address structure SYNOPSIS #include <sys/types.h> #include <sys/socket.h> #include <arpa/inet.h> int inet_pton(int af, const char *src, void *dst); У меня в мане так... вроде указатель на переменну пустого типа
- Не пойму, asso, 19:04 , 21-Янв-04 (5)
man надо читать до конца. Читай дальше, там через два абзаца про параметр dst написано. Еще ниже написано про то что возвращает эта функция. Первые два абзаца то же прочитать не забудь. На раздел BUGS обрати внимание. По ссылкам в SEE ALSO походи для общего развития. Короче, прочитай его целиком и убедись что все именно так как я описал с самого начала.
- Не пойму, dooman, 20:15 , 21-Янв-04 (6)
>man надо читать до конца. Читай дальше, там через два абзаца >про параметр dst написано. Еще ниже написано про то что >возвращает эта функция. Первые два абзаца то же прочитать не >забудь. На раздел BUGS обрати внимание. По ссылкам в >SEE ALSO походи для общего развития. Короче, прочитай его целиком >и убедись что все именно так как я описал с самого >начала. Cорри, может я английский и не очень хорошо знаю(так и не понял приводит он его к вду структуры и копирует в dst или как по дугому) но вот этот код работает вполне корректно: #include <sys/socket.h> #include <netinet/in.h> #include <arpa/inet.h> #include <sys/types.h> #include <stdlib.h> #include <stdio.h>int inet_pton_loose(const char *addr,int af) { struct in_addr *k; int dst; if(af==AF_INET){
if( inet_pton(AF_INET,addr,&dst) ==0) { printf("inet_pton==0\n"); inet_aton(addr,k); return(k->s_addr); } } if(af==AF_INET6){ if( inet_pton(AF_INET6,addr,&dst) ==0) { printf("inet_pton==0\n"); inet_aton(addr,k); return(k->s_addr); } } return(dst); } int main(int argc,char *argv[]) { if(argc<1){ printf("No arguments\n"); exit(1); } //if(argv[2]=="AF_INET") // printf("%i\n",inet_pton_loose(argv[1],AF_INET)); //if(argv[2]=="AF_INET6") // printf("%i\n",inet_pton_loose(argv[1],AF_INET6)); printf("%i\n",inet_pton_loose(argv[1],AF_INET)); return 0; } Правда 2 if'a закоментированых не работает
- Не пойму, asso, 07:36 , 22-Янв-04 (7)
Учи английский, потому что без английского невозможно быть программистом.Да, это _иногда_ работает, но только потому что sizeof(int) == sizeof(struct in_addr). Если ввести адрес с ошибкой, то в зависимости от внешних условий программа может выпасть в кору. Если inet_pton вернул 0 то в случае AF_INET функция inet_aton то же всегда вернет 0. В случае AF_INET6 inet_aton может сработать если адрес был IPv4. Помимо этого у тебя происходит запись в случайную область памяти и переполнение буфера. В случае IPv6 твоя функция в любом случае вернет бессмысслицу, потому что в IPv6 адрес в int не влезет. Кроме того, адрес - это беззнаковое целое а ты его используешь как знаковое, из-за чего могут быть проблемы позже. Если тебе надо преобразовать адрес IPv4 то пиши так: u_int32_t inet_pton_loose(const char *addr) { struct in_addr k; if (inet_pton(AF_INET, addr, &k) > 0) return k.s_addr; else return 0; } Для IPv6 аналогично только структура будет in6_addr. И не забывай что в IPv6 адрес занимает 16 байт. Сравнение строк в Си делается при помощи функции strcmp. Читай man strcmp.
- Не пойму, dooman, 16:54 , 22-Янв-04 (8)
>Учи английский, потому что без английского невозможно быть программистом. > >Да, это _иногда_ работает, но только потому что sizeof(int) == sizeof(struct in_addr). > Если ввести адрес с ошибкой, то в зависимости от внешних >условий программа может выпасть в кору. Если inet_pton вернул 0 >то в случае AF_INET функция inet_aton то же всегда вернет 0. > В случае AF_INET6 inet_aton может сработать если адрес был IPv4. > Помимо этого у тебя происходит запись в случайную область памяти >и переполнение буфера. В случае IPv6 твоя функция в любом >случае вернет бессмысслицу, потому что в IPv6 адрес в int не >влезет. Кроме того, адрес - это беззнаковое целое а ты >его используешь как знаковое, из-за чего могут быть проблемы позже. > >Если тебе надо преобразовать адрес IPv4 то пиши так: > >u_int32_t inet_pton_loose(const char *addr) >{ > struct in_addr k; > if (inet_pton(AF_INET, addr, &k) > 0) > return k.s_addr; > else > return 0; >} > >Для IPv6 аналогично только структура будет in6_addr. И не забывай что >в IPv6 адрес занимает 16 байт. > >Сравнение строк в Си делается при помощи функции strcmp. Читай man >strcmp. ааааа...спасибо. Да про строки я чё то забыл а inet_aton мне нужен был чтобы обабатывать неправильные адреса(например 123.5) которые inet_pton не может. Только inet_aton работает только с IPv4 как я понял а как преобразовать адрес IPv4 k IPv6 я так и не понял ну и хрен с ним. Спасибо за всё!!!
|