From: Andrew Deason <adeason@sinenomine.net>
Date: Fri, 2 Mar 2012 23:18:25 +0000 (-0600)
Subject: afs: Log a message on invalid FetchStatus receipt
X-Git-Tag: openafs-stable-1_8_0pre1~2680
X-Git-Url: http://git.openafs.org/?p=openafs.git;a=commitdiff_plain;h=577d5d99fefb62f0b52a568207d2352946b36137

afs: Log a message on invalid FetchStatus receipt

If we get an invalid AFSFetchStatus structure from a server, log a
message to indicate as such. This serves as a warning to urge people
to fix their fileservers, and to explain what is doing.

Change-Id: I33845f6228f96f693dcbfddcee38dae583f1b092
Reviewed-on: http://gerrit.openafs.org/6881
Reviewed-by: Alistair Ferguson <alistair.ferguson@mac.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementix.org>
---

diff --git a/src/afs/VNOPS/afs_vnop_lookup.c b/src/afs/VNOPS/afs_vnop_lookup.c
index c14b4a5..60064c4 100644
--- a/src/afs/VNOPS/afs_vnop_lookup.c
+++ b/src/afs/VNOPS/afs_vnop_lookup.c
@@ -630,6 +630,10 @@ afs_CheckBulkStatus(struct afs_conn *tc, int nFids, AFSBulkStats *statParm,
     int code;
 
     if (statParm->AFSBulkStats_len != nFids || cbParm->AFSCBs_len != nFids) {
+	afs_warn("afs: BulkFetchStatus length %u/%u, expected %u\n",
+	         (unsigned)statParm->AFSBulkStats_len,
+	         (unsigned)cbParm->AFSCBs_len, nFids);
+	afs_BadFetchStatus(tc);
 	return VBUSY;
     }
     for (i = 0; i < nFids; i++) {
diff --git a/src/afs/afs_prototypes.h b/src/afs/afs_prototypes.h
index 2633528..448bcf4 100644
--- a/src/afs/afs_prototypes.h
+++ b/src/afs/afs_prototypes.h
@@ -1071,6 +1071,7 @@ extern void afs_FlushReclaimedVcaches(void);
 void afs_vcacheInit(int astatSize);
 extern struct vcache *afs_FindVCache(struct VenusFid *afid, afs_int32 * retry,
 				     afs_int32 flag);
+extern void afs_BadFetchStatus(struct afs_conn *tc);
 extern int afs_CheckFetchStatus(struct afs_conn *tc,
                                 struct AFSFetchStatus *status);
 extern afs_int32 afs_FetchStatus(struct vcache *avc, struct VenusFid *afid,
diff --git a/src/afs/afs_vcache.c b/src/afs/afs_vcache.c
index 4f1372b..8a92b99 100644
--- a/src/afs/afs_vcache.c
+++ b/src/afs/afs_vcache.c
@@ -2346,6 +2346,19 @@ afs_UpdateStatus(struct vcache *avc, struct VenusFid *afid,
     	afs_PutVolume(volp, READ_LOCK);
 }
 
+void
+afs_BadFetchStatus(struct afs_conn *tc)
+{
+    int addr = ntohl(tc->parent->srvr->sa_ip);
+    afs_warn("afs: Invalid AFSFetchStatus from server %u.%u.%u.%u\n",
+             (addr >> 24) & 0xff, (addr >> 16) & 0xff, (addr >> 8) & 0xff,
+             (addr) & 0xff);
+    afs_warn("afs: This suggests the server may be sending bad data that "
+             "can lead to availability issues or data corruption. The "
+             "issue has been avoided for now, but it may not always be "
+             "detectable. Please upgrade the server if possible.\n");
+}
+
 /**
  * Check if a given AFSFetchStatus structure is sane.
  *
@@ -2365,6 +2378,12 @@ afs_CheckFetchStatus(struct afs_conn *tc, struct AFSFetchStatus *status)
         !(status->FileType > Invalid && status->FileType <= SymbolicLink) ||
         status->ParentVnode == 0 || status->ParentUnique == 0) {
 
+	afs_warn("afs: FetchStatus ec %u iv %u ft %u pv %u pu %u\n",
+	         (unsigned)status->errorCode, (unsigned)status->InterfaceVersion,
+	         (unsigned)status->FileType, (unsigned)status->ParentVnode,
+	         (unsigned)status->ParentUnique);
+	afs_BadFetchStatus(tc);
+
 	return VBUSY;
     }
     return 0;