diff options
author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2023-03-16 16:40:00 +0100 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2023-03-16 16:46:44 +0100 |
commit | 42fff80cb8fcd0df2d22d9875319a8e6c52efcae (patch) | |
tree | faf5399366b3fe60229d3f3e99948da2267f6f28 /src/udev/v4l_id | |
parent | 8a4c11e0e2ef39dc3e94b655e7bfba3d68e60104 (diff) | |
download | systemd-42fff80cb8fcd0df2d22d9875319a8e6c52efcae.tar.gz |
udev/v4l_id: use O_CLOEXEC|O_NOCTTY
This is the usual set of flags. O_CLOEXEC doesn't matter because we don't spawn
anything, but O_NOCTTY could possibly make a difference if the helper is called
on a wrong device type.
Diffstat (limited to 'src/udev/v4l_id')
-rw-r--r-- | src/udev/v4l_id/v4l_id.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/udev/v4l_id/v4l_id.c b/src/udev/v4l_id/v4l_id.c index 1d176a387e..30527e9556 100644 --- a/src/udev/v4l_id/v4l_id.c +++ b/src/udev/v4l_id/v4l_id.c @@ -74,7 +74,7 @@ static int run(int argc, char *argv[]) { if (r <= 0) return r; - fd = open(arg_device, O_RDONLY); + fd = open(arg_device, O_RDONLY|O_CLOEXEC|O_NOCTTY); if (fd < 0) return log_error_errno(errno, "Failed to open %s: %m", arg_device); |