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. --- ChangeLog | 4 ++++ nptl/pthread_create.c | 21 +-------------------- 2 files changed, 5 insertions(+), 20 deletions(-) diff --git a/ChangeLog b/ChangeLog index 239c9fec6b..6199b7e692 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,9 @@ 2014-11-19 Roland McGrath + * nptl/pthread_create.c (__pthread_create_2_1): Don't try to validate + the sched_priority value here. It was already checked when the user + called pthread_attr_setschedparam. + * nptl/tst-bad-schedattr.c: New file. * nptl/Makefile (tests): Add it. 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 1.4.1