你在滥用inet_aton()
.
您正在将返回值分配inet_aton()
给您的sockaddr
变量,而不是分配输出的in_addr
结构inet_aton()
。 返回值不是 IP 地址!
inet_aton()
成功返回非零值,失败返回 0。换句话说,它的返回值基本上是一个布尔值。
使用(aka )bind()
的 IP 地址调用指示它将套接字绑定到指定端口上的所有本地网络接口,这对于服务器来说是非常常见的(并且通常是首选)。0.0.0.0
INADDR_ANY
AF_INET
调用inet_aton("localhost")
失败,因为您无法将主机名传递给inet_aton()
,因此它返回0,然后您使用设置为的bind()
套接字,这没关系,因此您的代码似乎可以使用,但只是侥幸。sockaddr_in
s_addr
INADDR_ANY
"localhost"
调用inet_aton("127.0.0.1")
orinet_aton("192.168.30.1")
成功,因此它返回非零,然后您使用设置为无效 IP 的套接字,导致完全bind()
失败,或者更糟的是绑定到与您期望的不同的现有网络接口.sockaddr_in
s_addr
bind()
此外,in_addr
那个inet_aton()
输出已经是网络字节顺序,所以不需要调用htonl()
它的输出。
试试这个:
struct in_addr ip;
#define HOST_PORT atoi(argv[2])
#define HOST_IP ((inet_aton(argv[1], &ip) != 0) ? ip.s_addr : 0)
struct sockaddr_in sockaddr;
memset(&sockaddr, 0, sizeof(sockaddr));
sockaddr.sin_family = AF_INET;
sockaddr.sin_addr.s_addr = HOST_IP;
sockaddr.sin_port = htons(HOST_PORT);
// use sockaddr as needed ...
或者,干脆去掉你的HOST_IP
和HOST_PORT
宏,因为它们并没有真正帮助你:
struct sockaddr_in sockaddr;
memset(&sockaddr, 0, sizeof(sockaddr));
sockaddr.sin_family = AF_INET;
inet_aton(argv[1], &(sockaddr.sin_addr));
sockaddr.sin_port = htons(atoi(argv[2]));
// use sockaddr as needed ...
话虽这么说,你真的应该使用getaddrinfo()
,例如:
struct addrinfo hints, *res;
memset(&hints, 0, sizeof(hints));
hints.ai_flags = AI_PASSIVE;
hints.ai_family = AF_INET;
hints.ai_socktype = SOCK_STREAM;
hints.ai_protocol = IPPROTO_TCP;
if (getaddrinfo(argv[1], argv[2], &hints, &res) == 0)
{
// use res->ai_addr and res->ai_addrlen with socket() and bind() as needed ...
freeaddrinfo(res);
}
getaddrinfo()
会将 IP 地址字符串和主机名字符串解析为适合sockaddr_...
您的结构,并愉快地处理与本地主机相关的字符串,如"localhost"
, "127.0.0.1"
,"192.168.30.1"
等。