PR/59645 CVS commit: src/tests/usr.sbin/inetd

["Taylor R Campbell" <riastradh%netbsd.org@localhost>]

The following reply was made to PR bin/59645; it has been noted by GNATS.

From: "Taylor R Campbell" <riastradh%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/59645 CVS commit: src/tests/usr.sbin/inetd
Date: Tue, 23 Jun 2026 18:53:29 +0000

 Module Name:	src
 Committed By:	riastradh
 Date:		Tue Jun 23 18:53:29 UTC 2026
 
 Modified Files:
 	src/tests/usr.sbin/inetd: t_accept_max.sh
 
 Log Message:
 tests/usr.sbin/inetd/t_accept_max: Handle SIGKRE.
 
 Fix some cleanup issues and simplify according to most of kre's
 suggestions.  Mostly this fixes the kill-background-jobs-on-trap
 logic by not trying to kill the nonexistent %0, and killing each job
 individually so `kill' doesn't stop early if one doesn't exist.
 
 I'm keeping the idiom `: >foo' and `: <foo' because I think it's a
 little clearer than a bare `>foo' or `<foo', and I'm keeping the -e
 and -u on shell scripts (except the one-liners) because I want to get
 early feedback as much as possible about unexpected failure modes --
 even if it counterintuitively elides some feedback.
 
 (I also generally use -o pipefail out of habit for the same reason
 but I'll acquiesce to removing it here in a short script where there
 are definitely no pipes.)
 
 PR bin/59645: inetd `rate-limiting' algorithm is stupid
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.3 -r1.4 src/tests/usr.sbin/inetd/t_accept_max.sh
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.