exec plugin: Don't pass an invalid egid to setgroups() if no group was given.
authorSebastian Harl <sh@tokkee.org>
Sun, 24 Feb 2008 11:17:24 +0000 (12:17 +0100)
committerFlorian Forster <octo@huhu.verplant.org>
Mon, 25 Feb 2008 07:29:35 +0000 (08:29 +0100)
In case that no group has been specified in the config file, egid = -1 used to
be passed to setgroups() which, obviously, is not what we want.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>
src/exec.c

index b8b538b..0ab259d 100644 (file)
@@ -187,7 +187,7 @@ static void exec_child (program_list_t *pl)
     glist[0] = gid;
     glist_len = 1;
 
-    if (gid != egid)
+    if ((gid != egid) && (egid != -1))
     {
       glist[1] = egid;
       glist_len = 2;