From f39e3e6597270bbc2fa93c1768fa82cd0ddb60ee Mon Sep 17 00:00:00 2001 From: Samuel Thibault Date: Sun, 21 Aug 2016 02:59:41 +0200 Subject: 2014-08-27 Samuel Thibault Fix bind when umask is e.g. 0777. * sysdeps/mach/hurd/bind.c (__bind): Pass mode 0666 to __dir_mkfile instead of final mode, so that call __ifsock_getsockaddr can always succeed, before calling __file_chmod to fix the mode according to umask, before calling __dir_link to show the file. --- sysdeps/mach/hurd/bind.c | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) (limited to 'sysdeps/mach/hurd/bind.c') diff --git a/sysdeps/mach/hurd/bind.c b/sysdeps/mach/hurd/bind.c index 4b09be3635..0f8bda1fd0 100644 --- a/sysdeps/mach/hurd/bind.c +++ b/sysdeps/mach/hurd/bind.c @@ -48,7 +48,7 @@ __bind (int fd, __CONST_SOCKADDR_ARG addrarg, socklen_t len) return -1; /* Create a new, unlinked node in the target directory. */ - err = __dir_mkfile (dir, O_CREAT, 0666 & ~_hurd_umask, &node); + err = __dir_mkfile (dir, O_CREAT, 0666, &node); if (! err) { @@ -76,11 +76,16 @@ __bind (int fd, __CONST_SOCKADDR_ARG addrarg, socklen_t len) err = EGRATUITOUS; if (! err) { - /* Link the node, now a socket with proper mode, into the - target directory. */ - err = __dir_link (dir, node, n, 1); - if (err == EEXIST) - err = EADDRINUSE; + /* Fix the access mode before showing the file. */ + err = __file_chmod (node, 0666 & ~_hurd_umask); + if (! err) + { + /* Link the node, now a socket with proper mode, into the + target directory. */ + err = __dir_link (dir, node, n, 1); + if (err == EEXIST) + err = EADDRINUSE; + } if (err) __mach_port_deallocate (__mach_task_self (), aport); } -- cgit v1.2.3