[rtems commit] semdestroy.c: Add missing _Objects_Allocator_unlock()

Joel Sherril joel at rtems.org
Tue Apr 14 19:08:27 UTC 2015


Module:    rtems
Branch:    master
Commit:    1047f37cac02909b33d16462f14dc5f45e98d376
Changeset: http://git.rtems.org/rtems/commit/?id=1047f37cac02909b33d16462f14dc5f45e98d376

Author:    Joel Sherrill <joel.sherrill at oarcorp.com>
Date:      Tue Apr 14 12:11:51 2015 -0500

semdestroy.c: Add missing _Objects_Allocator_unlock()

closes 2319.

---

 cpukit/posix/src/semdestroy.c | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/cpukit/posix/src/semdestroy.c b/cpukit/posix/src/semdestroy.c
index 1c695bb..9a8a993 100644
--- a/cpukit/posix/src/semdestroy.c
+++ b/cpukit/posix/src/semdestroy.c
@@ -44,19 +44,18 @@ int sem_destroy(
 
     case OBJECTS_LOCAL:
       /*
-       *  Undefined operation on a named semaphore.
+       *  Undefined operation on a named semaphore. Release the object
+       *  and fall to the EINVAL return at the bottom.
        */
-
       if ( the_semaphore->named == true ) {
         _Objects_Put( &the_semaphore->Object );
-        rtems_set_errno_and_return_minus_one( EINVAL );
+      } else {
+        _POSIX_Semaphore_Delete( the_semaphore );
+        _Objects_Put( &the_semaphore->Object );
+        _Objects_Allocator_unlock();
+        return 0;
       }
 
-      _POSIX_Semaphore_Delete( the_semaphore );
-      _Objects_Put( &the_semaphore->Object );
-      _Objects_Allocator_unlock();
-      return 0;
-
 #if defined(RTEMS_MULTIPROCESSING)
     case OBJECTS_REMOTE:
 #endif



More information about the vc mailing list