Hello, I would like to commit a really small patch that makes getsockname fill the sockaddr for non-bound PF_LOCAL sockets with sun_noname, instead of just setting the length parameter to 0 and return (without an error) like it does now. PF_INET and PF_INET6 sockets, for example, behave like this, and it is useful to get the address family of a socket, for example in the RPC library. SUSv2 says wrt getsockname: If the socket has not been bound to a local name, the value stored in the object pointed to by address is unspecified. So this does not break standard conformity (using this feature however could, of course, OTOH e.g. the RPC code is anyway specific to FreeBSD to a certain extent). >From a quick glance, it seems that NetBSD implements the behaviour this patch would give use, while OpenBSD uses the other one. Patch attached. Any comments or objections? - thomas
Index: sys/kern/uipc_usrreq.c =================================================================== RCS file: /home/ncvs/src/sys/kern/uipc_usrreq.c,v retrieving revision 1.63 diff -u -r1.63 uipc_usrreq.c --- sys/kern/uipc_usrreq.c 2001/02/21 06:39:55 1.63 +++ sys/kern/uipc_usrreq.c 2001/04/22 12:51:24 @@ -420,6 +420,8 @@ return EINVAL; if (unp->unp_addr) *nam = dup_sockaddr((struct sockaddr *)unp->unp_addr, 1); + else + *nam = dup_sockaddr((struct sockaddr *)&sun_noname, 1); return 0; }