2013-03-26 19 views
6
#include <stdio.h> 
    #include <string.h> /* for strncpy */ 
    #include <sys/types.h> 
    #include <sys/socket.h> 
    #include <sys/ioctl.h> 
    #include <netinet/in.h> 
    #include <net/if.h> 

    int 
    main() 
    { 
    int fd; 
    struct ifreq ifr; 

    fd = socket(AF_INET, SOCK_DGRAM, 0); 

    /* I want to get an IPv4 IP address */ 
    ifr.ifr_addr.sa_family = AF_INET; 

    /* I want IP address attached to "eth0" */ 
    strncpy(ifr.ifr_name, "eth0", IFNAMSIZ-1); 

    ioctl(fd, SIOCGIFADDR, &ifr); 

    close(fd); 

    /* display result */ 
    char* ipaddr; 
    ipaddr = inet_ntoa(((struct sockaddr_in *)&(ifr.ifr_addr))->sin_addr); 
    printf("%s\n", ipaddr); 

    return 0; 
    } 

dla tej linii:winy segmentacji dla inet_ntoa

 ipaddr = inet_ntoa(((struct sockaddr_in *)&(ifr.ifr_addr))->sin_addr);   

uzyskać

iptry.c: In function ‘main’: 
iptry.c:31:9: warning: assignment makes pointer from integer without a cast [enabled by default] 

i

 printf("%s\n", ipaddr); 

otrzymuję winy segmentacji.

Co jest nie tak z tym?

+2

w linux, 'inet_ntoa' jest zdefiniowane w nagłówku' ', być może trzeba' # include' że? – Petesh

+1

Czy otrzymujesz także __limplicit function declaration_ warning? – hmjd

+0

co rozumiesz przez domyślną deklarację funkcji? – user138126

Odpowiedz

19

inet_ntoa jest zdefiniowany w nagłówku <arpa/inet.h>,
potrzebę #include, w przeciwnym razie nie będzie błędów

+4

W jaki sposób kompilator nie wygenerował ostrzeżenia, jeśli nie zawiera prawego nagłówka? – Nikkolasg

1

inet_ntoa nie wymaga wskaźnik, ale wartość

char* ipaddr; 
ipaddr = inet_ntoa(((struct sockaddr_in)(ifr.ifr_addr))->sin_addr); 

Jeśli sin_addr jest wskaźnikiem, trzeba dereference niego.

inet_ntoa powróci NULL jeśli wystąpi błąd, więc próbuje printf NULL będzie cuase usterki segmentacji ...

Look here do informacji i here za roboczogodzinę stronie.

+0

jak zmodyfikować kod? – user138126

+0

Tak jak napisałem w mojej odpowiedzi i jeśli pojawią się błędy, przeczytaj dokumentację, ZAWSZE! –

+0

'sin_addr' nie jest wskaźnikiem i wszystko wydaje się być w porządku – user138126

1

Jeśli nie chcesz, aby zobaczyć ostrzeżenie, po prostu rzucać wartość zwracaną inet_ntoa

ipaddr = (char *) inet_ntoa(((struct sockaddr_in *)&(ifr.ifr_addr))->sin_addr); 

Usterka segmentacji jest prawdopodobnie dlatego, że niektóre z poprzedniej funkcji zwraca błąd (wartości null) a ty nie zrobił Sprawdź to

Ten sam kod działał w moim Linux Box.

+0

nadal nie może rozwiązać problemu. – user138126

+0

fd jest prawidłowym deskryptorem pliku? ioctl zwrócił -1 lub cokolwiek? ifr zawiera prawidłowe wartości? inet_ntoa zwrócił wskaźnik zerowy? odpowiedzieć na pytania, a następnie dostaniesz, co jest problem –

+0

fd jest poprawny, iotcl powrót 0, inet_ntoa zwraca poprawną wartość, jak sprawdzić ifr? – user138126