diff options
| author | Andrew Clayton <a.clayton@nginx.com> | 2022-11-19 23:58:51 +0000 |
|---|---|---|
| committer | Andrew Clayton <a.clayton@nginx.com> | 2022-11-19 23:58:51 +0000 |
| commit | 58812e74d3a0ab14b528933e1953ae28c8f9f627 (patch) | |
| tree | 7fea76965bcb62478a6c8a3f04479de287b7aa05 | |
| parent | b9c1a2977b161e6befbc1fa770f6fe482ea3e9ca (diff) | |
| download | unit-58812e74d3a0ab14b528933e1953ae28c8f9f627.tar.gz unit-58812e74d3a0ab14b528933e1953ae28c8f9f627.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; |