From 7df0cdad2656f7efa75b490355fb5db8a3ba057a Mon Sep 17 00:00:00 2001 From: Michael Mair-Keimberger Date: Sat, 30 Mar 2019 16:58:15 +0100 Subject: sys-apps/rng-tools: remove unused patch Signed-off-by: Michael Mair-Keimberger Closes: https://github.com/gentoo/gentoo/pull/11552 Signed-off-by: Aaron Bauman --- .../rng-tools/files/rng-tools-6-fix-noctty.patch | 45 ---------------------- 1 file changed, 45 deletions(-) delete mode 100644 sys-apps/rng-tools/files/rng-tools-6-fix-noctty.patch (limited to 'sys-apps/rng-tools') diff --git a/sys-apps/rng-tools/files/rng-tools-6-fix-noctty.patch b/sys-apps/rng-tools/files/rng-tools-6-fix-noctty.patch deleted file mode 100644 index e915150adb58..000000000000 --- a/sys-apps/rng-tools/files/rng-tools-6-fix-noctty.patch +++ /dev/null @@ -1,45 +0,0 @@ -From: Gokturk Yuksek -Subject: [PATCH] Fix rngd to open the entropy source with 'O_NOCTTY' flag - -When start-stop-daemon starts a rngd instance configured to use a tty -device as its entropy source, the application crashes due to not being -able to read from the entropy device. This is caused by -start-stop-daemon calling setsid() before executing rngd, which -disassociates the controlling terminal. When rngd attempts to open a -hardware entropy source that's a tty device, per POSIX rules, the -device becomes the controlling terminal for the process. Then rngd -calls daemon(), which internally calls setsid(), and consequently -disassociates the controlling terminal for the child. Meanwhile the -parent rngd process exits. This results in tty device hanging up. By -looking at the strace logs attached to the bug, it can be observed -that although the parent rngd process is able to read() from the -entropy source successfully, further attempts to read() by the child -rngd process return 0. This complies with the POSIX, which states that -read() calls on a hung up terminal shall return 0. - -Note that when rngd is started without start-stop-daemon, this problem -does not happen because at the time of opening the entropy source rngd -already has a controlling terminal. - -Prevent the entropy source from becoming the controlling terminal by -passing 'O_NOCTTY' flag to open() when opening an entropy source. This -flag prevents a tty device from becoming the controlling terminal for -a process without a controlling terminal at the time of open(). - -Thanks to John Bowler for debugging the problem and -pinpointing the issue as well as confirming the fix. - -Gentoo-Bug-URL: https://bugs.gentoo.org/556456 -Reported-By: John Bowler - ---- rng-tools-rng-tools-6/rngd_entsource.c -+++ rng-tools-rng-tools-6/rngd_entsource.c -@@ -162,7 +162,7 @@ - struct sysfs_attribute *rngavail; - char buf[16]; - -- ent_src->rng_fd = open(ent_src->rng_fname, O_RDONLY); -+ ent_src->rng_fd = open(ent_src->rng_fname, O_RDONLY | O_NOCTTY); - if (ent_src->rng_fd == -1) { - message(LOG_DAEMON|LOG_DEBUG, "Unable to open file: %s", ent_src->rng_fname); - return 1; -- cgit v1.2.3-65-gdbad