Github messages for voidlinux
 help / color / mirror / Atom feed
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"

             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).