From owner-cvs-all@FreeBSD.ORG  Fri Oct 14 11:21:21 2005
Return-Path: <owner-cvs-all@FreeBSD.ORG>
X-Original-To: cvs-all@FreeBSD.org
Delivered-To: cvs-all@FreeBSD.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 8297816A41F;
	Fri, 14 Oct 2005 11:21:21 +0000 (GMT) (envelope-from rse@FreeBSD.org)
Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 559B343D4C;
	Fri, 14 Oct 2005 11:21:21 +0000 (GMT) (envelope-from rse@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.13.1/8.13.1) with ESMTP id j9EBLL5G041612;
	Fri, 14 Oct 2005 11:21:21 GMT (envelope-from rse@repoman.freebsd.org)
Received: (from rse@localhost)
	by repoman.freebsd.org (8.13.1/8.13.1/Submit) id j9EBLLMn041611;
	Fri, 14 Oct 2005 11:21:21 GMT (envelope-from rse)
Message-Id: <200510141121.j9EBLLMn041611@repoman.freebsd.org>
From: "Ralf S. Engelschall" <rse@FreeBSD.org>
Date: Fri, 14 Oct 2005 11:21:21 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Cc: 
Subject: cvs commit: src/sbin/mdmfs mdmfs.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Fri, 14 Oct 2005 11:21:21 -0000

rse         2005-10-14 11:21:21 UTC

  FreeBSD src repository

  Modified files:
    sbin/mdmfs           mdmfs.c 
  Log:
  Fix parsing of mdmfs(8) option "-w <user>:<group>" in case <user> or
  <group> is a numeric user/group ID instead of a user/group name (as
  explicitly intended to be allowed by both the manual page and the
  implementation).
  
  Before this fix, mdmfs(8) aborted:
  
  | # mdmfs -s 32m -w 0:0 md /var/tmp/foo
  | Assertion failed: (mip->mi_have_uid), function extract_ugid, file /usr/src/sbin/mdmfs/mdmfs.c, line 555.
  | Abort trap (core dumped)
  
  The "mi_have_[ug]id" fields were only set in case a name lookup was
  successful. Instead they also have to be set in case the string to
  integer conversion was successful.
  
  Additionally, as a result of this fix, two assertions at the end of
  the function are now always true and hence can be just be removed. It
  is guarrantied that both the UID and the GID are set when the function
  returns regularily, else it would have been already bailed out with
  usage()/exit(3) or errx(3) before.
  
  Spotted by:     Christoph Schug <chris@schug.net>
  MFC after:      3 days
  
  Revision  Changes    Path
  1.24      +6 -10     src/sbin/mdmfs/mdmfs.c