]> git.michaelhowe.org Git - packages/o/openafs.git/commit
Windows: buf_CleanLocked protect against NULL bp->userp
authorJeffrey Altman <jaltman@your-file-system.com>
Sun, 18 Nov 2012 02:44:52 +0000 (21:44 -0500)
committerJeffrey Altman <jaltman@your-file-system.com>
Sat, 8 Dec 2012 16:19:31 +0000 (08:19 -0800)
commitad7cfa94e61217ee46b52bdf0704cd2aeca127a9
treec606b30cb9b81c69e4606c6752d6d930cfe579ad
parentc95d1aee53eb5f459e3cfff27bcdd08dbb00095a
Windows: buf_CleanLocked protect against NULL bp->userp

The cm_buf_t.userp field should never be NULL if the CM_BUF_DIRTY
flag is set but apparently it sometimes is.  cm_BufWrite() requires
that the userp parameter be non-NULL.  Otherwise, an assertion fails
and afsd_service.exe panics.  If bp->userp is NULL, use cm_rootUserp.
The worst that will happen is the write will fail due to an access
denied error.

Reviewed-on: http://gerrit.openafs.org/8475
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: Jeffrey Altman <jaltman@your-file-system.com>
(cherry picked from commit f436fe03c80e21b6a58a6de962070841e00791ef)

Change-Id: I33f5e73509cc2c6517ae4bf4214011383a341411
Reviewed-on: http://gerrit.openafs.org/8649
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
src/WINNT/afsd/cm_buf.c