Don't expect _hurd_fd_get to lock the descriptor.
authorroland <roland>
Tue, 14 Feb 1995 14:43:13 +0000 (14:43 +0000)
committerroland <roland>
Tue, 14 Feb 1995 14:43:13 +0000 (14:43 +0000)
Use critical sections while locking the descriptors.
Check for empty descriptor and return EBADF.

sysdeps/mach/hurd/dup2.c

index b3f6f97..f4ec623 100644 (file)
@@ -30,83 +30,82 @@ int
 DEFUN(__dup2, (fd, fd2), int fd AND int fd2)
 {
   struct hurd_fd *d;
-  struct hurd_fd *d2;
-  io_t port, ctty;
-  struct hurd_userlink ulink, ctty_ulink;
-  int flags;
-
-  HURD_CRITICAL_BEGIN;
 
   /* Extract the ports and flags from FD.  */
   d = _hurd_fd_get (fd);
   if (d == NULL)
     {
       errno = EBADF;
-      goto earlyout;
-    }
-
-  if (fd2 == fd)
-    {
-      /* FD is valid and FD2 is already the same; just return it.  */
-      __spin_unlock (&d->port.lock);
-      goto out;
+      return -1;
     }
 
-  flags = d->flags;
-  ctty = _hurd_port_get (&d->ctty, &ctty_ulink);
-  port = _hurd_port_locked_get (&d->port, &ulink); /* Unlocks D.d.  */
+  HURD_CRITICAL_BEGIN;
 
-  __mutex_lock (&_hurd_dtable_lock);
-  if (fd2 < 0 || fd2 >= _hurd_dtablesize)
+  __spin_lock (&d->port.lock);
+  if (d->port.port == MACH_PORT_NULL)
     {
+      __spin_unlock (&d->port.lock);
       errno = EBADF;
       fd2 = -1;
     }
+  else if (fd2 == fd)
+    /* FD is valid and FD2 is already the same; just return it.  */
+    __spin_unlock (&d->port.lock);
   else
     {
-      /* Get a hold of the destination descriptor.  */
-      d2 = _hurd_dtable[fd2];
-      if (d2 == NULL)
-       {
-         /* Must allocate a new one.  We don't initialize the port cells
-            with this call so that if it fails (out of memory), we will
-            not have already added user references for the ports, which we
-            would then have to deallocate.  */
-         d2 = _hurd_dtable[fd2] = _hurd_new_fd (MACH_PORT_NULL,
-                                                MACH_PORT_NULL);
-       }
-      if (d2 == NULL)
+      struct hurd_userlink ulink, ctty_ulink;
+      int flags = d->flags;
+      io_t ctty = _hurd_port_get (&d->ctty, &ctty_ulink);
+      io_t port = _hurd_port_locked_get (&d->port, &ulink); /* Unlocks D.  */
+
+      __mutex_lock (&_hurd_dtable_lock);
+      if (fd2 < 0 || fd2 >= _hurd_dtablesize)
        {
+         errno = EBADF;
          fd2 = -1;
-         if (errno == EINVAL)
-           errno = EBADF;      /* POSIX.1-1990 6.2.1.2 ll 54-55.  */
        }
       else
        {
-         /* Give the ports each a user ref for the new descriptor.  */
-         __mach_port_mod_refs (__mach_task_self (), port,
-                               MACH_PORT_RIGHT_SEND, 1);
-         if (ctty != MACH_PORT_NULL)
-           __mach_port_mod_refs (__mach_task_self (), ctty,
-                                 MACH_PORT_RIGHT_SEND, 1);
-
-         /* Install the ports and flags in the new descriptor slot.  */
-         __spin_lock (&d2->port.lock);
-         d2->flags = flags & ~FD_CLOEXEC; /* Duplication clears FD_CLOEXEC. */
-         _hurd_port_set (&d2->ctty, ctty);
-         _hurd_port_locked_set (&d2->port, port); /* Unlocks D2.  */
+         /* Get a hold of the destination descriptor.  */
+         struct hurd_fd *d2 = _hurd_dtable[fd2];
+         if (d2 == NULL)
+           {
+             /* Must allocate a new one.  We don't initialize the port cells
+                with this call so that if it fails (out of memory), we will
+                not have already added user references for the ports, which we
+                would then have to deallocate.  */
+             d2 = _hurd_dtable[fd2] = _hurd_new_fd (MACH_PORT_NULL,
+                                                    MACH_PORT_NULL);
+           }
+         if (d2 == NULL)
+           {
+             fd2 = -1;
+             if (errno == EINVAL)
+               errno = EBADF;  /* POSIX.1-1990 6.2.1.2 ll 54-55.  */
+           }
+         else
+           {
+             /* Give the ports each a user ref for the new descriptor.  */
+             __mach_port_mod_refs (__mach_task_self (), port,
+                                   MACH_PORT_RIGHT_SEND, 1);
+             if (ctty != MACH_PORT_NULL)
+               __mach_port_mod_refs (__mach_task_self (), ctty,
+                                     MACH_PORT_RIGHT_SEND, 1);
+
+             /* Install the ports and flags in the new descriptor slot.  */
+             __spin_lock (&d2->port.lock);
+             d2->flags = flags & ~FD_CLOEXEC; /* Dup clears FD_CLOEXEC. */
+             _hurd_port_set (&d2->ctty, ctty);
+             _hurd_port_locked_set (&d2->port, port); /* Unlocks D2.  */
+           }
        }
-    }
-  __mutex_unlock (&_hurd_dtable_lock);
-
-  _hurd_port_free (&d->port, &ulink, port);
-  if (ctty != MACH_PORT_NULL)
-    _hurd_port_free (&d->ctty, &ctty_ulink, port);
+      __mutex_unlock (&_hurd_dtable_lock);
 
- out:
-  __spin_unlock (&d->port.lock);
+      _hurd_port_free (&d->port, &ulink, port);
+      if (ctty != MACH_PORT_NULL)
+       _hurd_port_free (&d->ctty, &ctty_ulink, port);
+    }
 
- earlyout:
   HURD_CRITICAL_END;
 
   return fd2;