From: Andrew Deason <adeason@sinenomine.net>
Date: Thu, 26 Dec 2013 22:17:44 +0000 (-0500)
Subject: afs: Fix afs_CheckCode identifier collision
X-Git-Tag: upstream/1.8.0_pre1^2~808
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=e459f44efef8d102c54205556f65318d63fec24f;p=packages%2Fo%2Fopenafs.git

afs: Fix afs_CheckCode identifier collision

The last argument to afs_CheckCode should be unique so the call site
can be identified if fstrace is turned on. BStore and BPartialStore
were both using 43, so change BPartialStore to 430 to avoid the
collision.

Change-Id: I81a43ee41623fad10d0e70a7d9c8e6029aba30eb
Reviewed-on: http://gerrit.openafs.org/10635
Reviewed-by: Perry Ruiter <pruiter@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
---

diff --git a/src/afs/afs_daemons.c b/src/afs/afs_daemons.c
index da68e18f1..0aab8c921 100644
--- a/src/afs/afs_daemons.c
+++ b/src/afs/afs_daemons.c
@@ -608,7 +608,7 @@ BStore(struct brequest *ab)
 	 * can know the "raw" value for interpreting the value internally, as
 	 * well as the afs_CheckCode value to give to the OS. */
 	ab->code_raw = code;
-	ab->code_checkcode = afs_CheckCode(code, &treq, 43);
+	ab->code_checkcode = afs_CheckCode(code, &treq, 430);
 
 	ab->flags |= BUVALID;
 	if (ab->flags & BUWAIT) {