From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id 77FE91382C5 for ; Fri, 14 May 2021 22:14:17 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 6FDFDE0922; Fri, 14 May 2021 22:14:14 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 6B4E4E091D for ; Fri, 14 May 2021 22:14:13 +0000 (UTC) Received: from oystercatcher.gentoo.org (oystercatcher.gentoo.org [148.251.78.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id 12BFC340EA6 for ; Fri, 14 May 2021 22:14:12 +0000 (UTC) Received: from localhost.localdomain (localhost [IPv6:::1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id 4CB7A79B for ; Fri, 14 May 2021 22:14:08 +0000 (UTC) From: "David Seifert" To: gentoo-commits@lists.gentoo.org Content-Transfer-Encoding: 8bit Content-type: text/plain; charset=UTF-8 Reply-To: gentoo-dev@lists.gentoo.org, "David Seifert" Message-ID: <1621030430.2b243c164b0c2233e2301a6cd8d806bb28b7d41e.soap@gentoo> Subject: [gentoo-commits] repo/gentoo:master commit in: sys-process/procps/, sys-process/procps/files/ X-VCS-Repository: repo/gentoo X-VCS-Files: sys-process/procps/Manifest sys-process/procps/files/procps-3.3.16-SC_ARG_MAX_sanity_check.patch sys-process/procps/files/procps-3.3.16-toprc_backwards_compatibility.patch sys-process/procps/procps-3.3.16-r2.ebuild sys-process/procps/procps-3.3.16-r3.ebuild X-VCS-Directories: sys-process/procps/files/ sys-process/procps/ X-VCS-Committer: soap X-VCS-Committer-Name: David Seifert X-VCS-Revision: 2b243c164b0c2233e2301a6cd8d806bb28b7d41e X-VCS-Branch: master Date: Fri, 14 May 2021 22:14:08 +0000 (UTC) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-commits@lists.gentoo.org X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-Archives-Salt: 1e717d01-41f9-4526-bbac-b6bdf7087bd2 X-Archives-Hash: dbe0dc03177d02f4bffb4ccdd6a049e2 commit: 2b243c164b0c2233e2301a6cd8d806bb28b7d41e Author: David Seifert gentoo org> AuthorDate: Fri May 14 22:13:50 2021 +0000 Commit: David Seifert gentoo org> CommitDate: Fri May 14 22:13:50 2021 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=2b243c16 sys-process/procps: drop 3.3.16-r2, 3.3.16-r3 Signed-off-by: David Seifert gentoo.org> sys-process/procps/Manifest | 1 - .../procps-3.3.16-SC_ARG_MAX_sanity_check.patch | 60 --------------- ...ocps-3.3.16-toprc_backwards_compatibility.patch | 71 ----------------- sys-process/procps/procps-3.3.16-r2.ebuild | 87 --------------------- sys-process/procps/procps-3.3.16-r3.ebuild | 88 ---------------------- 5 files changed, 307 deletions(-) diff --git a/sys-process/procps/Manifest b/sys-process/procps/Manifest index 7c9362d1b1c..f9a2cf039d3 100644 --- a/sys-process/procps/Manifest +++ b/sys-process/procps/Manifest @@ -1,2 +1 @@ -DIST procps-ng-3.3.16.tar.xz 859736 BLAKE2B fbacc1edb0711cda0fecbf5f589c8cc18bd74f0430866e946f7c4612b78f383b9299840ab7fa5bbc65c581cc9484dbb37b111c493cf88f55ba247a343fd0c80d SHA512 d83da472df256b188c32082632202e7f8ebd1b161082387760876ae34d50221b3682299a2816a7d6a29afb40322743c31b87d92ca299c2944a6b55e50736d367 DIST procps-ng-3.3.17.tar.xz 1008428 BLAKE2B 43aa1a21d4f0725b1c02457f343cf1fc66bc1771b25c6eaf689c9775c190e90545bfac1729f027abc4d3d13de37ffac4828554b5b25e12bcf6af9540de1695f3 SHA512 59e9a5013430fd9da508c4655d58375dc32e025bb502bb28fb9a92a48e4f2838b3355e92b4648f7384b2050064d17079bf4595d889822ebb5030006bc154a1a7 diff --git a/sys-process/procps/files/procps-3.3.16-SC_ARG_MAX_sanity_check.patch b/sys-process/procps/files/procps-3.3.16-SC_ARG_MAX_sanity_check.patch deleted file mode 100644 index 1b85c2c855f..00000000000 --- a/sys-process/procps/files/procps-3.3.16-SC_ARG_MAX_sanity_check.patch +++ /dev/null @@ -1,60 +0,0 @@ -From bb96fc42956c9ed926a1b958ab715f8b4a663dec Mon Sep 17 00:00:00 2001 -From: Craig Small -Date: Sun, 5 Jan 2020 15:05:55 +1100 -Subject: [PATCH] pgrep: check sanity of SC_ARG_MAX - -A kernel change means we cannot trust what sysconf(SC_ARG_MAX) -returns. We clamp it so its more than 4096 and less than 128*1024 -which is what findutils does. - -References: - procps-ng/procps#152 - https://git.savannah.gnu.org/cgit/findutils.git/tree/lib/buildcmd.c#n535 - https://lwn.net/Articles/727862/ ---- - pgrep.c | 22 +++++++++++++++++++++- - 1 file changed, 21 insertions(+), 1 deletion(-) - -diff --git a/pgrep.c b/pgrep.c -index 01563db..bde7448 100644 ---- a/pgrep.c -+++ b/pgrep.c -@@ -485,6 +485,26 @@ static regex_t * do_regcomp (void) - return preg; - } - -+/* -+ * SC_ARG_MAX used to return the maximum size a command line can be -+ * however changes to the kernel mean this can be bigger than we can -+ * alloc. Clamp it to 128kB like xargs and friends do -+ * Should also not be smaller than POSIX_ARG_MAX which is 4096 -+ */ -+static size_t get_arg_max(void) -+{ -+#define MIN_ARG_SIZE 4096u -+#define MAX_ARG_SIZE (128u * 1024u) -+ -+ size_t val = sysconf(_SC_ARG_MAX); -+ -+ if (val < MIN_ARG_SIZE) -+ val = MIN_ARG_SIZE; -+ if (val > MAX_ARG_SIZE) -+ val = MAX_ARG_SIZE; -+ -+ return val; -+} - static struct el * select_procs (int *num) - { - PROCTAB *ptp; -@@ -497,7 +517,7 @@ static struct el * select_procs (int *num) - regex_t *preg; - pid_t myself = getpid(); - struct el *list = NULL; -- long cmdlen = sysconf(_SC_ARG_MAX) * sizeof(char); -+ long cmdlen = get_arg_max() * sizeof(char); - char *cmdline = xmalloc(cmdlen); - char *cmdsearch = xmalloc(cmdlen); - char *cmdoutput = xmalloc(cmdlen); --- -GitLab - diff --git a/sys-process/procps/files/procps-3.3.16-toprc_backwards_compatibility.patch b/sys-process/procps/files/procps-3.3.16-toprc_backwards_compatibility.patch deleted file mode 100644 index 19721205dfe..00000000000 --- a/sys-process/procps/files/procps-3.3.16-toprc_backwards_compatibility.patch +++ /dev/null @@ -1,71 +0,0 @@ -From 5cd29e5093efa3c6ee9c5310b64347f1d54b707d Mon Sep 17 00:00:00 2001 -From: Jim Warner -Date: Sat, 15 Feb 2020 00:00:00 -0600 -Subject: [PATCH] top: restore configuration file backward compatibility - -The Debian bug referenced below has nothing to do with -locales. In fact, top was made locale independent back -in release 3.3.13 (April, 2018). However, that bug did -reveal some misplaced logic which this patch corrects. - -Prompted by the Qualys audit, all rcfile field strings -were checked for potential duplicates which could only -have resulted from some user's manual/malicious edits. - -Unfortunately, that code was executed before top had a -chance to enforce the proper/maximum string length (in -the event an extremely old rcfile had just been read). -This created some potential string overrun references. - -In top's original 3.3.15 implementation, the potential -overrun extended for 15 characters. That is the number -of field characters added with 3.3.9 (December, 2013). -But, since strchr() was used, no error exit was taken. - -In the revised 3.3.16 implementation, the strchr() was -replaced with '&w->rc.fieldscur[n]'. This held overrun -to a single position while producing an error message. - -So, this commit just moves that logic to a point where -fieldscur is guaranteed to be longer than EU_MAXPFLGS. - -Reference(s): -https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=951335 -. revised 3.3.16 validation logic -commit 291d98ee5036567f93d21bc11142b0a7e2ee70ae -. original 3.3.15 validation logic -commit fdb58974e24c025a1f866f324c62f1d8f96234f8 - -Signed-off-by: Jim Warner ---- -diff --git a/top/top.c b/top/top.c -index 63ec5fe..b4fe21e 100644 ---- a/top/top.c -+++ b/top/top.c -@@ -3939,11 +3939,6 @@ static const char *configs_file (FILE *fp, const char *name, float *delay) { - // too bad fscanf is not as flexible with his format string as snprintf - #error Hey, fix the above fscanf 'PFLAGSSIZ' dependency ! - #endif -- // ensure there's been no manual alteration of fieldscur -- for (n = 0 ; n < EU_MAXPFLGS; n++) { -- if (&w->rc.fieldscur[n] != strrchr(w->rc.fieldscur, w->rc.fieldscur[n])) -- return p; -- } - // be tolerant of missing release 3.3.10 graph modes additions - if (3 > fscanf(fp, "\twinflags=%d, sortindx=%d, maxtasks=%d, graph_cpus=%d, graph_mems=%d\n" - , &w->rc.winflags, &w->rc.sortindx, &w->rc.maxtasks, &w->rc.graph_cpus, &w->rc.graph_mems)) -@@ -3989,6 +3984,11 @@ static const char *configs_file (FILE *fp, const char *name, float *delay) { - return p; - break; - } -+ // ensure there's been no manual alteration of fieldscur -+ for (n = 0 ; n < EU_MAXPFLGS; n++) { -+ if (&w->rc.fieldscur[n] != strrchr(w->rc.fieldscur, w->rc.fieldscur[n])) -+ return p; -+ } - #ifndef USE_X_COLHDR - OFFw(w, NOHIFND_xxx | NOHISEL_xxx); - #endif --- -2.24.1 - diff --git a/sys-process/procps/procps-3.3.16-r2.ebuild b/sys-process/procps/procps-3.3.16-r2.ebuild deleted file mode 100644 index fc0db915eef..00000000000 --- a/sys-process/procps/procps-3.3.16-r2.ebuild +++ /dev/null @@ -1,87 +0,0 @@ -# Copyright 1999-2021 Gentoo Authors -# Distributed under the terms of the GNU General Public License v2 - -EAPI=7 - -inherit flag-o-matic multilib-minimal usr-ldscript - -DESCRIPTION="Standard informational utilities and process-handling tools" -HOMEPAGE="http://procps-ng.sourceforge.net/ https://gitlab.com/procps-ng/procps" -SRC_URI="mirror://sourceforge/${PN}-ng/${PN}-ng-${PV}.tar.xz" - -LICENSE="GPL-2" -SLOT="0/8" # libprocps.so -KEYWORDS="~alpha amd64 arm arm64 hppa ~ia64 ~m68k ~mips ppc ppc64 ~riscv ~s390 sparc x86 ~amd64-linux ~x86-linux" -IUSE="elogind +kill modern-top +ncurses nls selinux static-libs systemd test unicode" -RESTRICT="!test? ( test )" - -COMMON_DEPEND=" - elogind? ( sys-auth/elogind ) - ncurses? ( >=sys-libs/ncurses-5.7-r7:=[unicode?] ) - selinux? ( sys-libs/libselinux[${MULTILIB_USEDEP}] ) - systemd? ( sys-apps/systemd[${MULTILIB_USEDEP}] ) -" -DEPEND="${COMMON_DEPEND} - elogind? ( virtual/pkgconfig ) - ncurses? ( virtual/pkgconfig ) - systemd? ( virtual/pkgconfig ) - test? ( dev-util/dejagnu )" -RDEPEND=" - ${COMMON_DEPEND} - kill? ( - !sys-apps/coreutils[kill] - !sys-apps/util-linux[kill] - ) - !