On Tue, Mar 04, 2014 at 09:58:30AM -0600, Serge Hallyn wrote: > If the user maps container root to his host uid, chown_mapped_rootid > tries to make the same mapping twice and gets -EINVAL. > > Reported-by: Andy Whitcroft <[email protected]> > Signed-off-by: Serge Hallyn <[email protected]>
Acked-by: Stéphane Graber <[email protected]> > --- > src/lxc/conf.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/src/lxc/conf.c b/src/lxc/conf.c > index 2622371..1cb058d 100644 > --- a/src/lxc/conf.c > +++ b/src/lxc/conf.c > @@ -3439,6 +3439,13 @@ int chown_mapped_root(char *path, struct lxc_conf > *conf) > } > return 0; > } > + > + if (rootid == geteuid()) { > + // nothing to do > + INFO("%s: container root is our uid; no need to chown" > ,__func__); > + return 0; > + } > + > pid = fork(); > if (pid < 0) { > SYSERROR("Failed forking"); > -- > 1.9.0 > > _______________________________________________ > lxc-devel mailing list > [email protected] > http://lists.linuxcontainers.org/listinfo/lxc-devel -- Stéphane Graber Ubuntu developer http://www.ubuntu.com
signature.asc
Description: Digital signature
_______________________________________________ lxc-devel mailing list [email protected] http://lists.linuxcontainers.org/listinfo/lxc-devel
