From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Serge E. Hallyn" Subject: [PATCH user-cr] use syscall(NR_getpid) to avoid cached pid Date: Tue, 16 Feb 2010 18:24:09 -0600 Message-ID: <20100217002409.GA3267@us.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: Oren Laadan Cc: Linux Containers List-Id: containers.vger.kernel.org otherwise glibc caches getpid() result and we put the parent process in the new cgroup, in the case of 'nsexec -cg /bin/sleep 10' Signed-off-by: Serge Hallyn --- nsexec.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/nsexec.c b/nsexec.c index 3a825f1..79fcc70 100644 --- a/nsexec.c +++ b/nsexec.c @@ -154,7 +154,7 @@ int move_to_new_cgroup(int newcgroup) fout = fopen(tasksfname, "w"); if (!fout) return 0; - fprintf(fout, "%d\n", getpid()); + fprintf(fout, "%d\n", syscall(__NR_getpid)); fclose(fout); return 1; } @@ -193,7 +193,7 @@ int do_child(void *vargv) /* if pid == 1 then remount /proc */ /* But if the container has no /proc don't fret */ - if (getpid() == 1) { + if (syscall(__NR_getpid) == 1) { umount2("/proc", MNT_DETACH); mount("proc", "/proc", "proc", 0, NULL); } -- 1.6.4.4