 @@ -1,544 +1,550 @@
-/*	$NetBSD: vfs_syscalls_50.c,v 1.12 2012/01/29 06:29:04 dholland Exp $	*/
+/*	$NetBSD: vfs_syscalls_50.c,v 1.13 2012/01/29 07:16:00 dholland Exp $	*/
 /*-
  * Copyright (c) 2008 The NetBSD Foundation, Inc.
  * All rights reserved.
+ *
  * This code is derived from software contributed to The NetBSD Foundation
  * by Christos Zoulas.
+ *
  * 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.
  * 3. All advertising materials mentioning features or use of this software
  *    must display the following acknowledgement:
  *        This product includes software developed by the NetBSD
  *        Foundation, Inc. and its contributors.
  * 4. Neither the name of The NetBSD Foundation nor the names of its
  *    contributors may be used to endorse or promote products derived
  *    from this software without specific prior written permission.
+ *
  * THIS SOFTWARE IS PROVIDED BY THE NETBSD FOUNDATION, INC. 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 THE FOUNDATION 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.
  */
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: vfs_syscalls_50.c,v 1.12 2012/01/29 06:29:04 dholland Exp $");
+__KERNEL_RCSID(0, "$NetBSD: vfs_syscalls_50.c,v 1.13 2012/01/29 07:16:00 dholland Exp $");
 #include <sys/param.h>
 #include <sys/systm.h>
 #include <sys/namei.h>
 #include <sys/filedesc.h>
 #include <sys/kernel.h>
 #include <sys/file.h>
 #include <sys/stat.h>
 #include <sys/socketvar.h>
 #include <sys/vnode.h>
 #include <sys/mount.h>
 #include <sys/proc.h>
 #include <sys/uio.h>
 #include <sys/dirent.h>
 #include <sys/malloc.h>
 #include <sys/kauth.h>
 #include <sys/time.h>
 #include <sys/vfs_syscalls.h>
 #ifndef LFS
 #define LFS
 #endif
 #include <sys/syscallargs.h>
 #include <ufs/lfs/lfs_extern.h>
 #include <sys/quota.h>
 #include <quota/quotaprop.h>
 #include <ufs/ufs/quota1.h>
 #include <compat/common/compat_util.h>
 #include <compat/sys/time.h>
 #include <compat/sys/stat.h>
 #include <compat/sys/dirent.h>
 #include <compat/sys/mount.h>
 static void cvtstat(struct stat30 *, const struct stat *);
 /*
  * Convert from a new to an old stat structure.
  */
 static void
 cvtstat(struct stat30 *ost, const struct stat *st)
+{
 	ost->st_dev = st->st_dev;
 	ost->st_ino = st->st_ino;
 	ost->st_mode = st->st_mode;
 	ost->st_nlink = st->st_nlink;
 	ost->st_uid = st->st_uid;
 	ost->st_gid = st->st_gid;
 	ost->st_rdev = st->st_rdev;
 	timespec_to_timespec50(&st->st_atimespec, &ost->st_atimespec);
 	timespec_to_timespec50(&st->st_mtimespec, &ost->st_mtimespec);
 	timespec_to_timespec50(&st->st_ctimespec, &ost->st_ctimespec);
 	timespec_to_timespec50(&st->st_birthtimespec, &ost->st_birthtimespec);
 	ost->st_size = st->st_size;
 	ost->st_blocks = st->st_blocks;
 	ost->st_blksize = st->st_blksize;
 	ost->st_flags = st->st_flags;
 	ost->st_gen = st->st_gen;
 	memset(ost->st_spare, 0, sizeof(ost->st_spare));
+}
 /*
  * Get file status; this version follows links.
  */
 /* ARGSUSED */
 int
 compat_50_sys___stat30(struct lwp *l, const struct compat_50_sys___stat30_args *uap, register_t *retval)
+{
 	/* {
 		syscallarg(const char *) path;
 		syscallarg(struct stat30 *) ub;
 	} */
 	struct stat sb;
 	struct stat30 osb;
 	int error;
 	error = do_sys_stat(SCARG(uap, path), FOLLOW, &sb);
 	if (error)
 		return error;
 	cvtstat(&osb, &sb);
 	error = copyout(&osb, SCARG(uap, ub), sizeof (osb));
 	return error;
+}
 /*
  * Get file status; this version does not follow links.
  */
 /* ARGSUSED */
 int
 compat_50_sys___lstat30(struct lwp *l, const struct compat_50_sys___lstat30_args *uap, register_t *retval)
+{
 	/* {
 		syscallarg(const char *) path;
 		syscallarg(struct stat30 *) ub;
 	} */
 	struct stat sb;
 	struct stat30 osb;
 	int error;
 	error = do_sys_stat(SCARG(uap, path), NOFOLLOW, &sb);
 	if (error)
 		return error;
 	cvtstat(&osb, &sb);
 	error = copyout(&osb, SCARG(uap, ub), sizeof (osb));
 	return error;
+}
 /*
  * Return status information about a file descriptor.
  */
 /* ARGSUSED */
 int
 compat_50_sys___fstat30(struct lwp *l, const struct compat_50_sys___fstat30_args *uap, register_t *retval)
+{
 	/* {
 		syscallarg(int) fd;
 		syscallarg(struct stat30 *) sb;
 	} */
 	struct stat sb;
 	struct stat30 osb;
 	int error;
 	error = do_sys_fstat(SCARG(uap, fd), &sb);
 	if (error)
 		return error;
 	cvtstat(&osb, &sb);
 	error = copyout(&osb, SCARG(uap, sb), sizeof (osb));
 	return error;
+}
 /* ARGSUSED */
 int
 compat_50_sys___fhstat40(struct lwp *l, const struct compat_50_sys___fhstat40_args *uap, register_t *retval)
+{
 	/* {
 		syscallarg(const void *) fhp;
 		syscallarg(size_t) fh_size;
 		syscallarg(struct stat30 *) sb;
 	} */
 	struct stat sb;
 	struct stat30 osb;
 	int error;
 	error = do_fhstat(l, SCARG(uap, fhp), SCARG(uap, fh_size), &sb);
 	if (error)
 		return error;
 	cvtstat(&osb, &sb);
 	error = copyout(&osb, SCARG(uap, sb), sizeof (osb));
 	return error;
+}
 static int
 compat_50_do_sys_utimes(struct lwp *l, struct vnode *vp, const char *path,
     int flag, const struct timeval50 *tptr)
+{
 	struct timeval tv[2], *tvp;
 	struct timeval50 tv50[2];
 	if (tptr) {
 		int error = copyin(tptr, tv50, sizeof(tv50));
 		if (error)
 			return error;
 		timeval50_to_timeval(&tv50[0], &tv[0]);
 		timeval50_to_timeval(&tv50[1], &tv[1]);
 		tvp = tv;
 	} else
 		tvp = NULL;
 	return do_sys_utimes(l, vp, path, flag, tvp, UIO_SYSSPACE);
+}
 /*
  * Set the access and modification times given a path name; this
  * version follows links.
  */
 /* ARGSUSED */
 int
 compat_50_sys_utimes(struct lwp *l, const struct compat_50_sys_utimes_args *uap,
     register_t *retval)
