Revert "linux-user: unix sockets - fix running dbus"
authorRiku Voipio <riku.voipio@nokia.com>
Mon, 20 Apr 2009 14:13:03 +0000 (17:13 +0300)
committerRiku Voipio <riku.voipio@nokia.com>
Mon, 20 Apr 2009 14:13:03 +0000 (17:13 +0300)
This reverts commit 7479ae68f8e5bea85885faca651b8d9096f106c3.

linux-user/syscall.c

index f0d2a91..36703d8 100644 (file)
@@ -44,7 +44,6 @@
 #include <signal.h>
 #include <sched.h>
 #include <sys/socket.h>
-#include <sys/un.h>
 #include <sys/uio.h>
 #include <sys/poll.h>
 #include <sys/times.h>
@@ -915,37 +914,13 @@ static inline abi_long target_to_host_sockaddr(struct sockaddr *addr,
                                                abi_ulong target_addr,
                                                socklen_t len)
 {
-    const socklen_t unix_maxlen = sizeof (struct sockaddr_un);
-    sa_family_t sa_family;
     struct target_sockaddr *target_saddr;
 
     target_saddr = lock_user(VERIFY_READ, target_addr, len, 1);
     if (!target_saddr)
         return -TARGET_EFAULT;
-
-    sa_family = tswap16(target_saddr->sa_family);
-
-    /* Oops. The caller might send a incomplete sun_path; sun_path
-     * must be terminated by \0 (see the manual page), but
-     * unfortunately it is quite common to specify sockaddr_un
-     * length as "strlen(x->sun_path)" while it should be
-     * "strlen(...) + 1". We'll fix that here if needed.
-     * Linux kernel has a similar feature.
-     */
-
-    if (sa_family == AF_UNIX) {
-        if (len < unix_maxlen) {
-            char *cp = (char*)target_saddr;
-
-            if ( cp[len-1] && !cp[len] )
-                len++;
-        }
-        if (len > unix_maxlen)
-            len = unix_maxlen;
-    }
-
     memcpy(addr, target_saddr, len);
-    addr->sa_family = sa_family;
+    addr->sa_family = tswap16(target_saddr->sa_family);
     unlock_user(target_saddr, target_addr, 0);
 
     return 0;
@@ -1396,7 +1371,7 @@ static abi_long do_bind(int sockfd, abi_ulong target_addr,
     if (addrlen < 0)
         return -TARGET_EINVAL;
 
-    addr = alloca(addrlen+1);
+    addr = alloca(addrlen);
 
     target_to_host_sockaddr(addr, target_addr, addrlen);
     return get_errno(bind(sockfd, addr, addrlen));