summary refs log tree commit diff
diff options
context:
space:
mode:
authorJune McEnroe <june@causal.agency>2017-07-23 00:34:26 -0400
committerJune McEnroe <june@causal.agency>2017-07-23 00:34:26 -0400
commit1aa1a575842ade0cc96f0638d38411a239111a3a (patch)
tree1fddcbf15de2ba6525cddaf8212225bfe74f68a3
parentClean up hnel a tiny bit (diff)
downloadsrc-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 '')
-rwxr-xr-xhome/.bin/bri.c11
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");