user: Always return supplementary groups

The caller can ignore it if they don't care about using them. Signed-off-by: Mrunal Patel <[email protected]>
opencontainers · Oct 17, 2018 · 46b41a3 · 46b41a3
1 parent 58592df
commit 46b41a3
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/libcontainer/user/user.go b/libcontainer/user/user.go
@@ -417,8 +417,8 @@ func GetExecUser(userSpec string, defaults *ExecUser, passwd, group io.Reader) (
 
 				// Okay, so it's numeric. We can just roll with this.
 			}
-		} else if len(groups) > 0 {
-			// Supplementary group ids only make sense if in the implicit form.
+		}
+		if len(groups) > 0 {
 			user.Sgids = make([]int, len(groups))
 			for i, group := range groups {
 				user.Sgids[i] = group.Gid