From 4c8a3cd789a8495d0c0b3f8ff1a7ebb2a20f5336 Mon Sep 17 00:00:00 2001
From: Derrick Brashear <shadow@dementix.org>
Date: Tue, 14 Feb 2012 21:58:09 -0500
Subject: [PATCH] libafs: dont handle outstatus on write error

if a write errored, we can't trust the OutStatus we got. don't.

Change-Id: I74934c32426c6f25aace06f6ee59c1c1b547af90
Reviewed-on: http://gerrit.openafs.org/6714
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@secure-endpoints.com>
Reviewed-by: Alistair Ferguson <alistair.ferguson@mac.com>
Reviewed-by: Derrick Brashear <shadow@dementix.org>
---
 src/afs/afs_fetchstore.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/afs/afs_fetchstore.c b/src/afs/afs_fetchstore.c
index 16c05502bd..9fb766fb43 100644
--- a/src/afs/afs_fetchstore.c
+++ b/src/afs/afs_fetchstore.c
@@ -543,6 +543,7 @@ afs_CacheStoreDCaches(struct vcache *avc, struct dcache **dclist,
 
     if (!code) {
 	code = (*ops->close)(rock, OutStatus, doProcessFS);
+	/* if this succeeds, dv has been bumped. */
 	if (*doProcessFS) {
 	    hadd32(*anewDV, 1);
 	}
@@ -550,6 +551,11 @@ afs_CacheStoreDCaches(struct vcache *avc, struct dcache **dclist,
     }
     if (ops)
 	code = (*ops->destroy)(&rock, code);
+
+    /* if we errored, can't trust this. */
+    if (code)
+	*doProcessFS = 0;
+
     return code;
 }