diff options
| author | Andrew Clayton <a.clayton@nginx.com> | 2022-11-19 23:58:51 +0000 |
|---|---|---|
| committer | Andrew Clayton <a.clayton@nginx.com> | 2023-02-17 21:24:18 +0000 |
| commit | 3351bb4ff5cd2ef4d8f7cfb87451f021bcccb37e (patch) | |
| tree | 6ae777d58d3b4a74b1568dff7445fee81e1369ab | |
| parent | 1388d311c6ead756e75cf3b7c616d054e57fe220 (diff) | |
| download | unit-3351bb4ff5cd2ef4d8f7cfb87451f021bcccb37e.tar.gz unit-3351bb4ff5cd2ef4d8f7cfb87451f021bcccb37e.tar.bz2 | |
Isolation: Remove the syscall(SYS_getpid) wrapper.
When using SYS_clone we used the getpid kernel system call directly via
syscall(SYS_getpid) to avoid issues with cached pids.
However since we are now only using fork(2) (+ unshare(2) for
namespaces) we no longer need to call the kernel getpid directly as the
fork(2) will ensure the cached pid is invalidated.
Reviewed-by: Alejandro Colomar <alx@nginx.com>
Signed-off-by: Andrew Clayton <a.clayton@nginx.com>
| -rw-r--r-- | src/nxt_process.h | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/src/nxt_process.h b/src/nxt_process.h index 1dd51521..de54e383 100644 --- a/src/nxt_process.h +++ b/src/nxt_process.h @@ -13,17 +13,8 @@ #endif -#if (NXT_HAVE_LINUX_NS) -/* - * Old glibc wrapper for getpid(2) returns a cached pid invalidated only by - * fork(2) calls. As we use clone(2) for container, it returns the wrong pid. - */ -#define nxt_getpid() \ - syscall(SYS_getpid) -#else #define nxt_getpid() \ getpid() -#endif typedef pid_t nxt_pid_t; |