+{
 	/* {
 		syscallarg(const char *) path;
 		syscallarg(const struct timeval50 *) tptr;
 	} */
 	return compat_50_do_sys_utimes(l, NULL, SCARG(uap, path), FOLLOW,
 	    SCARG(uap, tptr));
+}
 /*
  * Set the access and modification times given a file descriptor.
  */
 /* ARGSUSED */
 int
 compat_50_sys_futimes(struct lwp *l,
     const struct compat_50_sys_futimes_args *uap, register_t *retval)
+{
 	/* {
 		syscallarg(int) fd;
 		syscallarg(const struct timeval50 *) tptr;
 	} */
 	int error;
 	struct file *fp;
 	/* fd_getvnode() will use the descriptor for us */
 	if ((error = fd_getvnode(SCARG(uap, fd), &fp)) != 0)
 		return error;
 	error = compat_50_do_sys_utimes(l, fp->f_data, NULL, 0,
 	    SCARG(uap, tptr));
 	fd_putfile(SCARG(uap, fd));
 	return error;
+}
 /*
  * Set the access and modification times given a path name; this
  * version does not follow links.
  */
 int
 compat_50_sys_lutimes(struct lwp *l,
     const struct compat_50_sys_lutimes_args *uap, register_t *retval)
+{
 	/* {
 		syscallarg(const char *) path;
 		syscallarg(const struct timeval50 *) tptr;
 	} */
 	return compat_50_do_sys_utimes(l, NULL, SCARG(uap, path), NOFOLLOW,
 	    SCARG(uap, tptr));
+}
 int
 compat_50_sys_lfs_segwait(struct lwp *l,
     const struct compat_50_sys_lfs_segwait_args *uap, register_t *retval)
+{
 	/* {
 		syscallarg(fsid_t *) fsidp;
 		syscallarg(struct timeval50 *) tv;
 	} */
 #ifdef notyet
 	struct timeval atv;
 	struct timeval50 atv50;
 	fsid_t fsid;
 	int error;
 	/* XXX need we be su to segwait? */
 	if ((error = kauth_authorize_generic(l->l_cred, KAUTH_GENERIC_ISSUSER,
 	    NULL)) != 0)
 		return (error);
 	if ((error = copyin(SCARG(uap, fsidp), &fsid, sizeof(fsid_t))) != 0)
 		return (error);
 	if (SCARG(uap, tv)) {
 		error = copyin(SCARG(uap, tv), &atv50, sizeof(atv50));
 		if (error)
 			return (error);
 		timeval50_to_timeval(&atv50, &atv);
 		if (itimerfix(&atv))
 			return (EINVAL);
 	} else /* NULL or invalid */
 		atv.tv_sec = atv.tv_usec = 0;
 	return lfs_segwait(&fsid, &atv);
 #else
 	return ENOSYS;
 #endif
+}
 int
 compat_50_sys_mknod(struct lwp *l,
     const struct compat_50_sys_mknod_args *uap, register_t *retval)
+{
 	/* {
 		syscallarg(const char *) path;
 		syscallarg(mode_t) mode;
 		syscallarg(uint32_t) dev;
 	} */
 	return do_sys_mknod(l, SCARG(uap, path), SCARG(uap, mode),
 	    SCARG(uap, dev), retval, UIO_USERSPACE);
+}
 /* ARGSUSED */
 int
 compat_50_sys_quotactl(struct lwp *l, const struct compat_50_sys_quotactl_args *uap, register_t *retval)
+{
 	/* {
 		syscallarg(const char *) path;
 		syscallarg(int) cmd;
 		syscallarg(int) uid;
 		syscallarg(void *) arg;
 	} */
 	struct mount *mp;
 	int error;
 	uint8_t error8;
 	struct vnode *vp;
 	int q1cmd = SCARG(uap, cmd);
 	prop_dictionary_t dict, data, cmd;
 	prop_array_t cmds, datas;
 	char *bufpath;
 	struct dqblk dqblk;
 	struct quotaval qv[QUOTA_NLIMITS];
 	uint64_t *values[QUOTA_NLIMITS];
 	int idtype;
 	values[QUOTA_LIMIT_BLOCK] = &qv[QUOTA_LIMIT_BLOCK].qv_hardlimit;
 	values[QUOTA_LIMIT_FILE] = &qv[QUOTA_LIMIT_FILE].qv_hardlimit;
 	error = namei_simple_user(SCARG(uap, path),
 				NSM_FOLLOW_TRYEMULROOT, &vp);
 	if (error != 0)
 		return (error);
 	error = ENOMEM;
 	mp = vp->v_mount;
 	if ((dict = quota_prop_create()) == NULL)
 		goto out;
 	if ((cmds = prop_array_create()) == NULL)
 		goto out_dict;
 	if ((datas = prop_array_create()) == NULL)
 		goto out_cmds;
 	switch((q1cmd & ~SUBCMDMASK) >> SUBCMDSHIFT) {
 	case Q_QUOTAON:
 		data = prop_dictionary_create();
 		if (data == NULL)
 			goto out_datas;
 		bufpath = malloc(PATH_MAX * sizeof(char), M_TEMP, M_WAITOK);
 		if (bufpath == NULL)
 			goto out_data;
 		error = copyinstr(SCARG(uap, arg), bufpath, PATH_MAX, NULL);
 		if (error != 0) {
 			free(bufpath, M_TEMP);
 			goto out_data;
+		}
 		if (!prop_dictionary_set_cstring(data, "quotafile", bufpath))
 			error = ENOMEM;
 		free(bufpath, M_TEMP);
 		if (error)
 			goto out_data;
 		error = ENOMEM;
 		if (!prop_array_add_and_rel(datas, data))
 			goto out_datas;
 		idtype = quota_idtype_from_ufs(q1cmd & SUBCMDMASK);
 		if (!quota_prop_add_command(cmds, "quotaon",
-		    ufs_quota_class_names[qtype2ufsclass(q1cmd & SUBCMDMASK)],
+		    ufs_quota_class_names[idtype],
 		    datas))
 			goto out_cmds;
 		goto do_quotaonoff;
 	case Q_QUOTAOFF:
 		error = ENOMEM;
 		idtype = quota_idtype_from_ufs(q1cmd & SUBCMDMASK);
 		if (!quota_prop_add_command(cmds, "quotaoff",
-		    ufs_quota_class_names[qtype2ufsclass(q1cmd & SUBCMDMASK)],
+		    ufs_quota_class_names[idtype],
 		    datas))
 			goto out_cmds;
 do_quotaonoff:
 		if (!prop_dictionary_set_and_rel(dict, "commands", cmds))
 			goto out_dict;
 		error = vfs_quotactl(mp, dict);
 		if (error)
 			goto out_dict;
 		if ((error = quota_get_cmds(dict, &cmds)) != 0)
 			goto out_dict;
 		cmd = prop_array_get(cmds, 0);
 		if (cmd == NULL) {
 			error = EINVAL;
 			goto out_dict;
+		}
 		if (!prop_dictionary_get_int8(cmd, "return", &error8)) {
 			error = EINVAL;
 			goto out_dict;
+		}
 		error = error8;
 		goto out_dict;
 	case Q_GETQUOTA:
 		error = ENOMEM;
 		data = prop_dictionary_create();
 		if (data == NULL)
 			goto out_datas;
 		if (!prop_dictionary_set_uint32(data, "id", SCARG(uap, uid)))
 			goto out_data;
 		if (!prop_array_add_and_rel(datas, data))
 			goto out_datas;
 		idtype = quota_idtype_from_ufs(q1cmd & SUBCMDMASK);
 		if (!quota_prop_add_command(cmds, "get",
-		    ufs_quota_class_names[qtype2ufsclass(q1cmd & SUBCMDMASK)],
+		    ufs_quota_class_names[idtype],
 		    datas))
 			goto out_cmds;
 		if (!prop_dictionary_set_and_rel(dict, "commands", cmds))
 			goto out_dict;
 		error = vfs_quotactl(mp, dict);
 		if (error)
 			goto out_dict;
 		if ((error = quota_get_cmds(dict, &cmds)) != 0)
 			goto out_dict;
 		cmd = prop_array_get(cmds, 0);
 		if (cmd == NULL) {
 			error = EINVAL;
 			goto out_dict;
+		}
 		if (!prop_dictionary_get_int8(cmd, "return", &error8)) {
 			error = EINVAL;
 			goto out_dict;
+		}
 		error = error8;
 		if (error)
 			goto out_dict;
 		datas = prop_dictionary_get(cmd, "data");
 		error = EINVAL;
 		if (datas == NULL)
 			goto out_dict;
 		data = prop_array_get(datas, 0);
 		if (data == NULL)
 			goto out_dict;
 		error = proptoquota64(data, values,
 		    ufs_quota_entry_names, UFS_QUOTA_NENTRIES,
 		    ufs_quota_limit_names, QUOTA_NLIMITS);
 		if (error)
 			goto out_dict;
 		quotavals_to_dqblk(&qv[QUOTA_LIMIT_BLOCK],
 				   &qv[QUOTA_LIMIT_FILE],
 				   &dqblk);
 		error = copyout(&dqblk, SCARG(uap, arg), sizeof(dqblk));
 		goto out_dict;
 	case Q_SETQUOTA:
 		error = copyin(SCARG(uap, arg), &dqblk, sizeof(dqblk));
 		if (error)
 			goto out_datas;
 		dqblk_to_quotavals(&dqblk, &qv[QUOTA_LIMIT_BLOCK],
 				   &qv[QUOTA_LIMIT_FILE]);
 		error = ENOMEM;
 		data = quota64toprop(SCARG(uap, uid), 0, values,
 		    ufs_quota_entry_names, UFS_QUOTA_NENTRIES,
 		    ufs_quota_limit_names, QUOTA_NLIMITS);
 		if (data == NULL)
 			goto out_data;
 		if (!prop_array_add_and_rel(datas, data))
 			goto out_datas;
 		idtype = quota_idtype_from_ufs(q1cmd & SUBCMDMASK);
 		if (!quota_prop_add_command(cmds, "set",
-		    ufs_quota_class_names[qtype2ufsclass(q1cmd & SUBCMDMASK)],
+		    ufs_quota_class_names[idtype],
 		    datas))
 			goto out_cmds;
 		if (!prop_dictionary_set_and_rel(dict, "commands", cmds))
 			goto out_dict;
 		error = vfs_quotactl(mp, dict);
 		if (error)
 			goto out_dict;
 		if ((error = quota_get_cmds(dict, &cmds)) != 0)
 			goto out_dict;
 		cmd = prop_array_get(cmds, 0);
 		if (cmd == NULL) {
 			error = EINVAL;
 			goto out_dict;
+		}
 		if (!prop_dictionary_get_int8(cmd, "return", &error8)) {
 			error = EINVAL;
 			goto out_dict;
+		}
 		error = error8;
 		goto out_dict;
 	case Q_SYNC:
 		/*
 		 * not supported but used only to see if quota is supported,
 		 * emulate with a "get version"
 		 */
 		error = ENOMEM;
 		idtype = quota_idtype_from_ufs(q1cmd & SUBCMDMASK);
 		if (!quota_prop_add_command(cmds, "get version",
-		    ufs_quota_class_names[qtype2ufsclass(q1cmd & SUBCMDMASK)],
+		    ufs_quota_class_names[idtype],
 		    datas))
 			goto out_cmds;
 		if (!prop_dictionary_set_and_rel(dict, "commands", cmds))
 			goto out_dict;
 		error = vfs_quotactl(mp, dict);
 		if (error)
 			goto out_dict;
 		if ((error = quota_get_cmds(dict, &cmds)) != 0)
 			goto out_dict;
 		cmd = prop_array_get(cmds, 0);
 		if (cmd == NULL) {
 			error = EINVAL;
 			goto out_dict;
+		}
 		if (!prop_dictionary_get_int8(cmd, "return", &error8)) {
 			error = EINVAL;
 			goto out_dict;
+		}
 		error = error8;
 		goto out_dict;
 	case Q_SETUSE:
 	default:
 		error = EOPNOTSUPP;
 		goto out_datas;
+	}
 out_data:
 	prop_object_release(data);
 out_datas:
 	prop_object_release(datas);
 out_cmds:
 	prop_object_release(cmds);
 out_dict:
 	prop_object_release(dict);
 out:
 	vrele(vp);
 	return error;
+}

 @@ -1,1217 +1,1217 @@
-/*      $NetBSD: edquota.c,v 1.42 2012/01/29 06:23:20 dholland Exp $ */
+/*      $NetBSD: edquota.c,v 1.43 2012/01/29 07:16:00 dholland Exp $ */
 /*
  * Copyright (c) 1980, 1990, 1993
  *	The Regents of the University of California.  All rights reserved.
+ *
  * This code is derived from software contributed to Berkeley by
  * Robert Elz at The University of Melbourne.
+ *
  * 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.
  * 3. Neither the name of the University nor the names of its contributors
  *    may be used to endorse or promote products derived from this software
  *    without specific prior written permission.
+ *
  * THIS SOFTWARE IS PROVIDED BY THE REGENTS 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 THE REGENTS 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.
  */
 #include <sys/cdefs.h>
 #ifndef lint
 __COPYRIGHT("@(#) Copyright (c) 1980, 1990, 1993\
  The Regents of the University of California.  All rights reserved.");
 #endif /* not lint */
 #ifndef lint
 #if 0
 static char sccsid[] = "from: @(#)edquota.c	8.3 (Berkeley) 4/27/95";
 #else
-__RCSID("$NetBSD: edquota.c,v 1.42 2012/01/29 06:23:20 dholland Exp $");
+__RCSID("$NetBSD: edquota.c,v 1.43 2012/01/29 07:16:00 dholland Exp $");
 #endif
 #endif /* not lint */
 /*
  * Disk quota editor.
  */
 #include <sys/param.h>
 #include <sys/stat.h>
 #include <sys/file.h>
 #include <sys/wait.h>
 #include <sys/queue.h>
 #include <sys/types.h>
 #include <sys/statvfs.h>
 #include <quota.h>
 #include <quota/quotaprop.h>
 #include <quota/quota.h>
 #include <ufs/ufs/quota1.h>
 #include <sys/quota.h>
 #include <assert.h>
 #include <err.h>
 #include <errno.h>
 #include <fstab.h>
 #include <pwd.h>
 #include <grp.h>
 #include <ctype.h>
 #include <signal.h>
 #include <stdbool.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <unistd.h>
 #include "printquota.h"
 #include "getvfsquota.h"
 #include "quotautil.h"
 #include "pathnames.h"
 static const char *quotagroup = QUOTAGROUP;
 #define MAX_TMPSTR	(100+MAXPATHLEN)
 /* flags for quotause */
 #define	FOUND	0x01
 #define	QUOTA2	0x02
 #define	DEFAULT	0x04
 struct quotause {
 	struct	quotause *next;
 	long	flags;
 	struct	quotaval qv[QUOTA_NLIMITS];
 	char	fsname[MAXPATHLEN + 1];
 	char	*qfname;
 };
 struct quotalist {
 	struct quotause *head;
 	struct quotause *tail;
 };
 static void	usage(void) __dead;
 static int Hflag = 0;
 static int Dflag = 0;
 /* more compact form of constants */
 #define QL_BLK QUOTA_LIMIT_BLOCK
 #define QL_FL  QUOTA_LIMIT_FILE
 ////////////////////////////////////////////////////////////
 // support code
 /*
  * This routine converts a name for a particular quota class to
  * an identifier. This routine must agree with the kernel routine
  * getinoquota as to the interpretation of quota classes.
  */
 static int
 getidbyname(const char *name, int idtype)
+{
 	struct passwd *pw;
 	struct group *gr;
 	if (alldigits(name))
 		return atoi(name);
 	switch (idtype) {
 	case QUOTA_IDTYPE_USER:
 		if ((pw = getpwnam(name)) != NULL)
 			return pw->pw_uid;
 		warnx("%s: no such user", name);
 		break;
 	case QUOTA_IDTYPE_GROUP:
 		if ((gr = getgrnam(name)) != NULL)
 			return gr->gr_gid;
 		warnx("%s: no such group", name);
 		break;
 	default:
 		warnx("%d: unknown quota type", idtype);
 		break;
+	}
 	sleep(1);
 	return -1;
+}
 ////////////////////////////////////////////////////////////
 // quotause operations
 /*
  * Create an empty quotause structure.
  */
 static struct quotause *
 quotause_create(void)
+{
 	struct quotause *qup;
 	qup = malloc(sizeof(*qup));
 	if (qup == NULL) {
 		err(1, "malloc");
+	}
 	qup->next = NULL;
 	qup->flags = 0;
 	memset(qup->qv, 0, sizeof(qup->qv));
 	qup->fsname[0] = '\0';
 	qup->qfname = NULL;
 	return qup;
+}
 /*
  * Free a quotause structure.
  */
 static void
 quotause_destroy(struct quotause *qup)
+{
 	free(qup->qfname);
 	free(qup);
+}
 ////////////////////////////////////////////////////////////
 // quotalist operations
 /*
  * Create a quotause list.
  */
 static struct quotalist *
 quotalist_create(void)
+{
 	struct quotalist *qlist;
 	qlist = malloc(sizeof(*qlist));
 	if (qlist == NULL) {
 		err(1, "malloc");
+	}
 	qlist->head = NULL;
 	qlist->tail = NULL;
 	return qlist;
+}
 /*
  * Free a list of quotause structures.
  */
 static void
 quotalist_destroy(struct quotalist *qlist)
+{
 	struct quotause *qup, *nextqup;
 	for (qup = qlist->head; qup; qup = nextqup) {
 		nextqup = qup->next;
 		quotause_destroy(qup);
+	}
 	free(qlist);
+}
 static bool
 quotalist_empty(struct quotalist *qlist)
+{
 	return qlist->head == NULL;
+}
 static void
 quotalist_append(struct quotalist *qlist, struct quotause *qup)
+{
 	/* should not already be on a list */
 	assert(qup->next == NULL);
 	if (qlist->head == NULL) {
 		qlist->head = qup;
 	} else {
 		qlist->tail->next = qup;
+	}
 	qlist->tail = qup;
+}
 ////////////////////////////////////////////////////////////
 // ffs quota v1
 static void
 putprivs1(uint32_t id, int idtype, struct quotause *qup)
+{
 	struct dqblk dqblk;
 	int fd;
 	quotavals_to_dqblk(&qup->qv[QUOTA_LIMIT_BLOCK],
 			   &qup->qv[QUOTA_LIMIT_FILE],
 			   &dqblk);
 	assert((qup->flags & DEFAULT) == 0);
 	if ((fd = open(qup->qfname, O_WRONLY)) < 0) {
 		warnx("open `%s'", qup->qfname);
 	} else {
 		(void)lseek(fd,
 		    (off_t)(id * (long)sizeof (struct dqblk)),
 		    SEEK_SET);
 		if (write(fd, &dqblk, sizeof (struct dqblk)) !=
 		    sizeof (struct dqblk))
 			warnx("writing `%s'", qup->qfname);
 		close(fd);
+	}
+}
 static struct quotause *
 getprivs1(long id, int idtype, const char *filesys)
+{
 	struct fstab *fs;
 	char qfpathname[MAXPATHLEN];
 	struct quotause *qup;
 	struct dqblk dqblk;
 	int fd;
 	setfsent();
 	while ((fs = getfsent()) != NULL) {
 		if (strcmp(fs->fs_vfstype, "ffs"))
 			continue;
 		if (strcmp(fs->fs_spec, filesys) == 0 ||
 		    strcmp(fs->fs_file, filesys) == 0)
 			break;
+	}
 	if (fs == NULL)
 		return NULL;
 	if (!hasquota(qfpathname, sizeof(qfpathname), fs,
-	    ufsclass2qtype(idtype)))
+	    quota_idtype_to_ufs(idtype)))
 		return NULL;
 	qup = quotause_create();
 	strcpy(qup->fsname, fs->fs_file);
 	if ((fd = open(qfpathname, O_RDONLY)) < 0) {
 		fd = open(qfpathname, O_RDWR|O_CREAT, 0640);
 		if (fd < 0 && errno != ENOENT) {
 			warnx("open `%s'", qfpathname);
 			quotause_destroy(qup);
 			return NULL;
+		}
 		warnx("Creating quota file %s", qfpathname);
 		sleep(3);
 		(void)fchown(fd, getuid(),
 		    getidbyname(quotagroup, QUOTA_CLASS_GROUP));
 		(void)fchmod(fd, 0640);
+	}
 	(void)lseek(fd, (off_t)(id * sizeof(struct dqblk)),
 	    SEEK_SET);
 	switch (read(fd, &dqblk, sizeof(struct dqblk))) {
 	case 0:			/* EOF */
 		/*
 		 * Convert implicit 0 quota (EOF)
 		 * into an explicit one (zero'ed dqblk)
 		 */
 		memset(&dqblk, 0, sizeof(struct dqblk));
 		break;
 	case sizeof(struct dqblk):	/* OK */
 		break;
 	default:		/* ERROR */
 		warn("read error in `%s'", qfpathname);
 		close(fd);
 		quotause_destroy(qup);
 		return NULL;
+	}
 	close(fd);
 	qup->qfname = qfpathname;
 	endfsent();
 	dqblk_to_quotavals(&dqblk,
 			   &qup->qv[QUOTA_LIMIT_BLOCK],
 			   &qup->qv[QUOTA_LIMIT_FILE]);
 	return qup;
+}
 ////////////////////////////////////////////////////////////
 // ffs quota v2
 static struct quotause *
 getprivs2(long id, int idtype, const char *filesys, int defaultq)
+{
 	struct quotause *qup;
 	int8_t version;
 	qup = quotause_create();
 	strcpy(qup->fsname, filesys);
 	if (defaultq)
 		qup->flags |= DEFAULT;
 	if (!getvfsquota(filesys, qup->qv, &version,
 	    id, idtype, defaultq, Dflag)) {
 		/* no entry, get default entry */
 		if (!getvfsquota(filesys, qup->qv, &version,
 		    id, idtype, 1, Dflag)) {
 			free(qup);
 			return NULL;
+		}
+	}
 	if (version == 2)
 		qup->flags |= QUOTA2;
 	return qup;
+}
 static void
 putprivs2(uint32_t id, int idtype, struct quotause *qup)
+{
 	struct quotahandle *qh;
 	struct quotakey qk;
 	char idname[32];
 	if (qup->flags & DEFAULT) {
 		snprintf(idname, sizeof(idname), "%s default",
 			 idtype == QUOTA_IDTYPE_USER ? "user" : "group");
 		id = QUOTA_DEFAULTID;
 	} else {
 		snprintf(idname, sizeof(idname), "%s %u",
 			 idtype == QUOTA_IDTYPE_USER ? "uid" : "gid", id);
+	}
 	qh = quota_open(qup->fsname);
 	if (qh == NULL) {
 		err(1, "%s: quota_open", qup->fsname);
+	}
 	qk.qk_idtype = idtype;
 	qk.qk_id = id;
 	qk.qk_objtype = QUOTA_OBJTYPE_BLOCKS;
 	if (quota_put(qh, &qk, &qup->qv[QL_BLK])) {
 		err(1, "%s: quota_put (%s blocks)", qup->fsname, idname);
+	}
 	qk.qk_idtype = idtype;
 	qk.qk_id = id;
 	qk.qk_objtype = QUOTA_OBJTYPE_FILES;
 	if (quota_put(qh, &qk, &qup->qv[QL_FL])) {
 		err(1, "%s: quota_put (%s files)", qup->fsname, idname);
+	}
 	quota_close(qh);
+}
 ////////////////////////////////////////////////////////////
 // quota format switch
 /*
  * Collect the requested quota information.
  */
 static struct quotalist *
 getprivs(long id, int defaultq, int idtype, const char *filesys)
+{
 	struct statvfs *fst;
 	int nfst, i;
 	struct quotalist *qlist;
 	struct quotause *qup;
 	qlist = quotalist_create();
 	nfst = getmntinfo(&fst, MNT_WAIT);
 	if (nfst == 0)
 		errx(1, "no filesystems mounted!");
 	for (i = 0; i < nfst; i++) {
 		if ((fst[i].f_flag & ST_QUOTA) == 0)
 			continue;
 		if (filesys &&
 		    strcmp(fst[i].f_mntonname, filesys) != 0 &&
 		    strcmp(fst[i].f_mntfromname, filesys) != 0)
 			continue;
 		qup = getprivs2(id, idtype, fst[i].f_mntonname, defaultq);
 		if (qup == NULL) {
 			/*
 			 * XXX: returning NULL is totally wrong. On
 			 * serious error, abort; on minor error, warn
 			 * and continue.
+			 *
 			 * Note: we cannot warn unconditionally here
 			 * because this case apparently includes "no
 			 * quota entry on this volume" and that causes
 			 * the atf tests to fail. Bletch.
 			 */
 			/*return NULL;*/
 			/*warnx("getprivs2 failed");*/
 			continue;
+		}
 		quotalist_append(qlist, qup);
+	}
 	if (filesys && quotalist_empty(qlist)) {
 		if (defaultq)
 			errx(1, "no default quota for version 1");
 		/* if we get there, filesys is not mounted. try the old way */
 		qup = getprivs1(id, idtype, filesys);
 		if (qup == NULL) {
 			/* XXX. see above */
 			/*return NULL;*/
 			/*warnx("getprivs1 failed");*/
 			return qlist;
+		}
 		quotalist_append(qlist, qup);
+	}
 	return qlist;
+}
 /*
  * Store the requested quota information.
  */
 static void
 putprivs(uint32_t id, int idtype, struct quotalist *qlist)
+{
 	struct quotause *qup;
         for (qup = qlist->head; qup; qup = qup->next) {
 		if (qup->qfname == NULL)
 			putprivs2(id, idtype, qup);
 		else
 			putprivs1(id, idtype, qup);
+	}
+}
 static void
 clearpriv(int argc, char **argv, const char *filesys, int idtype)
+{
 	struct statvfs *fst;
 	int nfst, i;
 	int id;
 	id_t *ids;
 	unsigned nids, maxids, j;
 	struct quotahandle *qh;
 	struct quotakey qk;
 	char idname[32];
 	maxids = 4;
 	nids = 0;
 	ids = malloc(maxids * sizeof(ids[0]));
 	if (ids == NULL) {
 		err(1, "malloc");
+	}
 	for ( ; argc > 0; argc--, argv++) {
 		if ((id = getidbyname(*argv, idtype)) == -1)
 			continue;
 		if (nids + 1 > maxids) {
 			maxids *= 2;
 			ids = realloc(ids, maxids * sizeof(ids[0]));
 			if (ids == NULL) {
 				err(1, "realloc");
+			}
+		}
 		ids[nids++] = id;
+	}
 	/* now loop over quota-enabled filesystems */
 	nfst = getmntinfo(&fst, MNT_WAIT);
 	if (nfst == 0)
 		errx(1, "no filesystems mounted!");
 	for (i = 0; i < nfst; i++) {
 		if ((fst[i].f_flag & ST_QUOTA) == 0)
 			continue;
 		if (filesys && strcmp(fst[i].f_mntonname, filesys) != 0 &&
 		    strcmp(fst[i].f_mntfromname, filesys) != 0)
 			continue;
 		qh = quota_open(fst[i].f_mntonname);
 		if (qh == NULL) {
 			err(1, "%s: quota_open", fst[i].f_mntonname);
+		}
 		for (j = 0; j < nids; j++) {
 			snprintf(idname, sizeof(idname), "%s %u",
 				 idtype == QUOTA_IDTYPE_USER ?
 				 "uid" : "gid", ids[j]);
 			qk.qk_idtype = idtype;
 			qk.qk_id = ids[j];
 			qk.qk_objtype = QUOTA_OBJTYPE_BLOCKS;
 			if (quota_delete(qh, &qk)) {
 				err(1, "%s: quota_delete (%s blocks)",
 				    fst[i].f_mntonname, idname);
+			}
 			qk.qk_idtype = idtype;
 			qk.qk_id = ids[j];
 			qk.qk_objtype = QUOTA_OBJTYPE_FILES;
 			if (quota_delete(qh, &qk)) {
 				if (errno == ENOENT) {
  					/*
 					 * XXX ignore this case; due
 					 * to a weakness in the quota
 					 * proplib interface it can
 					 * appear spuriously.
 					 */
 				} else {
 					err(1, "%s: quota_delete (%s files)",
 					    fst[i].f_mntonname, idname);
+				}
+			}
+		}
 		quota_close(qh);
+	}
 	free(ids);
+}
 ////////////////////////////////////////////////////////////
 // editor
 /*
  * Take a list of privileges and get it edited.
  */
 static int
 editit(const char *ltmpfile)
