-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 This patch looks good to me. acked. Need this so that we can ship sandbox command without X. -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAk5Ws08ACgkQrlYvE4MpobPkegCeNlt2z/dSuRDDiOffwmUORHk0 IfcAoIQadmmT+x4fCNet2n5+28J+l1zT =EWyr -----END PGP SIGNATURE-----
>From ebaa5a8e9d11a88271dead8bcbbd5ba1001adc9d Mon Sep 17 00:00:00 2001 From: Eric Paris <eparis@xxxxxxxxxx> Date: Wed, 10 Aug 2011 16:28:36 -0400 Subject: [PATCH 48/77] policycoreutils: sandbox: check for paths up front When launching a sandbox x environment we should check up front to make sure that the seunshare and sandboxsh files exist and bail politely if they do not exist. Signed-off-by: Eric Paris <eparis@xxxxxxxxxx> --- policycoreutils/sandbox/sandbox | 9 +++++++++ 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/policycoreutils/sandbox/sandbox b/policycoreutils/sandbox/sandbox index 7dc719d..2015e65 100644 --- a/policycoreutils/sandbox/sandbox +++ b/policycoreutils/sandbox/sandbox @@ -174,6 +174,15 @@ class Sandbox: def __x_callback(self, option, opt, value, parser): self.__mount = True setattr(parser.values, option.dest, True) + if not os.path.exists(SEUNSHARE): + raise ValueError(_(""" +%s is required for the action you want to perform. +""") % SEUNSHARE) + + if not os.path.exists(SANDBOXSH): + raise ValueError(_(""" +%s is required for the action you want to perform. +""") % SANDBOXSH) def __validdir(self, option, opt, value, parser): if not os.path.isdir(value): -- 1.7.6