android系统native层的getsockopt调用有错吗

wliang511

纠结了很多天了，跟一个问题跟到这里，但是觉得google又不可能犯这么低级的错误...请大家指出我愚蠢的地方吧

android 2.2 froyo版本

在froyo\dalvik\libcore\luni\src\main\native\org_apache_harmony_luni_platform_OSNetworkSystem.cpp中（这里只贴出相关的2个case语句）:
我们暂时只看IPV4的部分:
    在case  JAVASOCKOPT_IP_MULTICAST_IF中，getsockopt(...,....,...,&sockval,...)语句之后有sockval.ss_family的引用，因此这个sockval至少是
           一个大小大于4个字节的结构体，从紧接的代码socketAddressToInetAddress（）中可以看出内核应该返回的sockval为struct sockaddr_in类型

  在case JAVASOCKOPT_IP_MULTICAST_IF2，AF_INET时，很明显getsockopt(...,....,...,&sockval,...)返回的sockval类型必须是struct ip_mreqn类型

但是从内核代码\kernel\net\ipv4\ip_sockglue.c中
   case IP_MULTICAST_IF中可以看到，getsockopt明明只支持返回类型为struct in_addr的结构体

这，问题究竟在哪里呢？


froyo\dalvik\libcore\luni\src\main\native\org_apache_harmony_luni_platform_OSNetworkSystem.cpp：

1. 
   
2. static jobject osNetworkSystem_getSocketOptionImpl(JNIEnv* env, jclass clazz,
   
3.         jobject fileDescriptor, jint anOption) {
   
4.     // LOGD("ENTER getSocketOptionImpl");
   
5. 
   
6.     int intValue = 0;
   
7.     socklen_t intSize = sizeof(int);
   
8.     int result;
   
9.     struct sockaddr_storage sockVal;
   
10.     socklen_t sockSize = sizeof(sockVal);
    
11. 
    
12.     int handle;
    
13.     if (!jniGetFd(env, fileDescriptor, handle)) {
    
14.         return 0;
    
15.     }
    
16. 
    
17.     switch ((int) anOption & 0xffff) {
    
18.         case JAVASOCKOPT_IP_MULTICAST_IF: {
    
19.             if ((anOption >> 16) & BROKEN_MULTICAST_IF) {
    
20.                 return NULL;
    
21.             }
    
22.             result = getsockopt(handle, IPPROTO_IP, IP_MULTICAST_IF,
    
23.                 &sockVal, &sockSize);
    
24.             if (result == -1) {
    
25.                 jniThrowSocketException(env, errno);
    
26.                 return NULL;
    
27.             }
    
28.             if (sockVal.ss_family != AF_INET) {
    
29.                 // Java expects an AF_INET INADDR_ANY, but Linux just returns AF_UNSPEC.
    
30.                 jbyteArray inAddrAny = env->NewByteArray(4); // { 0, 0, 0, 0 }
    
31.                 return byteArrayToInetAddress(env, inAddrAny);
    
32.             }
    
33.             return socketAddressToInetAddress(env, &sockVal);
    
34.         }
    
35. 
    
36.         case JAVASOCKOPT_IP_MULTICAST_IF2: {
    
37.             if ((anOption >> 16) & BROKEN_MULTICAST_IF) {
    
38.                 return NULL;
    
39.             }
    
40.             struct ip_mreqn multicastRequest;
    
41.             int interfaceIndex = 0;
    
42.             socklen_t optionLength;
    
43.             int addressFamily = getSocketAddressFamily(handle);
    
44.             switch (addressFamily) {
    
45.                 case AF_INET:
    
46.                     optionLength = sizeof(multicastRequest);
    
47.                     result = getsockopt(handle, IPPROTO_IP, IP_MULTICAST_IF,
    
48.                                         &multicastRequest, &optionLength);
    
49.                     if (result == 0)
    
50.                         interfaceIndex = multicastRequest.imr_ifindex;
    
51.                     break;
    
52.                 case AF_INET6:
    
53.                     optionLength = sizeof(interfaceIndex);
    
54.                     result = getsockopt(handle, IPPROTO_IPV6, IPV6_MULTICAST_IF,
    
55.                                         &interfaceIndex, &optionLength);
    
56.                     break;
    
57.                 default:
    
58.                     jniThrowSocketException(env, EAFNOSUPPORT);
    
59.                     return NULL;
    
60.             }
    
61. 
    
62.             if (0 != result) {
    
63.                 jniThrowSocketException(env, errno);
    
64.                 return NULL;
    
65.             }
    
66.             return newJavaLangInteger(env, interfaceIndex);
    
67.         }
    
68.     }
    
69. }
    

复制代码

但是从内核代码\kernel\net\ipv4\ip_sockglue.c中
case IP_MULTICAST_IF中可以看到，getsockopt明明只支持返回类型为struct in_addr的结构体


kernel\net\ipv4\ip_sockglue.c：

1. 
   
2. 
   
3. /*
   
4. *        Get the options. Note for future reference. The GET of IP options gets the
   
5. *        _received_ ones. The set sets the _sent_ ones.
   
6. */
   
7. 
   
8. static int do_ip_getsockopt(struct sock *sk, int level, int optname,
   
9.                             char __user *optval, int __user *optlen)
   
10. {
    
11.         struct inet_sock *inet = inet_sk(sk);
    
12.         int val;
    
13.         int len;
    
14. 
    
15.         if (level != SOL_IP)
    
16.                 return -EOPNOTSUPP;
    
17. 
    
18.         if (ip_mroute_opt(optname))
    
19.                 return ip_mroute_getsockopt(sk,optname,optval,optlen);
    
20. 
    
21.         if (get_user(len,optlen))
    
22.                 return -EFAULT;
    
23.         if (len < 0)
    
24.                 return -EINVAL;
    
25. 
    
26.         lock_sock(sk);
    
27. 
    
28.         switch (optname) {
    
29. 
    
30.               case IP_MULTICAST_IF:
    
31.               {
    
32.                 struct in_addr addr;
    
33.                 len = min_t(unsigned int, len, sizeof(struct in_addr));
    
34.                 addr.s_addr = inet->mc_addr;
    
35.                 release_sock(sk);
    
36. 
    
37.                 if (put_user(len, optlen))
    
38.                         return -EFAULT;
    
39.                 if (copy_to_user(optval, &addr, len))
    
40.                         return -EFAULT;
    
41.                 return 0;
    
42.               }
    
43.                 }
    
44. }
    

复制代码