From owner-cvs-all@FreeBSD.ORG  Tue Aug  7 10:35:28 2007
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 0325316A418;
	Tue,  7 Aug 2007 10:35:28 +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 E367113C46A;
	Tue,  7 Aug 2007 10:35:27 +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 l77AZRph049423;
	Tue, 7 Aug 2007 10:35:27 GMT (envelope-from bde@repoman.freebsd.org)
Received: (from bde@localhost)
	by repoman.freebsd.org (8.14.1/8.14.1/Submit) id l77AZR3v049422;
	Tue, 7 Aug 2007 10:35:27 GMT (envelope-from bde)
Message-Id: <200708071035.l77AZR3v049422@repoman.freebsd.org>
From: Bruce Evans <bde@FreeBSD.org>
Date: Tue, 7 Aug 2007 10:35:27 +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_vnops.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: Tue, 07 Aug 2007 10:35:28 -0000

bde         2007-08-07 10:35:27 UTC

  FreeBSD src repository

  Modified files:
    sys/fs/msdosfs       msdosfs_vnops.c 
  Log:
  In msdosfs_read() and msdosfs_write(), don't check explicitly for
  (uio_offset < 0) since this can't happen.  If this happens, then the
  general code handles the problem safely (better than before for reading,
  returning 0 (EOF) instead of the bogus errno EINVAL, and the same as
  before for writing, returning EFBIG).
  
  In msdosfs_read(), don't check for (uio_resid < 0).  msdosfs_write()
  already didn't check.
  
  In msdosfs_read(), document in a comment our assumptions that the caller
  passed a valid uio_offset and uio_resid.  ffs checks using KASSERT(),
  and that is enough sanity checking.  In the same comment, partly document
  there is no need to check for the EOVERFLOW case, unlike in ffs where this
  case can happen at least in theory.
  
  In msdosfs_write(), add a comment about why the checking of
  (uio_resid == 0) is explicit, unlike in ffs.
  
  In msdosfs_write(), check for impossibly large final offsets before
  checking if the file size rlimit would be exceeded, so that we don't
  have an overflow bug in the rlimit check and are consistent with ffs.
  We now return EFBIG instead of EFBIG plus a SIGXFSZ signal if the final
  offset would be impossibly large but not so large as to cause overflow.
  Overflow normally gave the benign behaviour of no signal.
  
  Approved by:    re (kensmith) (blanket)
  
  Revision  Changes    Path
  1.178     +21 -12    src/sys/fs/msdosfs/msdosfs_vnops.c