Date: Fri, 23 Dec 2011 12:06:44 -0500 From: Alexander Kabaev <kabaev@gmail.com> To: John Baldwin <jhb@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Colin Percival <cperciva@freebsd.org> Subject: Re: svn commit: r228843 - head/contrib/telnet/libtelnet head/crypto/heimdal/appl/telnet/libtelnet head/include head/lib/libc/gen head/lib/libc/iconv head/lib/libc/include head/lib/libc/net head/libexec... Message-ID: <20111223120644.75fe944d@kan.dyndns.org> In-Reply-To: <201112231122.34436.jhb@freebsd.org> References: <201112231500.pBNF0c0O071712@svn.freebsd.org> <201112231058.46642.jhb@freebsd.org> <201112231122.34436.jhb@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--Sig_/HBu.brn+1n7ff.WMuAnuq7Z Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Fri, 23 Dec 2011 11:22:34 -0500 John Baldwin <jhb@freebsd.org> wrote: > On Friday, December 23, 2011 10:58:46 am John Baldwin wrote: > > On Friday, December 23, 2011 10:00:38 am Colin Percival wrote: > > > Author: cperciva > > > Date: Fri Dec 23 15:00:37 2011 > > > New Revision: 228843 > > > URL: http://svn.freebsd.org/changeset/base/228843 > > >=20 > > > Log: > > > Fix a problem whereby a corrupt DNS record can cause named to > > > crash. [11:06]=20 > > > Add an API for alerting internal libc routines to the presence > > > of "unsafe" paths post-chroot, and use it in ftpd. [11:07] > >=20 > > Eh, the whole libc_dlopen() thing looks like a gross hack (and who > > came up with that weird symbol name for a public API????). Is it > > really even needed given the other fix to have ftpd drop privilege > > before execing a helper program? I guess the main reason I don't > > like it is it doesn't do anything to address the more general > > problem. I would have expected instead something to restrict > > dlopen() entirely including from other libraries than just libc in > > certain circumstances. >=20 > At the very least if we feel that the libc_dlopen() thing is a > temporary band-aid, we should move the new symbols into the private > namespace so we can remove them once the better fix is in rather than > being required to support them forever. >=20 > --=20 > John Baldwin Pardon for not catching that when I had a chance to influence the outcome, but I would like to voice my support to tucking the ugliness into private version namespace. --=20 Alexander Kabaev --Sig_/HBu.brn+1n7ff.WMuAnuq7Z Content-Type: application/pgp-signature; name=signature.asc Content-Disposition: attachment; filename=signature.asc -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.18 (FreeBSD) iD8DBQFO9LUoQ6z1jMm+XZYRAioCAJ4xZWtZWyGgyNxN5yWp6YuMMyAnTQCg2BzR RZYi7OL5KgkSfKzX3gYMslU= =NSef -----END PGP SIGNATURE----- --Sig_/HBu.brn+1n7ff.WMuAnuq7Z--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20111223120644.75fe944d>