From owner-cvs-src@FreeBSD.ORG  Sun Aug  5 12:58:35 2007
Return-Path: <owner-cvs-src@FreeBSD.ORG>
Delivered-To: cvs-src@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 7BB7F16A418;
	Sun,  5 Aug 2007 12:58:35 +0000 (UTC) (envelope-from bde@FreeBSD.org)
Received: from repoman.freebsd.org (repoman.freebsd.org
	[IPv6:2001:4f8:fff6::29])
	by mx1.freebsd.org (Postfix) with ESMTP id 7A2EA13C48E;
	Sun,  5 Aug 2007 12:58:35 +0000 (UTC) (envelope-from bde@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.14.1/8.14.1) with ESMTP id l75CwZSc098368;
	Sun, 5 Aug 2007 12:58:35 GMT (envelope-from bde@repoman.freebsd.org)
Received: (from bde@localhost)
	by repoman.freebsd.org (8.14.1/8.14.1/Submit) id l75CwZQ6098367;
	Sun, 5 Aug 2007 12:58:35 GMT (envelope-from bde)
Message-Id: <200708051258.l75CwZQ6098367@repoman.freebsd.org>
From: Bruce Evans <bde@FreeBSD.org>
Date: Sun, 5 Aug 2007 12:58:34 +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/sys/fs/msdosfs msdosfs_vfsops.c
X-BeenThere: cvs-src@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the src tree <cvs-src.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src>
List-Post: <mailto:cvs-src@freebsd.org>
List-Help: <mailto:cvs-src-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 05 Aug 2007 12:58:35 -0000

bde         2007-08-05 12:58:34 UTC

  FreeBSD src repository

  Modified files:
    sys/fs/msdosfs       msdosfs_vfsops.c 
  Log:
  Silently fix up the estimated next free cluster number from the fsinfo
  sector, instead of failing the whole mount if it is garbage.  Fields
  in the fsinfo sector are only advisory, so there are better sanity
  checks than this, and we already silently fix up the only other advisory
  field in the fsinfo (the free cluster count).
  
  This wasn't handled quite right in rev.1.92, 1.117, or in NetBSD.  1.92
  also failed the whole mount for the non-garbage magic value 0xffffffff
  1.117 fixed this well enough in practice since garbage values shouldn't
  occur in practice, but left the error handling larger and more convoluted
  than necessary.  Now we handle the magic value as a special case of
  fixing up all out of bounds values.
  
  Also fix up the estimated next free cluster number when there is no
  fsinfo sector.  We were using 0, but CLUST_FIRST is safer.
  
  Approved by:    re (kensmith)
  
  Revision  Changes    Path
  1.169     +12 -10    src/sys/fs/msdosfs/msdosfs_vfsops.c