Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 20 Nov 2012 14:03:09 +0000 (UTC)
From:      Baptiste Daroussin <bapt@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r243334 - head/lib/libutil
Message-ID:  <201211201403.qAKE39AM032117@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: bapt
Date: Tue Nov 20 14:03:09 2012
New Revision: 243334
URL: http://svnweb.freebsd.org/changeset/base/243334

Log:
  only rename(2) after chmod(2) has succeed
  report error if chmod(2) fails
  
  Reported by:	jh

Modified:
  head/lib/libutil/gr_util.c

Modified: head/lib/libutil/gr_util.c
==============================================================================
--- head/lib/libutil/gr_util.c	Tue Nov 20 12:32:18 2012	(r243333)
+++ head/lib/libutil/gr_util.c	Tue Nov 20 14:03:09 2012	(r243334)
@@ -318,14 +318,10 @@ gr_copy(int ffd, int tfd, const struct g
 int
 gr_mkdb(void)
 {
-	int ret;
+	if (chmod(tempname, 0644) != 0)
+		return (-1);
 
-	ret = rename(tempname, group_file);
-
-	if (ret == 0)
-		chmod(group_file, 0644);
-
-	return (ret);
+	return (rename(tempname, group_file));
 }
 
 /*



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201211201403.qAKE39AM032117>