[ruby-dev:28691] Socket.unpack_sockaddr_in(Socket.sockaddr_un("/tmp/s")) is [12148, "localhost"]?

ふと、sockaddr_in を unpack_sockaddr_un したり、
sockaddr_un を unpack_sockaddr_in したりしてもエラーにならず
になんか値が返ってくることに気がつきました。

% ruby -v -rsocket -e 'p Socket.unpack_sockaddr_un(Socket.sockaddr_in(80, "localhost"))'
ruby 1.9.0 (2006-06-04) [i686-linux]
""
% ruby -v -rsocket -e 'p Socket.unpack_sockaddr_in(Socket.sockaddr_un("/tmp/s"))'
ruby 1.9.0 (2006-06-04) [i686-linux]
[12148, "localhost"]

これってエラーのほうがいいんじゃないでしょうか。

Index: ext/socket/socket.c
===================================================================
RCS file: /src/ruby/ext/socket/socket.c,v
retrieving revision 1.168
diff -u -p -r1.168 socket.c
--- ext/socket/socket.c	4 Jun 2006 11:43:16 -0000	1.168
+++ ext/socket/socket.c	5 Jun 2006 05:15:53 -0000
@@ -3512,6 +3512,17 @@ sock_s_unpack_sockaddr_in(self, addr)
     VALUE host;
 
     sockaddr = (struct sockaddr_in*)StringValuePtr(addr);
+    if (((struct sockaddr *)sockaddr)->sa_family != AF_INET
+#ifdef INET6
+        && ((struct sockaddr *)sockaddr)->sa_family != AF_INET6
+#endif
+        ) {
+#ifdef INET6
+        rb_raise(rb_eArgError, "not an AF_INET/AF_INET6 sockaddr");
+#else
+        rb_raise(rb_eArgError, "not an AF_INET sockaddr");
+#endif
+    }
     host = make_ipaddr((struct sockaddr*)sockaddr);
     OBJ_INFECT(host, addr);
     return rb_assoc_new(INT2NUM(ntohs(sockaddr->sin_port)), host);
@@ -3549,6 +3560,9 @@ sock_s_unpack_sockaddr_un(self, addr)
     VALUE path;
 
     sockaddr = (struct sockaddr_un*)StringValuePtr(addr);
+    if (((struct sockaddr *)sockaddr)->sa_family != AF_UNIX) {
+        rb_raise(rb_eArgError, "not an AF_UNIX sockaddr");
+    }
     if (sizeof(struct sockaddr_un) < RSTRING(addr)->len) {
 	rb_raise(rb_eTypeError, "too long sockaddr_un - %ld longer than %d",
 		 RSTRING(addr)->len, sizeof(struct sockaddr_un));

-- 
[田中 哲][たなか あきら][Tanaka Akira]