OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-1548-g5836464

Gerrit Code Review gerrit@openafs.org
Mon, 22 Aug 2011 12:13:54 -0700 (PDT)


The following commit has been merged in the master branch:
commit 5836464f454f0bb4612c665d27061956d4c745af
Author: Garrett Wollman <wollman@csail.mit.edu>
Date:   Sat Aug 13 18:34:47 2011 -0400

    libafs: don't crash if afs_write() is called with zero-length uio
    
    If AFS_UIO_RESID(auio) is zero in afs_write(), we could end up
    calling afsio_free(NULL).  Guard the free.  (In the alternative,
    perhaps we should just osi_Assert(totalLength > 0) instead?)
    
    Change-Id: Ic218f039e2034b679cb4817a543af2e8307f36f8
    Found-by: clang static analyzer with the help of AFS_NONNULL
    Reviewed-on: http://gerrit.openafs.org/5259
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@dementix.org>

 src/afs/VNOPS/afs_vnop_write.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

-- 
OpenAFS Master Repository