From owner-svn-src-head@FreeBSD.ORG Tue Nov 20 14:05:46 2012 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id C0F8119A; Tue, 20 Nov 2012 14:05:46 +0000 (UTC) (envelope-from bapt@FreeBSD.org) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:1900:2254:2068::e6a:0]) by mx1.freebsd.org (Postfix) with ESMTP id A6C5D8FC18; Tue, 20 Nov 2012 14:05:46 +0000 (UTC) Received: from svn.freebsd.org (localhost [127.0.0.1]) by svn.freebsd.org (8.14.5/8.14.5) with ESMTP id qAKE5kKh032692; Tue, 20 Nov 2012 14:05:46 GMT (envelope-from bapt@svn.freebsd.org) Received: (from bapt@localhost) by svn.freebsd.org (8.14.5/8.14.5/Submit) id qAKE5kK6032691; Tue, 20 Nov 2012 14:05:46 GMT (envelope-from bapt@svn.freebsd.org) Message-Id: <201211201405.qAKE5kK6032691@svn.freebsd.org> From: Baptiste Daroussin Date: Tue, 20 Nov 2012 14:05:46 +0000 (UTC) To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r243335 - head/usr.sbin/pw X-SVN-Group: head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.14 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: Tue, 20 Nov 2012 14:05:46 -0000 Author: bapt Date: Tue Nov 20 14:05:46 2012 New Revision: 243335 URL: http://svnweb.freebsd.org/changeset/base/243335 Log: In NIS mode first chmod(2) the temporary file and is succeed then rename(2) Modified: head/usr.sbin/pw/pw_nis.c Modified: head/usr.sbin/pw/pw_nis.c ============================================================================== --- head/usr.sbin/pw/pw_nis.c Tue Nov 20 14:03:09 2012 (r243334) +++ head/usr.sbin/pw/pw_nis.c Tue Nov 20 14:05:46 2012 (r243335) @@ -66,10 +66,10 @@ pw_nisupdate(const char * path, struct p pw_fini(); err(1, "pw_copy()"); } + if (chmod(pw_tempname(), 0644) == -1) + err(1, "chmod()"); if (rename(pw_tempname(), path) == -1) err(1, "rename()"); - if (chmod(path, 0644) == -1) - err(1, "chmod()"); free(pw); pw_fini();