From 06a80120b0cc1d3c0af321ed8231b5683901e98e Mon Sep 17 00:00:00 2001 From: bchristiansen Date: Wed, 21 Jul 2010 14:35:56 +0000 Subject: [PATCH] fix for preventing users from setting system priority on their own jobs (Bas van der Vlies) git-svn-id: svn://opensvn.adaptivecomputing.com/maui/trunk@134 3f5042e3-fb1d-0410-be18-d6ca2573e517 --- CHANGELOG | 1 + src/server/UserI.c | 9 ++++++++- 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/CHANGELOG b/CHANGELOG index 7333a3e..4d6a9de 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -1,5 +1,6 @@ Maui 3.3.1 - Fixed issue where hostlist was being overwritten resulting in broken $PBS_NODEFILE. (Alexis Cousein) + - Fixed issue where users could set system priority on their own jobs. (Bas van der Vlies) Maui 3.3 - Fixed configure script. Was putting RMCFG[name] TYPE=PBS@RMNMHOST@. diff --git a/src/server/UserI.c b/src/server/UserI.c index a1abead..75a229a 100644 --- a/src/server/UserI.c +++ b/src/server/UserI.c @@ -5521,7 +5521,14 @@ int MUIJobSetAttr( { long tmpPrio; - tmpPrio = strtol(Val,NULL,0); + if (!(CFlags & ((1 << fAdmin1) | (1 << fAdmin2)))) + { + sprintf(Msg,"ERROR: not authorized to run this command"); + + return(FAILURE); + } + + tmpPrio = strtol(Val,NULL,0); if ((tmpPrio < 0) || (tmpPrio > 1000)) {