+{
 	pid_t pid;
 	int lst;
 	char p[MAX_TMPSTR];
 	const char *ed;
 	sigset_t s, os;
 	sigemptyset(&s);
 	sigaddset(&s, SIGINT);
 	sigaddset(&s, SIGQUIT);
 	sigaddset(&s, SIGHUP);
 	if (sigprocmask(SIG_BLOCK, &s, &os) == -1)
 		err(1, "sigprocmask");
 top:
 	switch ((pid = fork())) {
 	case -1:
 		if (errno == EPROCLIM) {
 			warnx("You have too many processes");
 			return 0;
+		}
 		if (errno == EAGAIN) {
 			sleep(1);
 			goto top;
+		}
 		warn("fork");
 		return 0;
 	case 0:
 		if (sigprocmask(SIG_SETMASK, &os, NULL) == -1)
 			err(1, "sigprocmask");
 		setgid(getgid());
 		setuid(getuid());
 		if ((ed = getenv("EDITOR")) == (char *)0)
 			ed = _PATH_VI;
 		if (strlen(ed) + strlen(ltmpfile) + 2 >= MAX_TMPSTR) {
 			errx(1, "%s", "editor or filename too long");
+		}
 		snprintf(p, sizeof(p), "%s %s", ed, ltmpfile);
 		execlp(_PATH_BSHELL, _PATH_BSHELL, "-c", p, NULL);
 		err(1, "%s", ed);
 	default:
 		if (waitpid(pid, &lst, 0) == -1)
 			err(1, "waitpid");
 		if (sigprocmask(SIG_SETMASK, &os, NULL) == -1)
 			err(1, "sigprocmask");
 		if (!WIFEXITED(lst) || WEXITSTATUS(lst) != 0)
 			return 0;
 		return 1;
+	}
+}
 /*
  * Convert a quotause list to an ASCII file.
  */
 static int
 writeprivs(struct quotalist *qlist, int outfd, const char *name,
     int idtype)
+{
 	struct quotause *qup;
 	FILE *fd;
 	char b0[32], b1[32], b2[32], b3[32];
 	(void)ftruncate(outfd, 0);
 	(void)lseek(outfd, (off_t)0, SEEK_SET);
 	if ((fd = fdopen(dup(outfd), "w")) == NULL)
 		errx(1, "fdopen");
 	if (name == NULL) {
 		fprintf(fd, "Default %s quotas:\n",
 		    ufs_quota_class_names[idtype]);
 	} else {
 		fprintf(fd, "Quotas for %s %s:\n",
 		    ufs_quota_class_names[idtype], name);
+	}
 	for (qup = qlist->head; qup; qup = qup->next) {
 		struct quotaval *q = qup->qv;
 		fprintf(fd, "%s (version %d):\n",
 		     qup->fsname, (qup->flags & QUOTA2) ? 2 : 1);
 		if ((qup->flags & DEFAULT) == 0 || (qup->flags & QUOTA2) != 0) {
 			fprintf(fd, "\tblocks in use: %s, "
 			    "limits (soft = %s, hard = %s",
 			    intprt(b1, 21, q[QL_BLK].qv_usage,
 			    HN_NOSPACE | HN_B, Hflag),
 			    intprt(b2, 21, q[QL_BLK].qv_softlimit,
 			    HN_NOSPACE | HN_B, Hflag),
 			    intprt(b3, 21, q[QL_BLK].qv_hardlimit,
 				HN_NOSPACE | HN_B, Hflag));
 			if (qup->flags & QUOTA2)
 				fprintf(fd, ", ");
 		} else
 			fprintf(fd, "\tblocks: (");
 		if (qup->flags & (QUOTA2|DEFAULT)) {
 		    fprintf(fd, "grace = %s",
 			timepprt(b0, 21, q[QL_BLK].qv_grace, Hflag));
+		}
 		fprintf(fd, ")\n");
 		if ((qup->flags & DEFAULT) == 0 || (qup->flags & QUOTA2) != 0) {
 			fprintf(fd, "\tinodes in use: %s, "
 			    "limits (soft = %s, hard = %s",
 			    intprt(b1, 21, q[QL_FL].qv_usage,
 			    HN_NOSPACE, Hflag),
 			    intprt(b2, 21, q[QL_FL].qv_softlimit,
 			    HN_NOSPACE, Hflag),
 			    intprt(b3, 21, q[QL_FL].qv_hardlimit,
 			     HN_NOSPACE, Hflag));
 			if (qup->flags & QUOTA2)
 				fprintf(fd, ", ");
 		} else
 			fprintf(fd, "\tinodes: (");
 		if (qup->flags & (QUOTA2|DEFAULT)) {
 		    fprintf(fd, "grace = %s",
 			timepprt(b0, 21, q[QL_FL].qv_grace, Hflag));
+		}
 		fprintf(fd, ")\n");
+	}
 	fclose(fd);
 	return 1;
+}
 /*
  * Merge changes to an ASCII file into a quotause list.
  */
 static int
 readprivs(struct quotalist *qlist, int infd, int dflag)
