From fab2200666ddb244ff91d280a19cce3aece836b2 Mon Sep 17 00:00:00 2001 From: Stefano Rivera Date: Sun, 19 Dec 2010 01:45:31 +0200 Subject: [PATCH] Strengthen sudo warning to an error and abort if user doesn't own $HOME --- pbuilder-dist | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/pbuilder-dist b/pbuilder-dist index ab3e658..e4596a6 100755 --- a/pbuilder-dist +++ b/pbuilder-dist @@ -88,12 +88,16 @@ class pbuilder_dist: self.base = os.path.expanduser(os.environ.get('PBUILDFOLDER', '~/pbuilder/')) + if 'SUDO_USER' in os.environ: + print >> stderr, ("Warning: pbuilder-dist running under sudo. " + "This is probably not what you want. " + "pbuilder-dist will use sudo itself, " + "when necessary.") + if os.stat(os.environ['HOME']).st_uid != os.getuid(): + print >> stderr, "Error: You don't own $HOME" + exit(1) + if not os.path.isdir(self.base): - if 'SUDO_USER' in os.environ: - print >> stderr, ("Warning: pbuilder-dist running under sudo. " - "This is probably not what you want. " - "pbuilder-dist will use sudo itself, " - "when necessary.") try: os.makedirs(self.base) except os.OSError: