From daf04e4ce912839383d649109a6aac66c2b48709 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?St=C3=A9phane=20Graber?= <stgraber@ubuntu.com>
Date: Tue, 4 Dec 2012 16:17:08 -0500
Subject: [PATCH] lxc-ls: Show a simple error message when non-root
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Instead of returning a python stacktrace, check what the current euid is
and show an argparse error message similar to that used in lxc-start-ephemeral.

Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>
Acked-by: Serge E. Hallyn <serge.hallyn@ubuntu.com>
---
 src/lxc/lxc-ls | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/src/lxc/lxc-ls b/src/lxc/lxc-ls
index 8a1d1ed88..2ad1f7f0c 100644
--- a/src/lxc/lxc-ls
+++ b/src/lxc/lxc-ls
@@ -32,6 +32,7 @@ warnings.filterwarnings("ignore", "The python-lxc API isn't yet stable")
 import argparse
 import gettext
 import lxc
+import os
 import re
 import sys
 
@@ -115,6 +116,12 @@ parser.add_argument("filter", metavar='FILTER', type=str, nargs="?",
 
 args = parser.parse_args()
 
+# Basic checks
+## The user needs to be uid 0
+if not os.geteuid() == 0:
+    parser.error(_("You must be root to run this script. Try running: sudo %s"
+                   % (sys.argv[0])))
+
 # --active is the same as --running --frozen
 if args.active:
     if not args.state: