From: pfpulux <pfpulux@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: [PR PATCH] incron: add patch to fix create zombie processes
Date: Tue, 23 Mar 2021 19:00:10 +0100 [thread overview]
Message-ID: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-29704@inbox.vuxu.org> (raw)
[-- Attachment #1: Type: text/plain, Size: 391 bytes --]
There is a new pull request by pfpulux against master on the void-packages repository
https://github.com/pf4sh/void-packages incron
https://github.com/void-linux/void-packages/pull/29704
incron: add patch to fix create zombie processes
see: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=930526
A patch file from https://github.com/void-linux/void-packages/pull/29704.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-incron-29704.patch --]
[-- Type: text/x-diff, Size: 2020 bytes --]
From ec45a027b19dc9ffc0c5f63f239979d860d05905 Mon Sep 17 00:00:00 2001
From: Pulux <pulux@pf4sh.de>
Date: Tue, 23 Mar 2021 18:55:58 +0100
Subject: [PATCH] incron: add patch to fix create zombie processes
see: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=930526
---
.../patches/incron-0.5.12-prevent-zombies.patch | 11 +++++++++++
srcpkgs/incron/template | 3 ++-
2 files changed, 13 insertions(+), 1 deletion(-)
create mode 100644 srcpkgs/incron/patches/incron-0.5.12-prevent-zombies.patch
diff --git a/srcpkgs/incron/patches/incron-0.5.12-prevent-zombies.patch b/srcpkgs/incron/patches/incron-0.5.12-prevent-zombies.patch
new file mode 100644
index 000000000000..63766c999509
--- /dev/null
+++ b/srcpkgs/incron/patches/incron-0.5.12-prevent-zombies.patch
@@ -0,0 +1,11 @@
+diff -Nur incron-0.5.12.orig/icd-main.cpp incron-0.5.12/icd-main.cpp
+--- incron-0.5.12.orig/icd-main.cpp 2019-01-05 11:43:19.722640603 -0800
++++ incron-0.5.12/icd-main.cpp 2019-01-05 11:45:41.236340779 -0800
+@@ -105,6 +105,7 @@
+ g_fFinish = true;
+ break;
+ case SIGCHLD:
++ do {} while (waitpid((pid_t)-1, 0, WNOHANG) > 0); /* Prevent zombies */
+ // first empty pipe (to prevent internal buffer overflow)
+ do {} while (read(g_cldPipe[0], g_cldPipeBuf, CHILD_PIPE_BUF_LEN) > 0);
+
diff --git a/srcpkgs/incron/template b/srcpkgs/incron/template
index 19c08b4befaa..cd7e3f34eaed 100644
--- a/srcpkgs/incron/template
+++ b/srcpkgs/incron/template
@@ -1,13 +1,14 @@
# Template file for 'incron'
pkgname=incron
version=0.5.12
-revision=2
+revision=3
short_desc="Daemon that executes commands due to inotify events"
maintainer="allan <mail@may.mooo.com>"
license="X11"
homepage="https://github.com/ar-/incron"
distfiles="https://github.com/ar-/incron/archive/${version}.tar.gz"
checksum=cce80bd723bafce59f35464f2f851d02707e32efa102e2b941ed0e42bdd38f91
+patch_args="-Np1"
make_dirs="/var/spool/incron 0755 root root
/etc/incron.d 0755 root root"
next reply other threads:[~2021-03-23 18:00 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-23 18:00 pfpulux [this message]
2021-03-24 3:10 ` [PR PATCH] [Merged]: " ericonr
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-29704@inbox.vuxu.org \
--to=pfpulux@users.noreply.github.com \
--cc=ml@inbox.vuxu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).