Changeset 17879f4 in rtems for c/src/exec/posix/src/semopen.c

Timestamp:

Nov 18, 1999, 7:43:13 PM (21 years ago)

Author:

Jennifer Averett <Jennifer.Averett@…>

Branches:

4.10, 4.11, 4.8, 4.9, 5, master

Children:

105d7872

Parents:

c9ae846

Message:

+ Debuged to the point that you could open, unlink and close a semaphore.

but all paths have not been checked, yet.

File:

• c/src/exec/posix/src/semopen.c (modified) (4 diffs)

c/src/exec/posix/src/semopen.c

@@ -44 +44 @@
   if ( oflag & O_CREAT ) {
     va_start(arg, oflag);
-    /*mode = (mode_t) va_arg( arg, mode_t * );*/
     mode = va_arg( arg, mode_t );
-    /*value = (unsigned int) va_arg( arg, unsigned int * );*/
     value = va_arg( arg, unsigned int );
     va_end(arg);
@@ -62 +60 @@
   if ( status ) {
 
-    if ( status == EINVAL ) {      /* name -> ID translation failed */
-      if ( !(oflag & O_CREAT) ) {  /* willing to create it? */
+    /*
+     * Unless we are willing to create name -> ID translation failure is
+     * an error.
+     */
+
+    if ( status == EINVAL ) { 
+      if ( !(oflag & O_CREAT) ) {
         set_errno_and_return_minus_one_cast( ENOENT, sem_t * );
       }
-      /* we are willing to create it */
     }
-    /* some type of error */
-    /*set_errno_and_return_minus_one_cast( status, sem_t * );*/
 
-  } else {                /* name -> ID translation succeeded */
+  } else { 
+
+    /*
+     * Check for existence with creation.
+     */
 
     if ( (oflag & (O_CREAT | O_EXCL)) == (O_CREAT | O_EXCL) ) {
@@ -77 +81 @@
     }
 
-    /* 
-     * XXX In this case we need to do an ID->pointer conversion to
-     *     check the mode.   This is probably a good place for a subroutine.
-     */
-
     the_semaphore = _POSIX_Semaphore_Get( &the_semaphore_id, &location );
     the_semaphore->open_count += 1;
-
     return (sem_t *)&the_semaphore->Object.id;
 
   } 
   
-  /* XXX verify this comment...
-   *
+  /* 
    *  At this point, the semaphore does not exist and everything has been
    *  checked. We should go ahead and create a semaphore.
@@ -102 +99 @@
   );
  
+  /*
+   * errno was set by Create_support, so don't set it again.
+   */
+
   if ( status == -1 )
-    return (sem_t *) -1;
+    return SEM_FAILED;
 
   return (sem_t *) &the_semaphore->Object.id;
- 
 }