fix requiretty check, use 'sudo -l' 01/18501/1
authorRyota MIBU <r-mibu@cq.jp.nec.com>
Wed, 10 Aug 2016 22:11:32 +0000 (07:11 +0900)
committerRyota MIBU <r-mibu@cq.jp.nec.com>
Wed, 10 Aug 2016 22:11:32 +0000 (07:11 +0900)
A normal user doesn't have a right to read /etc/sudoers, so current
script won't work if executed by non-root users.

This patch replacing the check by geeting config from 'sudo -l'
command.

Change-Id: I608edabb73761cea0f6199045f1cde2217a34bf3
Signed-off-by: Ryota MIBU <r-mibu@cq.jp.nec.com>
utils/jenkins-jnlp-connect.sh

index d268a28..8dcaf35 100755 (executable)
@@ -49,8 +49,8 @@ main () {
     fi
 
     if [[ $(whoami) != "root" ]]; then
-      if grep "^Defaults requiretty" /etc/sudoers
-        then echo "please comment out Defaults requiretty from /etc/sudoers"
+      if sudo -l | grep "requiretty"; then
+        echo "please comment out Defaults requiretty from /etc/sudoers"
         exit 1
       fi
     fi