capsh --user is available from f14 only
authorThierry Parmentelat <thierry.parmentelat@inria.fr>
Sat, 6 Jul 2013 13:16:47 +0000 (15:16 +0200)
committerThierry Parmentelat <thierry.parmentelat@inria.fr>
Sat, 6 Jul 2013 13:16:47 +0000 (15:16 +0200)
use caps --uid instead for now

lxcsu-internal

index 238d5e0..e303583 100755 (executable)
@@ -176,11 +176,37 @@ def main ():
 
     pid = os.fork()
 
 
     pid = os.fork()
 
+    # capsh has a --user option starting with f14
+    # so if only for f12 we need to fake this one
+    #
+    # capsh.c does essentially the following when invoked with --user:
+    #      pwd = getpwnam(user); ...
+    #      ngroups = MAX_GROUPS; 
+    #      status = getgrouplist(user, pwd->pw_gid, groups, &ngroups); ...
+    #      status = setgroups(ngroups, groups); ...
+    #      status = setgid(pwd->pw_gid); ...
+    #      status = setuid(pwd->pw_uid); ...
+    # however we cannot simulate that ourselves because if we did in this process then
+    # capsh could not be allowed to mess with caps any more
+
+    def getuid (slicename):
+        import pwd
+        try:
+            return pwd.getpwnam(slicename).pw_uid
+        except:
+            return
+
     if (pid == 0):
         cap_arg = '--drop='+drop_capabilities
 
         if (not args.root):
     if (pid == 0):
         cap_arg = '--drop='+drop_capabilities
 
         if (not args.root):
-            exec_args = [arch,'/usr/sbin/capsh',cap_arg,'--user=%s'%slice_name,'--','--login',]+args.command_to_run
+            uid = getuid (slice_name)
+            if not uid:
+                print "lxcsu-internal could not spot %s in /etc/passwd - exiting"%slice_name
+                exit(1)
+            exec_args = [arch,'/usr/sbin/capsh',cap_arg,'--uid=%s'%uid,'--','--login',]+args.command_to_run
+# once we can drop f12, it would be nicer to instead go for
+#            exec_args = [arch,'/usr/sbin/capsh',cap_arg,'--user=%s'%slice_name,'--','--login',]+args.command_to_run
         else:
             exec_args = [arch,'/usr/sbin/capsh','--','--login']+args.command_to_run
 
         else:
             exec_args = [arch,'/usr/sbin/capsh','--','--login']+args.command_to_run