cbaba16b23
r285050 fixed a bug in pw that could lead to /etc/passwd or /etc/group corruption on power loss. However, it fixed it by opening those files with O_SYNC, which is very slow, especially on ZFS. This change replaces O_SYNC with appropriately placed fsync()s instead, which is much faster. Using a ZFS tmpdir, the time to run pw's kyua tests drops from 245s to 35s. Reviewed by: allanjude, bapt, vangyzen, garga Tested on pfSense by: garga MFC after: 4 weeks Sponsored by: Spectra Logic Corp Differential Revision: https://reviews.freebsd.org/D8319
100 lines
2.6 KiB
C
100 lines
2.6 KiB
C
/*-
|
|
* Copyright (C) 1996
|
|
* David L. Nugent. All rights reserved.
|
|
*
|
|
* Redistribution and use in source and binary forms, with or without
|
|
* modification, are permitted provided that the following conditions
|
|
* are met:
|
|
* 1. Redistributions of source code must retain the above copyright
|
|
* notice, this list of conditions and the following disclaimer.
|
|
* 2. Redistributions in binary form must reproduce the above copyright
|
|
* notice, this list of conditions and the following disclaimer in the
|
|
* documentation and/or other materials provided with the distribution.
|
|
*
|
|
* THIS SOFTWARE IS PROVIDED BY DAVID L. NUGENT AND CONTRIBUTORS ``AS IS'' AND
|
|
* ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
|
|
* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
|
|
* ARE DISCLAIMED. IN NO EVENT SHALL DAVID L. NUGENT OR CONTRIBUTORS BE LIABLE
|
|
* FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
|
|
* DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
|
|
* OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
|
|
* HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
|
|
* LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
|
|
* OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
|
|
* SUCH DAMAGE.
|
|
*/
|
|
|
|
#ifndef lint
|
|
static const char rcsid[] =
|
|
"$FreeBSD$";
|
|
#endif /* not lint */
|
|
|
|
#include <sys/types.h>
|
|
|
|
#include <err.h>
|
|
#include <pwd.h>
|
|
#include <libutil.h>
|
|
#include <unistd.h>
|
|
|
|
#include "pw.h"
|
|
|
|
static int
|
|
pw_nisupdate(const char * path, struct passwd * pwd, char const * user)
|
|
{
|
|
int pfd, tfd;
|
|
struct passwd *pw = NULL;
|
|
struct passwd *old_pw = NULL;
|
|
|
|
printf("===> %s\n", path);
|
|
if (pwd != NULL)
|
|
pw = pw_dup(pwd);
|
|
|
|
if (user != NULL)
|
|
old_pw = GETPWNAM(user);
|
|
|
|
if (pw_init(NULL, path))
|
|
err(1,"pw_init()");
|
|
if ((pfd = pw_lock()) == -1) {
|
|
pw_fini();
|
|
err(1, "pw_lock()");
|
|
}
|
|
if ((tfd = pw_tmp(-1)) == -1) {
|
|
pw_fini();
|
|
err(1, "pw_tmp()");
|
|
}
|
|
if (pw_copy(pfd, tfd, pw, old_pw) == -1) {
|
|
pw_fini();
|
|
close(tfd);
|
|
err(1, "pw_copy()");
|
|
}
|
|
fsync(tfd);
|
|
close(tfd);
|
|
if (chmod(pw_tempname(), 0644) == -1)
|
|
err(1, "chmod()");
|
|
if (rename(pw_tempname(), path) == -1)
|
|
err(1, "rename()");
|
|
|
|
free(pw);
|
|
pw_fini();
|
|
|
|
return (0);
|
|
}
|
|
|
|
int
|
|
addnispwent(const char *path, struct passwd * pwd)
|
|
{
|
|
return pw_nisupdate(path, pwd, NULL);
|
|
}
|
|
|
|
int
|
|
chgnispwent(const char *path, char const * login, struct passwd * pwd)
|
|
{
|
|
return pw_nisupdate(path, pwd, login);
|
|
}
|
|
|
|
int
|
|
delnispwent(const char *path, const char *login)
|
|
{
|
|
return pw_nisupdate(path, NULL, login);
|
|
}
|