Add asserts that the user doesn't try to add the empty or deleted values.

2 files changed, 16 insertions, 0 deletions

diff --git a/hash_table.c b/hash_table.c
index 8390472..95f5bd3 100644
--- a/hash_table.c
+++ b/hash_table.c
@@ -32,6 +32,7 @@
  *    Keith Packard <keithp@keithp.com>
  */
 
+#include <assert.h>
 #include <stdlib.h>
 
 #include "hash_table.h"
@@ -244,6 +245,13 @@ hash_table_insert(struct hash_table *ht, const void *key, void *data)
 {
 	uint32_t hash = ht->hash_function(key);
 
+	/* Make sure nobody tries to add one of the magic values as a
+	 * key. If you need to do so, either do so in a wrapper, or
+	 * store keys with the magic values separately in the struct
+	 * hash_table.
+	 */
+	assert(key != NULL);
+
 	return hash_table_insert_pre_hashed(ht, hash, key, data);
 }
 
diff --git a/set.c b/set.c
index 5033892..b4bde73 100644
--- a/set.c
+++ b/set.c
@@ -32,6 +32,7 @@
  *    Keith Packard <keithp@keithp.com>
  */
 
+#include <assert.h>
 #include <stdlib.h>
 
 #include "set.h"
@@ -258,6 +259,13 @@ set_add(struct set *set, const void *key)
 {
 	uint32_t hash = set->hash_function(key);
 
+	/* Make sure nobody tries to add one of the magic values as a
+	 * key. If you need to do so, either do so in a wrapper, or
+	 * store keys with the magic values separately in the struct
+	 * set.
+	 */
+	assert(key != NULL);
+
 	return set_add_pre_hashed(set, hash, key);
 }