From adam@veda.is  Thu Jul 31 13:01:02 1997
Received: from veda.is (root@veda.is [193.4.230.1])
          by hub.freebsd.org (8.8.5/8.8.5) with ESMTP id NAA29500
          for <FreeBSD-gnats-submit@freebsd.org>; Thu, 31 Jul 1997 13:01:01 -0700 (PDT)
Received: from ubiq.veda.is (adam@ubiq.veda.is [193.4.230.60])
	by veda.is (8.8.6/8.8.5) with ESMTP id UAA07250
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 31 Jul 1997 20:00:58 GMT
Received: (from adam@localhost)
	by ubiq.veda.is (8.8.6/8.8.5) id UAA02655;
	Thu, 31 Jul 1997 20:00:55 GMT
Message-Id: <199707312000.UAA02655@ubiq.veda.is>
Date: Thu, 31 Jul 1997 20:00:55 GMT
From: Adam David <adam@veda.is>
Reply-To: adam@veda.is
To: FreeBSD-gnats-submit@freebsd.org
Subject: pwd_mkdb trashes .db
X-Send-Pr-Version: 3.2

>Number:         4202
>Category:       bin
>Synopsis:       pwd_mkdb trashes .db
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    freebsd-bugs
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Thu Jul 31 13:10:02 PDT 1997
>Closed-Date:    Tue Feb 17 20:43:50 MET 1998
>Last-Modified:  Tue Feb 17 20:46:11 MET 1998
>Originator:     Adam David
>Release:        FreeBSD 3.0-CURRENT i386
>Organization:
Veda Internet
>Environment:

	FreeBSD

>Description:

	pwd_mkdb trashes spwd.db when the root partition is full.
	this is not acceptable behaviour, it should fail gracefully.

>How-To-Repeat:

	(almost) fill the root partition and add a user.

>Fix:
	
	a) don't do that
	b) move the old database files out of the way and move them back
	   if we fail.
>Release-Note:
>Audit-Trail:
State-Changed-From-To: open->feedback 
State-Changed-By: jlemon 
State-Changed-When: Mon Aug 18 13:49:20 PDT 1997 
State-Changed-Why:  

Should be fixed by pwd_mkdb.c, rev 1.19. 
State-Changed-From-To: feedback->closed 
State-Changed-By: guido 
State-Changed-When: Tue Feb 17 20:43:50 MET 1998 
State-Changed-Why:  
The fix in rev 1.19 was partial. rev 1.23 should have it really fixed. 
This was comfirmed by the submitter. 
The fix was backported to 2.2-stable 
>Unformatted:
