Switch the locknum generation for qpznode to random

Instead of using on hash of the name modulo number of the buckets,
assign the locknum randomly with isc_random_uniform().  This makes
the locknum assignment aligned with qpcache and allows the bucket
number to be non-prime in the future.

diff --git a/lib/dns/qpzone.c b/lib/dns/qpzone.c
index a1b7b97a207cff2312f52eee471b00d61fb8ef1f..e0ddab3bfbfc35bf19cfe6e51e0f928473256d34 100644 (file)
--- a/lib/dns/qpzone.c
+++ b/lib/dns/qpzone.c
@@ -595,10 +595,11 @@ new_qpznode(qpzonedb_t *qpdb, const dns_name_t *name) {
        *newdata = (qpznode_t){
                .name = DNS_NAME_INITEMPTY,
                .references = ISC_REFCOUNT_INITIALIZER(1),
+               .locknum = isc_random_uniform(qpdb->buckets_count),
        };
-       newdata->locknum = dns_name_hash(name) % qpdb->buckets_count;
-       dns_name_dupwithoffsets(name, qpdb->common.mctx, &newdata->name);
+
        isc_mem_attach(qpdb->common.mctx, &newdata->mctx);
+       dns_name_dupwithoffsets(name, qpdb->common.mctx, &newdata->name);
 
 #if DNS_DB_NODETRACE
        fprintf(stderr, "new_qpznode:%s:%s:%d:%p->references = 1\n", __func__,