On reflection, freeing keys and values in hashtable processing is not a good idea because we don't know what they are, and can put up with some small memory losses.

server/core/hashtable.c

@@ -763,8 +763,6 @@ char		buf[40];
 		value = valueread(fd);
 		if (key == NULL || value == NULL)
                 {
-                    free(key);
-                    free(value);
                     break;
                 }
 		hashtable_add(table, key, value);