Changeset ccd5434 in rtems for cpukit/posix/src/pthreadcreate.c

Timestamp:

01/07/16 08:55:45 (7 years ago)

Author:

Sebastian Huber <sebastian.huber@…>

Branches:

5, master

Children:

1506658c

Parents:

3d1becf

git-author:

Sebastian Huber <sebastian.huber@…> (01/07/16 08:55:45)

git-committer:

Sebastian Huber <sebastian.huber@…> (01/11/16 07:47:01)

Message:

score: Introduce Thread_Entry_information

This avoids potential dead code in _Thread_Handler(). It gets rid of
the dangerous function pointer casts.

Update #2514.

File:

• cpukit/posix/src/pthreadcreate.c (modified) (2 diffs)

cpukit/posix/src/pthreadcreate.c

@@ -53 +53 @@
 )
 {
+  Thread_Entry_information entry = {
+    .adaptor = _Thread_Entry_adaptor_pointer,
+    .Kinds = {
+      .Pointer = {
+        .entry = start_routine,
+        .argument = arg
+      }
+    }
+  };
   const pthread_attr_t               *the_attr;
   Priority_Control                    core_priority;
@@ -220 +229 @@
    *  POSIX threads are allocated and started in one operation.
    */
-  status = _Thread_Start(
-    the_thread,
-    THREAD_START_POINTER,
-    start_routine,
-    arg,
-    0,                    /* unused */
-    NULL
-  );
+  status = _Thread_Start( the_thread, &entry, NULL );
 
   #if defined(RTEMS_DEBUG)