1 files changed, 12 insertions, 10 deletions

diff --git a/ot_accesslist.c b/ot_accesslist.c
index ef845d6..4bf1701 100644
--- a/ot_accesslist.c
+++ b/ot_accesslist.c
@@ -21,8 +21,6 @@
 #include <errno.h>
 #endif
 
-/* Libowfat */
-
 /* Opentracker */
 #include "ot_accesslist.h"
 #include "ot_hex.h"
@@ -110,7 +108,7 @@ size_t accesslist_parse_hashes(ot_hash *dest, size_t capacity, const char *buf,
   size_t      count = 0;
 
   while (count < capacity && end - offs >= 40) {
-    if (hex_tohash(dest[count], offs)) {
+    if (hex_to_hash(dest[count], offs)) {
       offs += 40;
 
       if (offs == end || !hex_isdigit((unsigned char)*offs))
@@ -162,6 +160,9 @@ static void accesslist_readfile(void) {
   const char    *map;
   size_t         maplen, capacity, found;
 
+  if (!g_accesslist_filename)
+    return;
+
   if (accesslist_map(g_accesslist_filename, &map, &maplen) < 0) {
     char *wd = getcwd(NULL, 0);
     fprintf(stderr, "Warning: Can't open accesslist file: %s (but will try to create it later, if necessary and possible).\nPWD: %s\n", g_accesslist_filename, wd);
@@ -187,7 +188,8 @@ static void accesslist_readfile(void) {
   if (map)
     munmap((void *)map, maplen);
 
-  qsort(accesslist_new->list, found, sizeof(ot_hash), vector_compare_hash);
+  if (found)
+    qsort(accesslist_new->list, found, sizeof(ot_hash), vector_compare_hash);
   accesslist_new->size = found;
 
   /* Now exchange the accesslist vector in the least race condition prone way */
@@ -225,18 +227,18 @@ int accesslist_hashisvalid(const ot_hash hash) {
 #endif
   void *exactmatch = NULL;
 
-  if (accesslist)
+  if (accesslist && accesslist->size)
     exactmatch = bsearch(hash, accesslist->list, accesslist->size, OT_HASH_COMPARE_SIZE, vector_compare_hash);
 
 #ifdef WANT_DYNAMIC_ACCESSLIST
   /* If we had no match on the main list, scan the list of dynamically added hashes */
   accesslist_add = g_accesslist_add;
-  if ((exactmatch == NULL) && accesslist_add)
+  if ((exactmatch == NULL) && accesslist_add && accesslist_add->size)
     exactmatch = bsearch(hash, accesslist_add->list, accesslist_add->size, OT_HASH_COMPARE_SIZE, vector_compare_hash);
 
   /* If we found a matching hash on the main list, scan the list of dynamically deleted hashes */
   accesslist_delete = g_accesslist_delete;
-  if ((exactmatch != NULL) && accesslist_delete && bsearch(hash, accesslist_delete->list, accesslist_delete->size, OT_HASH_COMPARE_SIZE, vector_compare_hash))
+  if ((exactmatch != NULL) && accesslist_delete && accesslist_delete->size && bsearch(hash, accesslist_delete->list, accesslist_delete->size, OT_HASH_COMPARE_SIZE, vector_compare_hash))
     exactmatch = NULL;
 #endif
 
@@ -308,7 +310,7 @@ static void     *accesslist_adddel_worker(const char *fifoname, ot_accesslist *_
       if (linelen < 41)
         continue;
 
-      if (!hex_tohash(info_hash, line))
+      if (!hex_to_hash(info_hash, line))
         continue;
 
       /* From now on we modify g_accesslist_add and g_accesslist_delete, so prevent the
@@ -386,8 +388,9 @@ void accesslist_init(void) {
 }
 
 void accesslist_deinit(void) {
-  /* Wake up sleeping worker */
+  /* Wake the worker and wait for it to notice that the tracker is stopping. */
   pthread_kill(thread_id, SIGHUP);
+  pthread_join(thread_id, NULL);
 
   pthread_mutex_lock(&g_accesslist_mutex);
 
@@ -399,7 +402,6 @@ void accesslist_deinit(void) {
 #endif
 
   pthread_mutex_unlock(&g_accesslist_mutex);
-  pthread_cancel(thread_id);
   pthread_mutex_destroy(&g_accesslist_mutex);
 }