afs: Fix afs_CheckCode identifier collision
authorAndrew Deason <adeason@sinenomine.net>
Thu, 26 Dec 2013 22:17:44 +0000 (17:17 -0500)
committerDerrick Brashear <shadow@your-file-system.com>
Wed, 29 Jan 2014 15:34:55 +0000 (07:34 -0800)
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>

src/afs/afs_daemons.c

index da68e18..0aab8c9 100644 (file)
@@ -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) {