From 37c5c707511e632a39a5e69e03f9ec7ae17d6ede Mon Sep 17 00:00:00 2001 From: Antoine Verheijen Date: Tue, 17 May 2011 09:40:15 -0600 Subject: [PATCH] Replace uintptr_t type cast with uintptrsz in afs_vcache.c A recent change (commit 80fe111f0044aa7a67215ad92210dc72cb7eb2c0) to afs_vcache.c contains a call to afs_warn() whose second parameter contains a "(uintptr_t)" type cast as part of a double type cast. This presents an issue on some systems, such as OpenBSD, where this object type is defined in a header that is not presently included. This change modifies that type cast to instead use the AFS-internal "(uintptrsz)" type which should provide the same effect. Note that an earlier version of this patch attempted to remove the "offending" type cast as redundant but it was pointed out that some systems require this kind of cascading type cast when casting pointers to integers to deal with possible size issues. Reviewed-on: http://gerrit.openafs.org/4671 Reviewed-by: Derrick Brashear Tested-by: Derrick Brashear (cherry picked from commit b3004fbd5007d400a4cd3fb2a5b1c52d98618aab) Change-Id: I3c7452d1f09dc7fdbcffe9b047e4ec682ac6ea03 Reviewed-on: http://gerrit.openafs.org/4705 Tested-by: BuildBot Reviewed-by: Derrick Brashear --- src/afs/afs_vcache.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/afs/afs_vcache.c b/src/afs/afs_vcache.c index 192663007..d775b7c40 100644 --- a/src/afs/afs_vcache.c +++ b/src/afs/afs_vcache.c @@ -799,7 +799,7 @@ afs_FlushAllVCaches(void) nvc = tvc->hnext; if (afs_FlushVCache(tvc, &slept)) { - afs_warn("Failed to flush vcache 0x%lx\n", (unsigned long)(uintptr_t)tvc); + afs_warn("Failed to flush vcache 0x%lx\n", (unsigned long)(uintptrsz)tvc); } if (slept) { goto retry; -- 2.39.5