Re: [PATCH libdrm] amdgpu: make sure to set CLOEXEC on duplicated FDs

2018-07-17 Thread Zhang, Jerry (Junwei)
On 07/17/2018 05:04 PM, Christian König wrote: Otherwise we leak file descriptors into child processes. Signed-off-by: Christian König Yeah, that's the key point that dup could remove CLOEXEC effect. Reviewed-and-Tested-by: Junwei Zhang --- amdgpu/amdgpu_device.c | 5 +++-- 1 file

Re: [PATCH libdrm] amdgpu: make sure to set CLOEXEC on duplicated FDs

2018-07-17 Thread Michel Dänzer
On 2018-07-17 11:04 AM, Christian König wrote: > Otherwise we leak file descriptors into child processes. > > Signed-off-by: Christian König > --- > amdgpu/amdgpu_device.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/amdgpu/amdgpu_device.c

[PATCH libdrm] amdgpu: make sure to set CLOEXEC on duplicated FDs

2018-07-17 Thread Christian König
Otherwise we leak file descriptors into child processes. Signed-off-by: Christian König --- amdgpu/amdgpu_device.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/amdgpu/amdgpu_device.c b/amdgpu/amdgpu_device.c index 34ac95b8..d7aec6a4 100644 ---