linux select 返回-1 errno是Bad address

liqingfang

现在linux平台用select还是存在的，这个只是测试代码，但应该没问题，出错的原因老是select调用返回-1,errno的信息是Bad address,但select之前的bind是没有报错的，怎么会是Bad address呢，望各位高手指点。谢谢！

1.         struct sockaddr_in myname,cliaddr;
   
2.         fd_set fdread;
   
3.         struct timeval timeout;
   
4.         pid_t pid;
   
5.         socklen_t addrlen;
   
6.         int new_sock = 0;
   
7.         myname.sin_family = AF_INET;
   
8.         myname.sin_port = htons(7878);
   
9.         bzero(&myname.sin_zero,8);
   
10.         inet_aton("192.168.1.216",&myname.sin_addr);       
    
11.        
    
12.        
    
13. 
    
14.         int sock = socket(AF_INET,SOCK_STREAM,0);
    
15.         if(sock<0)
    
16.         {
    
17.                 syslog(LOG_ERR,"can not create socket");
    
18.                 return -1;        
    
19.             }
    
20. 
    
21. 
    
22.         fcntl(sock,F_SETFL,O_NONBLOCK);
    
23. 
    
24.         int flag = 0;
    
25.         if(setsockopt(sock,SOL_SOCKET,SO_REUSEADDR,(char *)&flag,sizeof(flag))<0)
    
26.             {
    
27.                 syslog(LOG_ERR,"can not set sock opt");
    
28.                 return -1;
    
29.             }
    
30.        
    
31. 
    
32.         syslog(LOG_INFO,"before bind errno is %s",strerror(errno));       
    
33. 
    
34.         if(bind(sock,(struct sockaddr *)&myname,sizeof(myname))<0)
    
35.             {
    
36.                  syslog(LOG_ERR,"can not bind addr");
    
37.                  return -1;
    
38.             }
    
39.         syslog(LOG_INFO,"after bind  errno is %s",strerror(errno));        
    
40.             if(listen(sock,7)<0)
    
41.             {
    
42.                 syslog(LOG_ERR,"can not listen");
    
43.                 return -1;
    
44.             }
    
45.          syslog(LOG_INFO,"after listen  errno is %s",strerror(errno));
    
46.             //fd_set  fdread;
    
47.         syslog(LOG_INFO,"Listening for connections on port %d\n",htons(myname.sin_port));
    
48. 
    
49. 
    
50.                
    
51.         while(1)       
    
52.         {
    
53.                 while(1)
    
54.                 {
    
55.                
    
56.                        FD_ZERO(&fdread);
    
57.                       FD_SET(sock,&fdread);
    
58.                        
    
59.                         timeout.tv_sec = 0;
    
60.                         timeout.tv_usec = 500000;
    
61.                
    
62.                         syslog(LOG_INFO,"before select errno is %s",strerror(errno));       
    
63.                          if(!FD_ISSET(sock,&fdread))
    
64.                         {
    
65.                                 syslog(LOG_INFO,"fd is not set");
    
66.                         }
    
67. 
    
68.                         int reval = select(sock+1,&fread,NULL,NULL,&timeout);
    
69.                         //syslog(LOG_INFO,"select get val is %d\n",reval);
    
70. //                        printf("errno is %d\n",errno);
    
71. //                        perror("perror");
    
72. //                        printf("sterror:%s\n",strerror(errno));
    
73. 
    
74.                 if(reval<0)
    
75.                 {
    
76.                         syslog(LOG_INFO,"Error");
    
77.                         syslog(LOG_INFO,"errno is %s",strerror(errno));       
    
78.                 }
    
79.                 else if(reval==0)
    
80.                 {
    
81.                         syslog(LOG_INFO,"Wait----");       
    
82.                         continue;
    
83.                 }
    
84.                 else
    
85.                 {
    
86.                         syslog(LOG_INFO,"(((((()))))))");
    
87.                        
    
88.                         new_sock = accept(sock,0,0);
    
89.        
    
90.                 ------
    
91. ｝
    
92. ｝       

复制代码

log输出为：

hellioncu

每次select之前都要
         FD_ZERO(&fdread);

         FD_SET(sock,&fdread);

hellioncu

另外select中用两个&fread可能也是不对的，得用两个fd_set

liqingfang

之前我就是写在while循环里面的，现在改了，测试，也不行，

liqingfang

回复 3# hellioncu


    fd是在文件描述符集合中增加一个新的文件描述符，其实是不用的另外再添加一个fd_set（sock，&fdread），我已经修改了代码，还是不能通过，请指教，这个问题困扰很长时间了

hellioncu

回复  hellioncu


    fd是在文件描述符集合中增加一个新的文件描述符，其实是不用的另外再添加一个fd ...
liqingfang 发表于 2011-06-08 11:55

    int reval = select(sock+1,&fread,NULL,NULL,&timeout);

你写错了，应该是fdread{:3_184:}

liqingfang

哎呦，我的妈啊，怎么还出这种低级错误，用vim编译的，谢谢楼上