From: Antoine Verheijen <antoine@ualberta.ca>
Date: Wed, 19 Jan 2011 18:52:10 +0000 (-0700)
Subject: OpenBSD: Fix use of macros for AFS_KALLOC/AFS_KFREE
X-Git-Tag: openafs-devel-1_7_1~1023
X-Git-Url: https://git.openafs.org/?p=openafs.git;a=commitdiff_plain;h=d5367b5f1494ca2cf09f62d7f2ffa6304e38a648

OpenBSD: Fix use of macros for AFS_KALLOC/AFS_KFREE

The macro definitions for AFS_KALLOC and AFS_KFREE used by
afs_osi_alloc.c to perform OS-specific memory allocation
are only defined by the OpenBSD osi_machdep.h header file
if they already exist, which means: never. This patch fixes
this situtation and makes sure that afs_osi_alloc.c
actually uses them, eliminating a direct OS-specific call
in afs_osi_alloc.c.

Change-Id: Ida1608774de3bc5861b223574f3cbf0d9647266c
Reviewed-on: http://gerrit.openafs.org/3682
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
---

diff --git a/src/afs/OBSD/osi_machdep.h b/src/afs/OBSD/osi_machdep.h
index 61ed43e..c0599f2 100644
--- a/src/afs/OBSD/osi_machdep.h
+++ b/src/afs/OBSD/osi_machdep.h
@@ -66,13 +66,13 @@ extern void osi_obsd_Free(void *p, size_t asize);
 
 #ifdef AFS_KALLOC
 #undef AFS_KALLOC
-#define AFS_KALLOC(s) osi_obsd_Alloc((s), 1 /* cansleep */)
 #endif
+#define AFS_KALLOC(s) osi_obsd_Alloc((s), 1 /* cansleep */)
 
 #ifdef AFS_KFREE
 #undef AFS_KFREE
-#define AFS_KFREE(p, s) (osi_obsd_Free((p), (s)))
 #endif
+#define AFS_KFREE(p, s) (osi_obsd_Free((p), (s)))
 
 #ifdef AFS_OBSD42_ENV
 /* removed, live with it */
diff --git a/src/afs/afs_osi_alloc.c b/src/afs/afs_osi_alloc.c
index d2d9b34..726ccff 100644
--- a/src/afs/afs_osi_alloc.c
+++ b/src/afs/afs_osi_alloc.c
@@ -73,8 +73,6 @@ afs_osi_Free(void *x, size_t asize)
     osi_linux_free(x);
 #elif defined(AFS_FBSD_ENV)
     osi_fbsd_free(x);
-#elif defined(AFS_OBSD_ENV)
-    osi_obsd_Free(x, asize);
 #else
     AFS_KFREE(x, asize);
 #endif