From: Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca>
Date: Mon, 5 Oct 2009 19:29:09 +0000 (-0400)
Subject: Add type checking in urcu-pointer.h macros
X-Git-Tag: v0.2.2~1
X-Git-Url: http://git.liburcu.org/?p=urcu.git;a=commitdiff_plain;h=2ff1db4a7825af40f327413305de6f9955af0e35

Add type checking in urcu-pointer.h macros

Ensure we have type-checking around dynamic linking macro wrappers.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca>
---

diff --git a/urcu-pointer.c b/urcu-pointer.c
index 93d4987..da8b1ea 100644
--- a/urcu-pointer.c
+++ b/urcu-pointer.c
@@ -30,7 +30,7 @@
 
 extern void synchronize_rcu(void);
 
-void *rcu_dereference(void *p)
+void *rcu_dereference_sym(void *p)
 {
 	return _rcu_dereference(p);
 }
diff --git a/urcu-pointer.h b/urcu-pointer.h
index 0a3524f..d3718b0 100644
--- a/urcu-pointer.h
+++ b/urcu-pointer.h
@@ -60,21 +60,36 @@
 
 #else /* !_LGPL_SOURCE */
 
-extern void *rcu_dereference(void *p);
+extern void *rcu_dereference_sym(void *p);
+#define rcu_dereference(p)						     \
+	({								     \
+		typeof(p) _________p1 =					     \
+			rcu_dereference_sym((void *)(p));		     \
+		(_________p1);						     \
+	})
 
 extern void *rcu_cmpxchg_pointer_sym(void **p, void *old, void *_new);
-#define rcu_cmpxchg_pointer(p, old, _new)		\
-	rcu_cmpxchg_pointer_sym((void **)(p), (old), (_new))
+#define rcu_cmpxchg_pointer(p, old, _new)				     \
+	({								     \
+		typeof(*p) _________p1 =				     \
+			rcu_cmpxchg_pointer_sym((void **)(p), (old), (_new));\
+		(_________p1);						     \
+	})
 
 extern void *rcu_xchg_pointer_sym(void **p, void *v);
-#define rcu_xchg_pointer(p, v)				\
-	rcu_xchg_pointer_sym((void **)(p), (v))
+#define rcu_xchg_pointer(p, v)						     \
+	({								     \
+		typeof(*p) _________p1 =				     \
+			rcu_xchg_pointer_sym((void **)(p), (v));	     \
+		(_________p1);						     \
+	})
 
 extern void *rcu_set_pointer_sym(void **p, void *v);
-#define rcu_set_pointer(p, v)				\
-	rcu_set_pointer_sym((void **)(p), (v))
-
-extern void *rcu_assign_pointer_sym(void **p, void *v);
+#define rcu_set_pointer(p, v)						     \
+	({								     \
+		typeof(*p) _________p1 =				     \
+			rcu_set_pointer_sym((void **)(p), (v));		     \
+	})
 
 #endif /* !_LGPL_SOURCE */