error in running bash script as root

Question

I have a simple shutdown script which i want to run as root

shutdown -h +30;
echo "succesfull";

I have followed these answer1,2 to make my script root.

first I run this command

sudo chown root.root $HOME/test/test.sh
sudo chmod 4755 $HOME/test/test.sh

and then made changes in sudoers

sudo visudo

added this command after this line %sudo ALL=(ALL:ALL) ALL

eka ALL=(ALL) NOPASSWD: $HOME/test/test.sh

But when i executed my script its showing this error

shutdown: Need to be root
succesfull

chmod 4755 - I assume you meant to make a setuid script? Scripts can't be made setuid. — muru, May 22, 2015 at 9:07
That user is mistaken. setuid doesn't work on scripts. Period. — muru, May 22, 2015 at 9:17
@muru thanks for the info.. but how to revert back to normal executable script whether this code be enough sudo chmod +x $HOME/test/test.sh — Eka, May 22, 2015 at 9:19
You need to do chmod u-s, but as I said, it doesn't matter - setuid doesn't affect scripts. — muru, May 22, 2015 at 9:20

ckujau · Accepted Answer · 2015-05-22 09:57:35Z

4

Don't use setuid shell scripts, the SUID bit is not honored on shell scripts anyway on current systems. Use sudo instead, as you attempted anyway:

eka   ALL=NOPASSWD: /home/eka/test/test.sh

And then:

eka$ sudo ~/test/test.sh

With that, test.sh will be executed as "root". No need to use setuid here.

answered May 22, 2015 at 9:57

ckujau

3411 silver badge11 bronze badges

How to execute by double clicking the script?
– Eka
May 22, 2015 at 17:24
That follow-up is an entirely different question, see How to execute a script just by double clicking like .EXE files in Windows?
– ckujau
May 22, 2015 at 19:50

Add a comment |

1 Answer 1