summary refs log tree commit diff
path: root/home/.bin/bri.c
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 /home/.bin/bri.c
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");