Changeset 948721c in rtems

Timestamp:

08/15/96 16:24:44 (26 years ago)

Author:

Mark Johannes <Mark.Johannes@…>

Branches:

4.10, 4.11, 4.8, 4.9, 5, master

Children:

de45ddc

Parents:

48e685c

Message:

pthread_kill: moved check for SA_SIGINFO to allow for validation of all
parameters. Also changed returns to macro of -1 and errno set.

Files:

• c/src/exec/posix/src/psignal.c (modified) (3 diffs)
• cpukit/posix/src/psignal.c (modified) (3 diffs)

c/src/exec/posix/src/psignal.c

@@ -1255 +1255 @@
    */
 
-  if ( _POSIX_signals_Vectors[ sig ].sa_flags == SA_SIGINFO )
-    return ENOSYS;
-
   the_thread = _POSIX_Threads_Get( thread, &location );
   switch ( location ) {
     case OBJECTS_ERROR:
     case OBJECTS_REMOTE:
-      return ESRCH;
+      set_errno_and_return_minus_one( ESRCH );
     case OBJECTS_LOCAL:
       /*
@@ -1272 +1269 @@
       if ( sig ) {
 
+        if ( _POSIX_signals_Vectors[ sig ].sa_flags == SA_SIGINFO )
+          set_errno_and_return_minus_one( ENOSYS );
+
         /* XXX critical section */
 
@@ -1277 +1277 @@
 
         (void) _POSIX_signals_Unblock_thread( the_thread, sig, NULL );
-    }
-    _Thread_Enable_dispatch();
-    return 0;
+      }
+      _Thread_Enable_dispatch();
+      return 0;
   }
 

cpukit/posix/src/psignal.c

@@ -1255 +1255 @@
    */
 
-  if ( _POSIX_signals_Vectors[ sig ].sa_flags == SA_SIGINFO )
-    return ENOSYS;
-
   the_thread = _POSIX_Threads_Get( thread, &location );
   switch ( location ) {
     case OBJECTS_ERROR:
     case OBJECTS_REMOTE:
-      return ESRCH;
+      set_errno_and_return_minus_one( ESRCH );
     case OBJECTS_LOCAL:
       /*
@@ -1272 +1269 @@
       if ( sig ) {
 
+        if ( _POSIX_signals_Vectors[ sig ].sa_flags == SA_SIGINFO )
+          set_errno_and_return_minus_one( ENOSYS );
+
         /* XXX critical section */
 
@@ -1277 +1277 @@
 
         (void) _POSIX_signals_Unblock_thread( the_thread, sig, NULL );
-    }
-    _Thread_Enable_dispatch();
-    return 0;
+      }
+      _Thread_Enable_dispatch();
+      return 0;
   }