+{
 	struct quotause *qup;
 	FILE *fd;
 	int cnt;
 	char fsp[BUFSIZ];
 	static char line0[BUFSIZ], line1[BUFSIZ], line2[BUFSIZ];
 	static char scurb[BUFSIZ], scuri[BUFSIZ], ssoft[BUFSIZ], shard[BUFSIZ];
 	static char stime[BUFSIZ];
 	uint64_t softb, hardb, softi, hardi;
 	time_t graceb = -1, gracei = -1;
 	int version;
 	(void)lseek(infd, (off_t)0, SEEK_SET);
 	fd = fdopen(dup(infd), "r");
 	if (fd == NULL) {
 		warn("Can't re-read temp file");
 		return 0;
+	}
 	/*
 	 * Discard title line, then read pairs of lines to process.
 	 */
 	(void) fgets(line1, sizeof (line1), fd);
 	while (fgets(line0, sizeof (line0), fd) != NULL &&
 	       fgets(line1, sizeof (line2), fd) != NULL &&
 	       fgets(line2, sizeof (line2), fd) != NULL) {
 		if (sscanf(line0, "%s (version %d):\n", fsp, &version) != 2) {
 			warnx("%s: bad format", line0);
 			goto out;
+		}
 #define last_char(str) ((str)[strlen(str) - 1])
 		if (last_char(line1) != '\n') {
 			warnx("%s:%s: bad format", fsp, line1);
 			goto out;
+		}
 		last_char(line1) = '\0';
 		if (last_char(line2) != '\n') {
 			warnx("%s:%s: bad format", fsp, line2);
 			goto out;
+		}
 		last_char(line2) = '\0';
 		if (dflag && version == 1) {
 			if (sscanf(line1,
 			    "\tblocks: (grace = %s\n", stime) != 1) {
 				warnx("%s:%s: bad format", fsp, line1);
 				goto out;
+			}
 			if (last_char(stime) != ')') {
 				warnx("%s:%s: bad format", fsp, line1);
 				goto out;
+			}
 			last_char(stime) = '\0';
 			if (timeprd(stime, &graceb) != 0) {
 				warnx("%s:%s: bad number", fsp, stime);
 				goto out;
+			}
 			if (sscanf(line2,
 			    "\tinodes: (grace = %s\n", stime) != 1) {
 				warnx("%s:%s: bad format", fsp, line2);
 				goto out;
+			}
 			if (last_char(stime) != ')') {
 				warnx("%s:%s: bad format", fsp, line2);
 				goto out;
+			}
 			last_char(stime) = '\0';
 			if (timeprd(stime, &gracei) != 0) {
 				warnx("%s:%s: bad number", fsp, stime);
 				goto out;
+			}
 		} else {
 			cnt = sscanf(line1,
 			    "\tblocks in use: %s limits (soft = %s hard = %s "
 			    "grace = %s", scurb, ssoft, shard, stime);
 			if (cnt == 3) {
 				if (version != 1 ||
 				    last_char(scurb) != ',' ||
 				    last_char(ssoft) != ',' ||
 				    last_char(shard) != ')') {
 					warnx("%s:%s: bad format %d",
 					    fsp, line1, cnt);
 					goto out;
+				}
 				stime[0] = '\0';
 			} else if (cnt == 4) {
 				if (version < 2 ||
 				    last_char(scurb) != ',' ||
 				    last_char(ssoft) != ',' ||
 				    last_char(shard) != ',' ||
 				    last_char(stime) != ')') {
 					warnx("%s:%s: bad format %d",
 					    fsp, line1, cnt);
 					goto out;
+				}
 			} else {
 				warnx("%s: %s: bad format cnt %d", fsp, line1,
 				    cnt);
 				goto out;
+			}
 			/* drop last char which is ',' or ')' */
 			last_char(scurb) = '\0';
 			last_char(ssoft) = '\0';
 			last_char(shard) = '\0';
 			last_char(stime) = '\0';
 			if (intrd(ssoft, &softb, HN_B) != 0) {
 				warnx("%s:%s: bad number", fsp, ssoft);
 				goto out;
+			}
 			if (intrd(shard, &hardb, HN_B) != 0) {
 				warnx("%s:%s: bad number", fsp, shard);
 				goto out;
+			}
 			if (cnt == 4) {
 				if (timeprd(stime, &graceb) != 0) {
 					warnx("%s:%s: bad number", fsp, stime);
 					goto out;
+				}
+			}
 			cnt = sscanf(line2,
 			    "\tinodes in use: %s limits (soft = %s hard = %s "
 			    "grace = %s", scuri, ssoft, shard, stime);
 			if (cnt == 3) {
 				if (version != 1 ||
 				    last_char(scuri) != ',' ||
 				    last_char(ssoft) != ',' ||
 				    last_char(shard) != ')') {
 					warnx("%s:%s: bad format %d",
 					    fsp, line2, cnt);
 					goto out;
+				}
 				stime[0] = '\0';
 			} else if (cnt == 4) {
 				if (version < 2 ||
 				    last_char(scuri) != ',' ||
 				    last_char(ssoft) != ',' ||
 				    last_char(shard) != ',' ||
 				    last_char(stime) != ')') {
 					warnx("%s:%s: bad format %d",
 					    fsp, line2, cnt);
 					goto out;
+				}
 			} else {
 				warnx("%s: %s: bad format", fsp, line2);
 				goto out;
+			}
 			/* drop last char which is ',' or ')' */
 			last_char(scuri) = '\0';
 			last_char(ssoft) = '\0';
 			last_char(shard) = '\0';
 			last_char(stime) = '\0';
 			if (intrd(ssoft, &softi, 0) != 0) {
 				warnx("%s:%s: bad number", fsp, ssoft);
 				goto out;
+			}
 			if (intrd(shard, &hardi, 0) != 0) {
 				warnx("%s:%s: bad number", fsp, shard);
 				goto out;
+			}
 			if (cnt == 4) {
 				if (timeprd(stime, &gracei) != 0) {
 					warnx("%s:%s: bad number", fsp, stime);
 					goto out;
+				}
+			}
+		}
 		for (qup = qlist->head; qup; qup = qup->next) {
 			struct quotaval *q = qup->qv;
 			char b1[32], b2[32];
 			if (strcmp(fsp, qup->fsname))
 				continue;
 			if (version == 1 && dflag) {
 				q[QL_BLK].qv_grace = graceb;
 				q[QL_FL].qv_grace = gracei;
 				qup->flags |= FOUND;
 				continue;
+			}
 			if (strcmp(intprt(b1, 21, q[QL_BLK].qv_usage,
 			    HN_NOSPACE | HN_B, Hflag),
 			    scurb) != 0 ||
 			    strcmp(intprt(b2, 21, q[QL_FL].qv_usage,
 			    HN_NOSPACE, Hflag),
 			    scuri) != 0) {
 				warnx("%s: cannot change current allocation",
 				    fsp);
 				break;
+			}
 			/*
 			 * Cause time limit to be reset when the quota
 			 * is next used if previously had no soft limit
 			 * or were under it, but now have a soft limit
 			 * and are over it.
 			 */
 			if (q[QL_BLK].qv_usage &&
 			    q[QL_BLK].qv_usage >= softb &&
 			    (q[QL_BLK].qv_softlimit == 0 ||
 			     q[QL_BLK].qv_usage < q[QL_BLK].qv_softlimit))
 				q[QL_BLK].qv_expiretime = 0;
 			if (q[QL_FL].qv_usage &&
 			    q[QL_FL].qv_usage >= softi &&
 			    (q[QL_FL].qv_softlimit == 0 ||
 			     q[QL_FL].qv_usage < q[QL_FL].qv_softlimit))
 				q[QL_FL].qv_expiretime = 0;
 			q[QL_BLK].qv_softlimit = softb;
 			q[QL_BLK].qv_hardlimit = hardb;
 			if (version == 2)
 				q[QL_BLK].qv_grace = graceb;
 			q[QL_FL].qv_softlimit  = softi;
 			q[QL_FL].qv_hardlimit  = hardi;
 			if (version == 2)
 				q[QL_FL].qv_grace = gracei;
 			qup->flags |= FOUND;
+		}
+	}
 out:
 	fclose(fd);
 	/*
 	 * Disable quotas for any filesystems that have not been found.
 	 */
 	for (qup = qlist->head; qup; qup = qup->next) {
 		struct quotaval *q = qup->qv;
 		if (qup->flags & FOUND) {
 			qup->flags &= ~FOUND;
 			continue;
+		}
 		q[QL_BLK].qv_softlimit = UQUAD_MAX;
 		q[QL_BLK].qv_hardlimit = UQUAD_MAX;
 		q[QL_BLK].qv_grace = 0;
 		q[QL_FL].qv_softlimit = UQUAD_MAX;
 		q[QL_FL].qv_hardlimit = UQUAD_MAX;
 		q[QL_FL].qv_grace = 0;
+	}
 	return 1;
+}
 ////////////////////////////////////////////////////////////
 // actions
 static void
 replicate(const char *fs, int idtype, const char *protoname,
 	  char **names, int numnames)
+{
 	long protoid, id;
 	struct quotalist *protoprivs;
 	struct quotause *qup;
 	int i;
 	if ((protoid = getidbyname(protoname, idtype)) == -1)
 		exit(1);
 	protoprivs = getprivs(protoid, 0, idtype, fs);
 	for (qup = protoprivs->head; qup; qup = qup->next) {
 		qup->qv[QL_BLK].qv_expiretime = 0;
 		qup->qv[QL_FL].qv_expiretime = 0;
+	}
 	for (i=0; i<numnames; i++) {
 		id = getidbyname(names[i], idtype);
 		if (id == -1)
 			continue;
 		putprivs(id, idtype, protoprivs);
+	}
 	/* XXX */
 	/* quotalist_destroy(protoprivs); */
+}
 static void
 assign(const char *fs, int idtype,
        char *soft, char *hard, char *grace,
        char **names, int numnames)
