From owner-cvs-src-old@FreeBSD.ORG Sun Jan 24 14:31:18 2010 Return-Path: Delivered-To: cvs-src-old@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 999181065676 for ; Sun, 24 Jan 2010 14:31:18 +0000 (UTC) (envelope-from ed@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 5C2678FC1D for ; Sun, 24 Jan 2010 14:31:18 +0000 (UTC) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.3/8.14.3) with ESMTP id o0OEVIj7060763 for ; Sun, 24 Jan 2010 14:31:18 GMT (envelope-from ed@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.3/8.14.3/Submit) id o0OEVIlI060762 for cvs-src-old@freebsd.org; Sun, 24 Jan 2010 14:31:18 GMT (envelope-from ed@repoman.freebsd.org) Message-Id: <201001241431.o0OEVIlI060762@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to ed@repoman.freebsd.org using -f From: Ed Schouten Date: Sun, 24 Jan 2010 14:30:57 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: RELENG_8 Subject: cvs commit: src/lib/libutil login_tty.c X-BeenThere: cvs-src-old@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 24 Jan 2010 14:31:18 -0000 ed 2010-01-24 14:30:57 UTC FreeBSD src repository Modified files: (Branch: RELENG_8) lib/libutil login_tty.c Log: SVN rev 202929 on 2010-01-24 14:30:57Z by ed MFC r202500: Fix a regression that was introduced in r191882. I changed login_tty() to only work when the application is not a session leader yet. This works fine for applications in the base system, but it turns out various applications call this function after daemonizing, which means they already use their own session. If setsid() fails, just call tcsetsid() on the current session. tcsetsid() will already perform proper security checks. Reported by: Oliver Lehmann Revision Changes Path 1.11.2.2 +1 -1 src/lib/libutil/login_tty.c