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