diff options
author | June McEnroe <june@causal.agency> | 2017-07-23 00:34:26 -0400 |
---|---|---|
committer | June McEnroe <june@causal.agency> | 2017-07-23 00:34:26 -0400 |
commit | 1aa1a575842ade0cc96f0638d38411a239111a3a (patch) | |
tree | 1fddcbf15de2ba6525cddaf8212225bfe74f68a3 /home | |
parent | Clean up hnel a tiny bit (diff) | |
download | src-1aa1a575842ade0cc96f0638d38411a239111a3a.tar.gz src-1aa1a575842ade0cc96f0638d38411a239111a3a.zip |
Remove useless setuid in bri
Don't you think it would be better if the setuid bit only gave you permission to do it and didn't do it for you?
Diffstat (limited to 'home')
-rwxr-xr-x | home/.bin/bri.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/home/.bin/bri.c b/home/.bin/bri.c index 4c53475f..d085814b 100755 --- a/home/.bin/bri.c +++ b/home/.bin/bri.c @@ -1,7 +1,9 @@ #if 0 -cc -Wall -Wextra -pedantic $@ -o $(dirname $0)/bri $0 && \ -sudo chown root:root $(dirname $0)/bri && \ -sudo chmod u+s $(dirname $0)/bri +set -e +bin=$(dirname $0) +cc -Wall -Wextra -pedantic $@ -o $bin/bri $0 +sudo chown root:root $bin/bri +sudo chmod u+s $bin/bri exit #endif @@ -65,9 +67,6 @@ int main(int argc, char *argv[]) { value = buf; } - error = setuid(0); - if (error) err(EX_NOPERM, "setuid(0)"); - FILE *brightness = fopen("brightness", "w"); if (!brightness) err(EX_IOERR, "brightness"); |