From d960211ff52a6ab632eace2f905c4898f187df9e Mon Sep 17 00:00:00 2001 From: Roland McGrath Date: Wed, 19 Nov 2014 17:34:25 -0800 Subject: NPTL: Don't (re)validate sched_priority in pthread_create. --- nptl/pthread_create.c | 21 +-------------------- 1 file changed, 1 insertion(+), 20 deletions(-) (limited to 'nptl') diff --git a/nptl/pthread_create.c b/nptl/pthread_create.c index da3dc4603f..1dd1b951c0 100644 --- a/nptl/pthread_create.c +++ b/nptl/pthread_create.c @@ -605,6 +605,7 @@ __pthread_create_2_1 (newthread, attr, start_routine, arg) } if (iattr->flags & ATTR_FLAG_SCHED_SET) + /* The values were validated in pthread_attr_setschedparam. */ memcpy (&pd->schedparam, &iattr->schedparam, sizeof (struct sched_param)); else if ((pd->flags & ATTR_FLAG_SCHED_SET) == 0) @@ -612,26 +613,6 @@ __pthread_create_2_1 (newthread, attr, start_routine, arg) INTERNAL_SYSCALL (sched_getparam, scerr, 2, 0, &pd->schedparam); pd->flags |= ATTR_FLAG_SCHED_SET; } - - /* Check for valid priorities. */ - int minprio = INTERNAL_SYSCALL (sched_get_priority_min, scerr, 1, - iattr->schedpolicy); - int maxprio = INTERNAL_SYSCALL (sched_get_priority_max, scerr, 1, - iattr->schedpolicy); - if (pd->schedparam.sched_priority < minprio - || pd->schedparam.sched_priority > maxprio) - { - /* Perhaps a thread wants to change the IDs and if waiting - for this stillborn thread. */ - if (__builtin_expect (atomic_exchange_acq (&pd->setxid_futex, 0) - == -2, 0)) - lll_futex_wake (&pd->setxid_futex, 1, LLL_PRIVATE); - - __deallocate_stack (pd); - - retval = EINVAL; - goto out; - } } /* Pass the descriptor to the caller. */ -- cgit v1.2.3-70-g09d2