Messages in this thread | | | From | Andrey Mirkin <> | Subject | Re: [Devel] Re: [PATCH 05/10] Introduce function to dump process | Date | Fri, 24 Oct 2008 08:40:16 +0400 |
| |
On Monday 20 October 2008 21:48 Serge E. Hallyn wrote: > Quoting Andrey Mirkin (major@openvz.org): > > + t->cpt_uid = tsk->uid; > > + t->cpt_euid = tsk->euid; > > + t->cpt_suid = tsk->suid; > > + t->cpt_fsuid = tsk->fsuid; > > + t->cpt_gid = tsk->gid; > > + t->cpt_egid = tsk->egid; > > + t->cpt_sgid = tsk->sgid; > > + t->cpt_fsgid = tsk->fsgid; > > I don't see where any of these are restored. (Obviously, I wanted > to think about how you're verifying the restarter's authorization > to do so)
Well, right now I don't use them during restore to simplify restart procedure and make it more clear for reviewers. In OpenVZ we are doing all restart procedure with root's privileges and relying on fact that all such IDs will be the same during restart (as we are restarting a container and its file system will be the same during restart).
Andrey
| |