форумы  помощь  поиск  регистрация  майллист  ВХОД  слежка

Вариант для распечатки		Архивированная нить - только для чтения!  Пред. тема | След. тема
Форумы Программирование под UNIX (Public)
Изначальное сообщение		[Проследить за развитием треда]

"Не пойму"
Сообщение от dooman on 20-Янв-04, 01:32  (MSK)
Поскажите пожалуйста почему не работает эта маленькая функция???!!!: 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)); }
	Рекомендовать в FAQ | Cообщить модератору | Наверх

Индекс форумов | Темы | Пред. тема | След. тема

Сообщения по теме

1. "Не пойму"
Сообщение от asso on 20-Янв-04, 18:24  (MSK)
Она не работает потому что ты man inet_pton не прочитал.  Помимо того что эта функция в таком виде не имеет смысла, у тебя еще переполнение буфера есть. В случае ошибки inet_pton возвращает 0.  В случае неправильно заданного протокола - отрицательное число.  В случае успеха - положительное число. В зависимости от протокола параметром dst должен быть указатель на структуру struct in_addr или struct in6_addr.  Подробности читай в мане.
	Рекомендовать в FAQ | Cообщить модератору | Наверх

	2. "Не пойму"
	Сообщение от dooman on 20-Янв-04, 20:04  (MSK)
	>Она не работает потому что ты man inet_pton не прочитал.  Помимо >того что эта функция в таком виде не имеет смысла, у >тебя еще переполнение буфера есть. > >В случае ошибки inet_pton возвращает 0.  В случае неправильно заданного протокола >- отрицательное число.  В случае успеха - положительное число. В >зависимости от протокола параметром dst должен быть указатель на структуру struct >in_addr или struct in6_addr.  Подробности читай в мане. Ой блин...видимо я просто пересидел, если честно то только проснулся, но dst не обязательно должен быть указатель на структуру например return(dst) и потом вывод его приводят к корректному результату а вот inet_aton там да там только указатель на саму структуру
		Рекомендовать в FAQ | Cообщить модератору | Наверх

	3. "Не пойму"
	Сообщение от asso on 21-Янв-04, 05:51  (MSK)
	>Ой блин...видимо я просто пересидел, если честно то только проснулся, но dst >не обязательно должен быть указатель на структуру например return(dst) и потом >вывод его приводят к корректному результату а вот inet_aton там да >там только указатель на саму структуру В функции inet_pton параметр dst - указатель на структуру.  Обязательно.  Прочитай man inet_pton раз мне не веришь.
		Рекомендовать в FAQ | Cообщить модератору | Наверх

	4. "Не пойму"
	Сообщение от dooman on 21-Янв-04, 17:26  (MSK)
	>>Ой блин...видимо я просто пересидел, если честно то только проснулся, но 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); У меня в мане так... вроде указатель на переменну пустого типа
		Рекомендовать в FAQ | Cообщить модератору | Наверх

	5. "Не пойму"
	Сообщение от asso on 21-Янв-04, 19:04  (MSK)
	man надо читать до конца.  Читай дальше, там через два абзаца про параметр dst написано.  Еще ниже написано про то что возвращает эта функция.  Первые два абзаца то же прочитать не забудь.  На раздел BUGS обрати внимание.  По ссылкам в SEE ALSO походи для общего развития.  Короче, прочитай его целиком и убедись что все именно так как я описал с самого начала.
		Рекомендовать в FAQ | Cообщить модератору | Наверх

	6. "Не пойму"
	Сообщение от dooman on 21-Янв-04, 20:15  (MSK)
	>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 закоментированых не работает
		Рекомендовать в FAQ | Cообщить модератору | Наверх

	7. "Не пойму"
	Сообщение от asso on 22-Янв-04, 07:36  (MSK)
	Учи английский, потому что без английского невозможно быть программистом. Да, это _иногда_ работает, но только потому что 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.
		Рекомендовать в FAQ | Cообщить модератору | Наверх

	8. "Не пойму"
	Сообщение от dooman on 22-Янв-04, 16:54  (MSK)
	>Учи английский, потому что без английского невозможно быть программистом. > >Да, это _иногда_ работает, но только потому что 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 я так и не понял ну и хрен с ним. Спасибо за всё!!!
		Рекомендовать в FAQ | Cообщить модератору | Наверх

Удалить

Индекс форумов | Темы | Пред. тема | След. тема

Пожалуйста, прежде чем написать сообщение, ознакомьтесь с данными рекомендациями.