From 51140065a24329869353f970e37bb4f85110482d Mon Sep 17 00:00:00 2001 From: Sascha Silbe Date: Fri, 22 Jan 2010 13:18:16 +0000 Subject: [PATCH] sugar, sugar-emulator: refuse to run as root (#1675) It's a very bad idea to run Sugar as root for a variety of reasons, but there seem to be people who try it (and fail) nevertheless. Failing early and with a clear message increases safety and safes precious developer and user time. --- bin/sugar-emulator | 5 +++++ bin/sugar.in | 5 +++++ 2 files changed, 10 insertions(+), 0 deletions(-) diff --git a/bin/sugar-emulator b/bin/sugar-emulator index 800ca33..308aac7 100755 --- a/bin/sugar-emulator +++ b/bin/sugar-emulator @@ -1,5 +1,10 @@ #!/bin/sh +if [ "$(id -u)" -eq 0 -o "$(id -ru)" -eq 0 ] ; then + echo Refusing to run as root. + exit 3 +fi + # Source debug definitions if [ -f ~/.sugar/debug ]; then . ~/.sugar/debug diff --git a/bin/sugar.in b/bin/sugar.in index 12768a5..498b573 100644 --- a/bin/sugar.in +++ b/bin/sugar.in @@ -1,5 +1,10 @@ #!/bin/sh +if [ "$(id -u)" -eq 0 -o "$(id -ru)" -eq 0 ] ; then + echo Refusing to run as root. + exit 3 +fi + usage() { cat <