From owner-freebsd-security@FreeBSD.ORG Mon Oct 1 11:39:27 2012 Return-Path: Delivered-To: freebsd-security@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 0017B1065673 for ; Mon, 1 Oct 2012 11:39:26 +0000 (UTC) (envelope-from erik@cederstrand.dk) Received: from csmtp2.one.com (csmtp2.one.com [91.198.169.22]) by mx1.freebsd.org (Postfix) with ESMTP id B1E4A8FC08 for ; Mon, 1 Oct 2012 11:39:26 +0000 (UTC) Received: from [192.168.1.18] (unknown [217.157.7.221]) by csmtp2.one.com (Postfix) with ESMTPA id 221193043E31; Mon, 1 Oct 2012 11:39:25 +0000 (UTC) Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 6.0 \(1486\)) From: Erik Cederstrand In-Reply-To: <20121001110805.GL35915@deviant.kiev.zoral.com.ua> Date: Mon, 1 Oct 2012 13:39:33 +0200 Content-Transfer-Encoding: quoted-printable Message-Id: <2E17A82C-16D8-4DCD-86F0-93C28C5C4257@cederstrand.dk> References: <9DD86238-51C8-4F38-B7EB-BD773039888B@cederstrand.dk> <20121001104901.GJ35915@deviant.kiev.zoral.com.ua> <20121001110805.GL35915@deviant.kiev.zoral.com.ua> To: Konstantin Belousov X-Mailer: Apple Mail (2.1486) Cc: "freebsd-security@freebsd.org" Subject: Re: Opinion on checking return value of setuid(getuid())? X-BeenThere: freebsd-security@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Security issues \[members-only posting\]" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 01 Oct 2012 11:39:27 -0000 Den 01/10/2012 kl. 13.08 skrev Konstantin Belousov = : >=20 > I do not believe in the dreadful 'flood ping' security breach. Is a > local escalation possible with non-dropped root ? No idea. Reading the code, I see some functionality the author decided = should only be accessible to root users. There's 600 lines of code left = in main() and I'm not skilled enough to see if there are any potential = exploits left. If it's not a security breach then I'm on the wrong list, but I guess it = still leads to unintended behavior if setuid() fails? Erik=