From owner-svn-src-head@FreeBSD.ORG Fri Aug 22 15:31:45 2014 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 35FA4833; Fri, 22 Aug 2014 15:31:45 +0000 (UTC) Received: from kib.kiev.ua (kib.kiev.ua [IPv6:2001:470:d5e7:1::1]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id C910F3034; Fri, 22 Aug 2014 15:31:44 +0000 (UTC) Received: from tom.home (kib@localhost [127.0.0.1]) by kib.kiev.ua (8.14.9/8.14.9) with ESMTP id s7MFVdx0067461 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Fri, 22 Aug 2014 18:31:39 +0300 (EEST) (envelope-from kostikbel@gmail.com) DKIM-Filter: OpenDKIM Filter v2.9.2 kib.kiev.ua s7MFVdx0067461 Received: (from kostik@localhost) by tom.home (8.14.9/8.14.9/Submit) id s7MFVdJd067460; Fri, 22 Aug 2014 18:31:39 +0300 (EEST) (envelope-from kostikbel@gmail.com) X-Authentication-Warning: tom.home: kostik set sender to kostikbel@gmail.com using -f Date: Fri, 22 Aug 2014 18:31:39 +0300 From: Konstantin Belousov To: Jilles Tjoelker Subject: Re: svn commit: r265003 - head/secure/usr.sbin/sshd Message-ID: <20140822153139.GN2737@kib.kiev.ua> References: <201404270528.s3R5SEIm054377@svn.freebsd.org> <53F4B381.5010205@FreeBSD.org> <20140820151310.GB2737@kib.kiev.ua> <53F4BC9B.3090405@FreeBSD.org> <53F4BEB1.6070000@FreeBSD.org> <53F4C022.5050804@FreeBSD.org> <20140821080541.GE2737@kib.kiev.ua> <53F5D42E.9080908@FreeBSD.org> <20140821123246.GH2737@kib.kiev.ua> <20140822134353.GA21891@stack.nl> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="LlyFhrrzptpgNMGs" Content-Disposition: inline In-Reply-To: <20140822134353.GA21891@stack.nl> User-Agent: Mutt/1.5.23 (2014-03-12) X-Spam-Status: No, score=-2.0 required=5.0 tests=ALL_TRUSTED,BAYES_00, DKIM_ADSP_CUSTOM_MED,FREEMAIL_FROM,NML_ADSP_CUSTOM_MED autolearn=no autolearn_force=no version=3.4.0 X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on tom.home Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, Roger Pau Monn? , src-committers@freebsd.org, Bryan Drewery X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 22 Aug 2014 15:31:45 -0000 --LlyFhrrzptpgNMGs Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Aug 22, 2014 at 03:43:53PM +0200, Jilles Tjoelker wrote: > This is good and necessary for SA_SIGINFO (because of the type of the > SIG_DFL and SIG_IGN constants, and because POSIX says so in the > description of SA_RESETHAND in the sigaction() page). However, there > seems no reason to clear the other flags, which have no effect when the > disposition is SIG_DFL or SIG_IGN but have historically always been > preserved. (Slight exception: if kernel code erroneously loops on > ERESTART, it will eat CPU time iff SA_RESTART is set, independent of the > signal's disposition.) Well, I already committed the patch with several bugs fixed comparing with what was mailed, before your feedback arrived. Do you consider it is important enough to revert the resetting of other flags ? In particular, your note about the traditional historic behaviour makes me wonder. I do not see why SA_SIGINFO is so special that it must be reset, while other flags are not. The absence of the cases where the default/ignored disposition is affected by the flags seems rather arbitrary. >=20 > I notice a bug in POSIX here: it should specify that execve() clear > SA_SIGINFO bits when it resets signals to SIG_DFL. >=20 > --=20 > Jilles Tjoelker --LlyFhrrzptpgNMGs Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBAgAGBQJT92JbAAoJEJDCuSvBvK1ByhsP/iggX+JU9BLduvXkQJMEXN7F aUuKwaidYPaulVVHAMHRHy8Bs2U3ZI3nmmBfGa8penqlJgm7U10dtRLRFgbufe+6 CgC293/eBU9U/5exF8M8hv36ieF2TqdAXKFjKDMWOaY4tFeoedP0ZR30Ywz6IyTt dIYSYD1ZP+5ChsnkjWV0ykMHRiO0cfYjOdNyYt81Pyc+V1imUFxPjjb13nuZRcde mjZOOY6VUnY2R4LPvT2uL62nDjrojD7Xz8SqWVVVx8GVpRVrwCIcz9S4uA6PYykV fkRIzf2SlgwTLwgLhPl4TkIlC59OWhPpEBPGnk/PRXNWHGO+4Ky1j0hBuAJdefhd Dafs4VROhWKGSj81mC5s0+L/3XuEnmD6gD/EzAVF9c2rx1zQKZhB5HjstlpgNtzg yGEokuCEC0qmLuUlmleaBiace9KkBibQAHhDNJ4OTt0c3dH++ZLMx7LtncrwYTS5 06+G1DgKsUopzksVJ2lV+mgme4IQH0uv1lYqWX38UIYsO6d41j7acnzOjQfZE2EW V5K1H9Jb35A39qrUo6cPPal/YR1glKO413M53s8Ognbox9yxRkiMOuXNpBETU4Oo dnmWjByi0sAw5rB+/GBuS01shL9AREVeF+plAX1qH6fUenh8bOwx/xgNmSdt5dUM PLr3UHJqVfX8F/r33dEJ =HYcX -----END PGP SIGNATURE----- --LlyFhrrzptpgNMGs--