Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] incron: add patch to fix create zombie processes
@ 2021-03-23 18:00 pfpulux
  2021-03-24  3:10 ` [PR PATCH] [Merged]: " ericonr
  0 siblings, 1 reply; 2+ messages in thread
From: pfpulux @ 2021-03-23 18:00 UTC (permalink / raw)
  To: ml

[-- 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"

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PR PATCH] [Merged]: incron: add patch to fix create zombie processes
  2021-03-23 18:00 [PR PATCH] incron: add patch to fix create zombie processes pfpulux
@ 2021-03-24  3:10 ` ericonr
  0 siblings, 0 replies; 2+ messages in thread
From: ericonr @ 2021-03-24  3:10 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 243 bytes --]

There's a merged pull request on the void-packages repository

incron: add patch to fix create zombie processes
https://github.com/void-linux/void-packages/pull/29704

Description:
see: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=930526

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-03-24  3:10 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-23 18:00 [PR PATCH] incron: add patch to fix create zombie processes pfpulux
2021-03-24  3:10 ` [PR PATCH] [Merged]: " ericonr

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