+{
 	struct quotalist *curprivs;
 	struct quotause *lqup;
 	u_int64_t softb, hardb, softi, hardi;
 	time_t  graceb, gracei;
 	char *str;
 	long id;
 	int dflag;
 	int i;
 	if (soft) {
 		str = strsep(&soft, "/");
 		if (str[0] == '\0' || soft == NULL || soft[0] == '\0')
 			usage();
 		if (intrd(str, &softb, HN_B) != 0)
 			errx(1, "%s: bad number", str);
 		if (intrd(soft, &softi, 0) != 0)
 			errx(1, "%s: bad number", soft);
+	}
 	if (hard) {
 		str = strsep(&hard, "/");
 		if (str[0] == '\0' || hard == NULL || hard[0] == '\0')
 			usage();
 		if (intrd(str, &hardb, HN_B) != 0)
 			errx(1, "%s: bad number", str);
 		if (intrd(hard, &hardi, 0) != 0)
 			errx(1, "%s: bad number", hard);
+	}
 	if (grace) {
 		str = strsep(&grace, "/");
 		if (str[0] == '\0' || grace == NULL || grace[0] == '\0')
 			usage();
 		if (timeprd(str, &graceb) != 0)
 			errx(1, "%s: bad number", str);
 		if (timeprd(grace, &gracei) != 0)
 			errx(1, "%s: bad number", grace);
+	}
 	for (i=0; i<numnames; i++) {
 		if (names[i] == NULL) {
 			id = 0;
 			dflag = 1;
 		} else {
 			id = getidbyname(names[i], idtype);
 			if (id == -1)
 				continue;
 			dflag = 0;
+		}
 		curprivs = getprivs(id, dflag, idtype, fs);
 		for (lqup = curprivs->head; lqup; lqup = lqup->next) {
 			struct quotaval *q = lqup->qv;
 			if (soft) {
 				if (!dflag && softb &&
 				    q[QL_BLK].qv_usage >= softb &&
 				    (q[QL_BLK].qv_softlimit == 0 ||
 				     q[QL_BLK].qv_usage <
 				     q[QL_BLK].qv_softlimit))
 					q[QL_BLK].qv_expiretime = 0;
 				if (!dflag && softi &&
 				    q[QL_FL].qv_usage >= softb &&
 				    (q[QL_FL].qv_softlimit == 0 ||
 				     q[QL_FL].qv_usage <
 				     q[QL_FL].qv_softlimit))
 					q[QL_FL].qv_expiretime = 0;
 				q[QL_BLK].qv_softlimit = softb;
 				q[QL_FL].qv_softlimit = softi;
+			}
 			if (hard) {
 				q[QL_BLK].qv_hardlimit = hardb;
 				q[QL_FL].qv_hardlimit = hardi;
+			}
 			if (grace) {
 				q[QL_BLK].qv_grace = graceb;
 				q[QL_FL].qv_grace = gracei;
+			}
+		}
 		putprivs(id, idtype, curprivs);
 		quotalist_destroy(curprivs);
+	}
+}
 static void
 clear(const char *fs, int idtype, char **names, int numnames)
+{
 	clearpriv(numnames, names, fs, idtype);
+}
 static void
 editone(const char *fs, int idtype, const char *name,
 	int tmpfd, const char *tmppath)
+{
 	struct quotalist *curprivs;
 	long id;
 	int dflag;
 	if (name == NULL) {
 		id = 0;
 		dflag = 1;
 	} else {
 		id = getidbyname(name, idtype);
 		if (id == -1)
 			return;
 		dflag = 0;
+	}
 	curprivs = getprivs(id, dflag, idtype, fs);
 	if (writeprivs(curprivs, tmpfd, name, idtype) == 0)
 		goto fail;
 	if (editit(tmppath) == 0)
 		goto fail;
 	if (readprivs(curprivs, tmpfd, dflag) == 0)
 		goto fail;
 	putprivs(id, idtype, curprivs);
 fail:
 	quotalist_destroy(curprivs);
+}
 static void
 edit(const char *fs, int idtype, char **names, int numnames)
+{
 	char tmppath[] = _PATH_TMPFILE;
 	int tmpfd, i;
 	tmpfd = mkstemp(tmppath);
 	fchown(tmpfd, getuid(), getgid());
 	for (i=0; i<numnames; i++) {
 		editone(fs, idtype, names[i], tmpfd, tmppath);
+	}
 	close(tmpfd);
 	unlink(tmppath);
+}
 ////////////////////////////////////////////////////////////
 // main
 static void
 usage(void)
+{
 	const char *p = getprogname();
 	fprintf(stderr,
 	    "Usage: %s [-D] [-H] [-u] [-p <username>] [-f <filesystem>] "
 		"-d | <username> ...\n"
 	    "\t%s [-D] [-H] -g [-p <groupname>] [-f <filesystem>] "
 		"-d | <groupname> ...\n"
 	    "\t%s [-D] [-u] [-f <filesystem>] [-s b#/i#] [-h b#/i#] [-t t#/t#] "
 		"-d | <username> ...\n"
 	    "\t%s [-D] -g [-f <filesystem>] [-s b#/i#] [-h b#/i#] [-t t#/t#] "
 		"-d | <groupname> ...\n"
 	    "\t%s [-D] [-H] [-u] -c [-f <filesystem>] username ...\n"
 	    "\t%s [-D] [-H] -g -c [-f <filesystem>] groupname ...\n",
 	    p, p, p, p, p, p);
 	exit(1);
+}
 int
 main(int argc, char *argv[])
+{
 	int idtype;
 	char *protoname;
 	char *soft = NULL, *hard = NULL, *grace = NULL;
 	char *fs = NULL;
 	int ch;
 	int pflag = 0;
 	int cflag = 0;
 	int dflag = 0;
 	if (argc < 2)
 		usage();
 	if (getuid())
 		errx(1, "permission denied");
 	protoname = NULL;
 	idtype = QUOTA_IDTYPE_USER;
 	while ((ch = getopt(argc, argv, "DHcdugp:s:h:t:f:")) != -1) {
 		switch(ch) {
 		case 'D':
 			Dflag++;
 			break;
 		case 'H':
 			Hflag++;
 			break;
 		case 'c':
 			cflag++;
 			break;
 		case 'd':
 			dflag++;
 			break;
 		case 'p':
 			protoname = optarg;
 			pflag++;
 			break;
 		case 'g':
 			idtype = QUOTA_IDTYPE_GROUP;
 			break;
 		case 'u':
 			idtype = QUOTA_IDTYPE_USER;
 			break;
 		case 's':
 			soft = optarg;
 			break;
 		case 'h':
 			hard = optarg;
 			break;
 		case 't':
 			grace = optarg;
 			break;
 		case 'f':
 			fs = optarg;
 			break;
 		default:
 			usage();
+		}
+	}
 	argc -= optind;
 	argv += optind;
 	if (pflag) {
 		if (soft || hard || grace || dflag || cflag)
 			usage();
 		replicate(fs, idtype, protoname, argv, argc);
 	} else if (soft || hard || grace) {
 		if (cflag)
 			usage();
 		if (dflag) {
 			/* use argv[argc], which is null, to mean 'default' */
 			argc++;
+		}
 		assign(fs, idtype, soft, hard, grace, argv, argc);
 	} else if (cflag) {
 		if (dflag)
 			usage();
 		clear(fs, idtype, argv, argc);
 	} else {
 		if (dflag) {
 			/* use argv[argc], which is null, to mean 'default' */
 			argc++;
+		}
 		edit(fs, idtype, argv, argc);
+	}
 	return 0;
+}