[pkgsrc/trunk]: pkgsrc/net/delegate net/delegate: fix potential endless loop ...

[rillig <rillig%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/a9b3273a1e3f
branches:  trunk
changeset: 430728:a9b3273a1e3f
user:      rillig <rillig%pkgsrc.org@localhost>
date:      Sun May 03 17:22:50 2020 +0000

description:
net/delegate: fix potential endless loop in CGI part

Not reported upstream since the homepage is not reachable.

diffstat:

 net/delegate/Makefile |  16 ++++++++++++++--
 1 files changed, 14 insertions(+), 2 deletions(-)

diffs (33 lines):

diff -r 8c79e2219142 -r a9b3273a1e3f net/delegate/Makefile
--- a/net/delegate/Makefile     Sun May 03 16:49:32 2020 +0000
+++ b/net/delegate/Makefile     Sun May 03 17:22:50 2020 +0000
@@ -1,8 +1,8 @@
-# $NetBSD: Makefile,v 1.46 2019/11/03 11:45:33 rillig Exp $
+# $NetBSD: Makefile,v 1.47 2020/05/03 17:22:50 rillig Exp $
 
 DISTNAME=      delegate9.9.1
 PKGNAME=       delegate-9.9.1
-PKGREVISION=   1
+PKGREVISION=   2
 CATEGORIES=    net
 MASTER_SITES=  http://www.delegate.org/anonftp/DeleGate/
 MASTER_SITES+= ftp://ftp.u-aizu.ac.jp/pub/net/DeleGate/delegate.org/
@@ -13,6 +13,18 @@
 
 MAKE_JOBS_SAFE=                no
 
+# several usages of array[char]: cgi.c was dangerous, see below;
+# the other uses are all safe as of version 9.9.1.
+BUILDLINK_TRANSFORM+=  rm:-Werror=char-subscripts
+
+# Prevent undefined behavior when there are more than 127 environment
+# variables, in which case argv and the rest of the stack would be
+# interpreted as environment variables.
+SUBST_CLASSES+=                char
+SUBST_STAGE.char=      pre-configure
+SUBST_FILES.char=      src/cgi.c
+SUBST_SED.char=                -e 's|char ei;|int ei;|'
+
 .include "../../mk/bsd.prefs.mk"
 
 BUILD_DEFS+=           DELEGATE_MANAGER