]> git.ipfire.org Git - thirdparty/unbound.git/commitdiff
- openbsd-lint fixes: acl_list_get_mem used if debug-alloc enabled.
authorWouter Wijngaards <wouter@nlnetlabs.nl>
Fri, 20 Aug 2010 13:30:41 +0000 (13:30 +0000)
committerWouter Wijngaards <wouter@nlnetlabs.nl>
Fri, 20 Aug 2010 13:30:41 +0000 (13:30 +0000)
  iterator get_mem includes priv_get_mem.  delegpt nodup removed.
  listen_pushback, query_info_allocqname, write_socket, send_packet,
  comm_point_set_cb_arg and listen_resume removed.

git-svn-id: file:///svn/unbound/trunk@2222 be551aaa-1e26-0410-a405-d3ace91eadb9

25 files changed:
daemon/remote.c
daemon/worker.c
daemon/worker.h
doc/Changelog
iterator/iter_delegpt.c
iterator/iter_delegpt.h
iterator/iter_fwd.c
iterator/iter_hints.c
iterator/iter_utils.c
iterator/iterator.c
libunbound/libworker.c
libunbound/libworker.h
services/cache/dns.c
services/listen_dnsport.c
services/listen_dnsport.h
testcode/fake_event.c
util/data/msgreply.c
util/data/msgreply.h
util/fptr_wlist.c
util/fptr_wlist.h
util/module.h
util/net_help.c
util/net_help.h
util/netevent.c
util/netevent.h

index 22c0cceb282ace85c45d5fd57f54a0174aa7a0a4..3a1c32fa76c30571155420053447a44f0423d794 100644 (file)
@@ -1317,7 +1317,7 @@ parse_delegpt(SSL* ssl, struct regional* region, char* args, uint8_t* root)
                        return NULL;
                }
                /* add address */
-               if(!delegpt_add_addr(dp, region, &addr, addrlen, 0, 0, 1)) {
+               if(!delegpt_add_addr(dp, region, &addr, addrlen, 0, 0)) {
                        (void)ssl_printf(ssl, "error out of memory\n");
                        return NULL;
                }
index cfb484a6755d36e6f5e194f7e0027dbaaaab62a9..6cb6bb22685f91487ce5affcaa4717dbc3babacf 100644 (file)
@@ -178,6 +178,8 @@ worker_mem_report(struct worker* ATTR_UNUSED(worker),
                + sizeof(*worker->env.scratch_buffer) 
                + ldns_buffer_capacity(worker->env.scratch_buffer)
                + forwards_get_mem(worker->env.fwds);
+       if(worker->thread_num == 0)
+               me += acl_list_get_mem(worker->daemon->acl_list);
        if(cur_serv) {
                me += serviced_get_mem(cur_serv);
        }
@@ -1125,7 +1127,6 @@ worker_init(struct worker* worker, struct config_file *cfg,
        if(worker->thread_num == 0)
                log_set_time(worker->env.now);
        worker->env.worker = worker;
-       worker->env.send_packet = &worker_send_packet;
        worker->env.send_query = &worker_send_query;
        worker->env.alloc = &worker->alloc;
        worker->env.rnd = worker->rndstate;
@@ -1214,19 +1215,6 @@ worker_delete(struct worker* worker)
        free(worker);
 }
 
-int 
-worker_send_packet(ldns_buffer* pkt, struct sockaddr_storage* addr,
-        socklen_t addrlen, int timeout, struct module_qstate* q, int use_tcp)
-{
-       struct worker* worker = q->env->worker;
-       if(use_tcp) {
-               return pending_tcp_query(worker->back, pkt, addr, addrlen, 
-                       timeout, worker_handle_reply, q) != 0;
-       }
-       return pending_udp_query(worker->back, pkt, addr, addrlen, 
-               timeout*1000, worker_handle_reply, q) != 0;
-}
-
 /** compare outbound entry qstates */
 static int
 outbound_entry_compare(void* a, void* b)
index a2a234faa6b47a0e9c9118f6ce217e1d1ba88720..8f7066a7c29ddfc76e7673e098ae0d90a1ac08d6 100644 (file)
@@ -164,20 +164,6 @@ void worker_send_cmd(struct worker* worker, enum worker_commands cmd);
  */
 void worker_sighandler(int sig, void* arg);
 
-/**
- * Worker service routine to send udp messages for modules.
- * @param pkt: packet to send.
- * @param addr: where to.
- * @param addrlen: length of addr.
- * @param timeout: seconds to wait until timeout.
- * @param q: wich query state to reactivate upon return.
- * @param use_tcp: true to use TCP, false for UDP.
- * @return: false on failure (memory or socket related). no query was
- *      sent.
- */
-int worker_send_packet(ldns_buffer* pkt, struct sockaddr_storage* addr,
-       socklen_t addrlen, int timeout, struct module_qstate* q, int use_tcp);
-
 /**
  * Worker service routine to send serviced queries to authoritative servers.
  * @param qname: query name. (host order)
index 142387f527108d95b06c7f3698ce932d651b8dcd..df5a52f423292da4b073b61f82a0fbae4983a400 100644 (file)
@@ -1,3 +1,9 @@
+20 August 2010: Wouter
+       - openbsd-lint fixes: acl_list_get_mem used if debug-alloc enabled.
+         iterator get_mem includes priv_get_mem.  delegpt nodup removed.
+         listen_pushback, query_info_allocqname, write_socket, send_packet,
+         comm_point_set_cb_arg and listen_resume removed.
+
 19 August 2010: Wouter
        - Fix bug#321: resolution of rs.ripe.net artifacts with 0x20.
          Delegpt structures checked for duplicates always.
index 33f5bb492f9ee78b86d4ba589a59dfc9bd9b69e7..7d22646f3d526655882df4427d20a0c3bfe728b6 100644 (file)
@@ -81,7 +81,7 @@ struct delegpt* delegpt_copy(struct delegpt* dp, struct regional* region)
        }
        for(a = dp->target_list; a; a = a->next_target) {
                if(!delegpt_add_addr(copy, region, &a->addr, a->addrlen, 
-                       a->bogus, a->lame, 0))
+                       a->bogus, a->lame))
                        return NULL;
        }
        return copy;
@@ -154,7 +154,7 @@ delegpt_find_addr(struct delegpt* dp, struct sockaddr_storage* addr,
 int 
 delegpt_add_target(struct delegpt* dp, struct regional* region, 
        uint8_t* name, size_t namelen, struct sockaddr_storage* addr, 
-       socklen_t addrlen, int bogus, int lame, int nodup)
+       socklen_t addrlen, int bogus, int lame)
 {
        struct delegpt_ns* ns = delegpt_find_ns(dp, name, namelen);
        if(!ns) {
@@ -168,23 +168,22 @@ delegpt_add_target(struct delegpt* dp, struct regional* region,
                if(ns->got4 && ns->got6)
                        ns->resolved = 1;
        }
-       return delegpt_add_addr(dp, region, addr, addrlen, bogus, lame, nodup);
+       return delegpt_add_addr(dp, region, addr, addrlen, bogus, lame);
 }
 
 int 
 delegpt_add_addr(struct delegpt* dp, struct regional* region, 
        struct sockaddr_storage* addr, socklen_t addrlen, int bogus, 
-       int lame, int nodup)
+       int lame)
 {
        struct delegpt_addr* a;
-       if(nodup) {
-               if((a = delegpt_find_addr(dp, addr, addrlen))) {
-                       if(bogus)
-                               a->bogus = bogus;
-                       if(!lame)
-                               a->lame = 0;
-                       return 1;
-               }
+       /* check for duplicates */
+       if((a = delegpt_find_addr(dp, addr, addrlen))) {
+               if(bogus)
+                       a->bogus = bogus;
+               if(!lame)
+                       a->lame = 0;
+               return 1;
        }
 
        a = (struct delegpt_addr*)regional_alloc(region,
@@ -361,10 +360,10 @@ delegpt_from_message(struct dns_msg* msg, struct regional* region)
                        continue;
 
                if(ntohs(s->rk.type) == LDNS_RR_TYPE_A) {
-                       if(!delegpt_add_rrset_A(dp, region, s, 0, 1))
+                       if(!delegpt_add_rrset_A(dp, region, s, 0))
                                return NULL;
                } else if(ntohs(s->rk.type) == LDNS_RR_TYPE_AAAA) {
-                       if(!delegpt_add_rrset_AAAA(dp, region, s, 0, 1))
+                       if(!delegpt_add_rrset_AAAA(dp, region, s, 0))
                                return NULL;
                }
        }
@@ -394,7 +393,7 @@ delegpt_rrset_add_ns(struct delegpt* dp, struct regional* region,
 
 int 
 delegpt_add_rrset_A(struct delegpt* dp, struct regional* region,
-       struct ub_packed_rrset_key* ak, int lame, int nodup)
+       struct ub_packed_rrset_key* ak, int lame)
 {
         struct packed_rrset_data* d=(struct packed_rrset_data*)ak->entry.data;
         size_t i;
@@ -409,7 +408,7 @@ delegpt_add_rrset_A(struct delegpt* dp, struct regional* region,
                 memmove(&sa.sin_addr, d->rr_data[i]+2, INET_SIZE);
                 if(!delegpt_add_target(dp, region, ak->rk.dname,
                         ak->rk.dname_len, (struct sockaddr_storage*)&sa,
-                        len, (d->security==sec_status_bogus), lame, nodup))
+                        len, (d->security==sec_status_bogus), lame))
                         return 0;
         }
         return 1;
@@ -417,7 +416,7 @@ delegpt_add_rrset_A(struct delegpt* dp, struct regional* region,
 
 int 
 delegpt_add_rrset_AAAA(struct delegpt* dp, struct regional* region,
-       struct ub_packed_rrset_key* ak, int lame, int nodup)
+       struct ub_packed_rrset_key* ak, int lame)
 {
         struct packed_rrset_data* d=(struct packed_rrset_data*)ak->entry.data;
         size_t i;
@@ -432,7 +431,7 @@ delegpt_add_rrset_AAAA(struct delegpt* dp, struct regional* region,
                 memmove(&sa.sin6_addr, d->rr_data[i]+2, INET6_SIZE);
                 if(!delegpt_add_target(dp, region, ak->rk.dname,
                         ak->rk.dname_len, (struct sockaddr_storage*)&sa,
-                        len, (d->security==sec_status_bogus), lame, nodup))
+                        len, (d->security==sec_status_bogus), lame))
                         return 0;
         }
         return 1;
@@ -447,9 +446,9 @@ delegpt_add_rrset(struct delegpt* dp, struct regional* region,
        if(ntohs(rrset->rk.type) == LDNS_RR_TYPE_NS)
                return delegpt_rrset_add_ns(dp, region, rrset, lame);
        else if(ntohs(rrset->rk.type) == LDNS_RR_TYPE_A)
-               return delegpt_add_rrset_A(dp, region, rrset, lame, 1);
+               return delegpt_add_rrset_A(dp, region, rrset, lame);
        else if(ntohs(rrset->rk.type) == LDNS_RR_TYPE_AAAA)
-               return delegpt_add_rrset_AAAA(dp, region, rrset, lame, 1);
+               return delegpt_add_rrset_AAAA(dp, region, rrset, lame);
        log_warn("Unknown rrset type added to delegpt");
        return 1;
 }
index 266322fb8eaace2c26b0e648d1532fdd4a9f7fec..d7fef0afb1af85f53dc4411dd4628e972fabcadd 100644 (file)
@@ -201,13 +201,11 @@ int delegpt_rrset_add_ns(struct delegpt* dp, struct regional* regional,
  * @param addrlen: the length of addr.
  * @param bogus: security status for the address, pass true if bogus.
  * @param lame: address is lame.
- * @param nodup: if true, no address duplicates are made by this add. 
- *     name duplicates are always filtered.
  * @return false on error.
  */
 int delegpt_add_target(struct delegpt* dp, struct regional* regional, 
        uint8_t* name, size_t namelen, struct sockaddr_storage* addr, 
-       socklen_t addrlen, int bogus, int lame, int nodup);
+       socklen_t addrlen, int bogus, int lame);
 
 /**
  * Add A RRset to delegpt.
@@ -215,11 +213,10 @@ int delegpt_add_target(struct delegpt* dp, struct regional* regional,
  * @param regional: where to allocate the info.
  * @param rrset: RRset A to add.
  * @param lame: rrset is lame, disprefer it.
- * @param nodup: if true, no duplicates are made by this add. takes time.
  * @return 0 on alloc error.
  */
 int delegpt_add_rrset_A(struct delegpt* dp, struct regional* regional, 
-       struct ub_packed_rrset_key* rrset, int lame, int nodup);
+       struct ub_packed_rrset_key* rrset, int lame);
 
 /**
  * Add AAAA RRset to delegpt.
@@ -227,11 +224,10 @@ int delegpt_add_rrset_A(struct delegpt* dp, struct regional* regional,
  * @param regional: where to allocate the info.
  * @param rrset: RRset AAAA to add.
  * @param lame: rrset is lame, disprefer it.
- * @param nodup: if true, no duplicates are made by this add. takes time.
  * @return 0 on alloc error.
  */
 int delegpt_add_rrset_AAAA(struct delegpt* dp, struct regional* regional, 
-       struct ub_packed_rrset_key* rrset, int lame, int nodup);
+       struct ub_packed_rrset_key* rrset, int lame);
 
 /**
  * Add any RRset to delegpt.
@@ -253,12 +249,10 @@ int delegpt_add_rrset(struct delegpt* dp, struct regional* regional,
  * @param addrlen: the length of addr.
  * @param bogus: if address is bogus.
  * @param lame: if address is lame.
- * @param nodup: if true, no duplicates are made by this add. takes time.
  * @return false on error.
  */
 int delegpt_add_addr(struct delegpt* dp, struct regional* regional, 
-       struct sockaddr_storage* addr, socklen_t addrlen, int bogus, 
-       int lame, int nodup);
+       struct sockaddr_storage* addr, socklen_t addrlen, int bogus, int lame);
 
 /** 
  * Find NS record in name list of delegation point.
index 4d3ff16a0c5c0d98b21d564805bfbb701ad6421f..8e655b97ab043820d88bbe96b60c143228b009c4 100644 (file)
@@ -215,8 +215,7 @@ read_fwds_addr(struct iter_forwards* fwd, struct config_stub* s,
                                s->name, p->str);
                        return 0;
                }
-               if(!delegpt_add_addr(dp, fwd->region, &addr, addrlen, 
-                       0, 0, 1)) {
+               if(!delegpt_add_addr(dp, fwd->region, &addr, addrlen, 0, 0)) {
                        log_err("out of memory");
                        return 0;
                }
index d896d680f90d54a45be186a26aa985e501993ade..ee2cb3118bf4de9333d40e77b8a2cc117f344402 100644 (file)
@@ -88,7 +88,7 @@ ah(struct delegpt* dp, struct regional* r, const char* sv, const char* ip)
        if(!delegpt_add_ns(dp, r, ldns_rdf_data(rdf), 0) ||
           !extstrtoaddr(ip, &addr, &addrlen) ||
           !delegpt_add_target(dp, r, ldns_rdf_data(rdf), ldns_rdf_size(rdf),
-               &addr, addrlen, 0, 0, 1)) {
+               &addr, addrlen, 0, 0)) {
                ldns_rdf_deep_free(rdf);
                return 0;
        }
@@ -229,8 +229,7 @@ read_stubs_addr(struct iter_hints* hints, struct config_stub* s,
                                s->name, p->str);
                        return 0;
                }
-               if(!delegpt_add_addr(dp, hints->region, &addr, addrlen, 
-                       0, 0, 1)) {
+               if(!delegpt_add_addr(dp, hints->region, &addr, addrlen, 0, 0)) {
                        log_err("out of memory");
                        return 0;
                }
@@ -325,7 +324,7 @@ read_root_hints(struct iter_hints* hints, char* fname)
                                        ldns_rdf_data(ldns_rr_owner(rr)),
                                        ldns_rdf_size(ldns_rr_owner(rr)),
                                        (struct sockaddr_storage*)&sa, len, 
-                                       0, 0, 1)) {
+                                       0, 0)) {
                                log_err("out of memory reading root hints");
                                goto stop_read;
                        }
@@ -341,7 +340,7 @@ read_root_hints(struct iter_hints* hints, char* fname)
                                        ldns_rdf_data(ldns_rr_owner(rr)),
                                        ldns_rdf_size(ldns_rr_owner(rr)),
                                        (struct sockaddr_storage*)&sa, len,
-                                       0, 0, 1)) {
+                                       0, 0)) {
                                log_err("out of memory reading root hints");
                                goto stop_read;
                        }
index dda1745f3a508f1583d2dc585a30d10d80742144..f01aea633c082358b21688af3d99312a123412e5 100644 (file)
@@ -852,7 +852,7 @@ int iter_lookup_parent_glue_from_cache(struct module_env* env,
                        log_rrset_key(VERB_ALGO, "found parent-side", akey);
                        ns->done_pside4 = 1;
                        /* a negative-cache-element has no addresses it adds */
-                       if(!delegpt_add_rrset_A(dp, region, akey, 1, 1))
+                       if(!delegpt_add_rrset_A(dp, region, akey, 1))
                                log_err("malloc failure in lookup_parent_glue");
                        lock_rw_unlock(&akey->entry.lock);
                }
@@ -864,7 +864,7 @@ int iter_lookup_parent_glue_from_cache(struct module_env* env,
                        log_rrset_key(VERB_ALGO, "found parent-side", akey);
                        ns->done_pside6 = 1;
                        /* a negative-cache-element has no addresses it adds */
-                       if(!delegpt_add_rrset_AAAA(dp, region, akey, 1, 1))
+                       if(!delegpt_add_rrset_AAAA(dp, region, akey, 1))
                                log_err("malloc failure in lookup_parent_glue");
                        lock_rw_unlock(&akey->entry.lock);
                }
index f170e78b452343c6dad6df532d8f5415c5eaecb5..d52cfea05a306886e6d5ea239fca454c6dae3be1 100644 (file)
@@ -2663,7 +2663,8 @@ iter_get_mem(struct module_env* env, int id)
        if(!ie)
                return 0;
        return sizeof(*ie) + sizeof(int)*((size_t)ie->max_dependency_depth+1)
-               + hints_get_mem(ie->hints) + donotq_get_mem(ie->donotq);
+               + hints_get_mem(ie->hints) + donotq_get_mem(ie->donotq)
+               + priv_get_mem(ie->priv);
 }
 
 /**
index 0c6f5ddc82b7617b08bd7c2bfc7f8b2c9c941ef8..489df698a197f8eda1af9b2bb31299072ea9ec6e 100644 (file)
@@ -185,7 +185,6 @@ libworker_setup(struct ub_ctx* ctx, int is_bg)
                libworker_delete(w);
                return NULL;
        }
-       w->env->send_packet = &libworker_send_packet;
        w->env->send_query = &libworker_send_query;
        w->env->detach_subs = &mesh_detach_subs;
        w->env->attach_sub = &mesh_attach_sub;
@@ -671,18 +670,6 @@ void libworker_alloc_cleanup(void* arg)
         slabhash_clear(w->env->msg_cache);
 }
 
-int libworker_send_packet(ldns_buffer* pkt, struct sockaddr_storage* addr,
-        socklen_t addrlen, int timeout, struct module_qstate* q, int use_tcp)
-{
-       struct libworker* w = (struct libworker*)q->env->worker;
-       if(use_tcp) {
-               return pending_tcp_query(w->back, pkt, addr, addrlen,
-                       timeout, libworker_handle_reply, q) != 0;
-       }
-       return pending_udp_query(w->back, pkt, addr, addrlen,
-               timeout*1000, libworker_handle_reply, q) != 0;
-}
-
 /** compare outbound entry qstates */
 static int
 outbound_entry_compare(void* a, void* b)
index 192094d8d72f071163dabc857b84551e332a68f8..7aa35126ab489deb7bf5b2520cc93401d1db7f9c 100644 (file)
@@ -107,20 +107,6 @@ int libworker_fg(struct ub_ctx* ctx, struct ctx_query* q);
 /** cleanup the cache to remove all rrset IDs from it, arg is libworker */
 void libworker_alloc_cleanup(void* arg);
 
-/**
- * Worker service routine to send udp messages for modules.
- * @param pkt: packet to send.
- * @param addr: where to.
- * @param addrlen: length of addr.
- * @param timeout: seconds to wait until timeout.
- * @param q: wich query state to reactivate upon return.
- * @param use_tcp: true to use TCP, false for UDP.
- * @return: false on failure (memory or socket related). no query was
- *      sent.
- */
-int libworker_send_packet(ldns_buffer* pkt, struct sockaddr_storage* addr,
-        socklen_t addrlen, int timeout, struct module_qstate* q, int use_tcp);
-
 /**
  * Worker service routine to send serviced queries to authoritative servers.
  * @param qname: query name. (host order)
index c5984d01d13c5dca56f50a27a840af7d6df07073..3270982efeb0e0c5cd094c48a1269b8e52de662f 100644 (file)
@@ -180,7 +180,7 @@ find_add_addrs(struct module_env* env, uint16_t qclass,
                akey = rrset_cache_lookup(env->rrset_cache, ns->name, 
                        ns->namelen, LDNS_RR_TYPE_A, qclass, 0, now, 0);
                if(akey) {
-                       if(!delegpt_add_rrset_A(dp, region, akey, 0, 1)) {
+                       if(!delegpt_add_rrset_A(dp, region, akey, 0)) {
                                lock_rw_unlock(&akey->entry.lock);
                                return 0;
                        }
@@ -198,7 +198,7 @@ find_add_addrs(struct module_env* env, uint16_t qclass,
                akey = rrset_cache_lookup(env->rrset_cache, ns->name, 
                        ns->namelen, LDNS_RR_TYPE_AAAA, qclass, 0, now, 0);
                if(akey) {
-                       if(!delegpt_add_rrset_AAAA(dp, region, akey, 0, 1)) {
+                       if(!delegpt_add_rrset_AAAA(dp, region, akey, 0)) {
                                lock_rw_unlock(&akey->entry.lock);
                                return 0;
                        }
@@ -230,7 +230,7 @@ cache_fill_missing(struct module_env* env, uint16_t qclass,
                akey = rrset_cache_lookup(env->rrset_cache, ns->name, 
                        ns->namelen, LDNS_RR_TYPE_A, qclass, 0, now, 0);
                if(akey) {
-                       if(!delegpt_add_rrset_A(dp, region, akey, (int)ns->lame, 1)) {
+                       if(!delegpt_add_rrset_A(dp, region, akey, (int)ns->lame)) {
                                lock_rw_unlock(&akey->entry.lock);
                                return 0;
                        }
@@ -248,7 +248,7 @@ cache_fill_missing(struct module_env* env, uint16_t qclass,
                akey = rrset_cache_lookup(env->rrset_cache, ns->name, 
                        ns->namelen, LDNS_RR_TYPE_AAAA, qclass, 0, now, 0);
                if(akey) {
-                       if(!delegpt_add_rrset_AAAA(dp, region, akey, (int)ns->lame, 1)) {
+                       if(!delegpt_add_rrset_AAAA(dp, region, akey, (int)ns->lame)) {
                                lock_rw_unlock(&akey->entry.lock);
                                return 0;
                        }
index da033b9a3c1fab8ed8c7ebb9fa7e00b33ca89c76..49cbeed983f432bf62c2c336cf78438e6fba8653 100644 (file)
@@ -715,32 +715,6 @@ listen_delete(struct listen_dnsport* front)
        free(front);
 }
 
-void listen_pushback(struct listen_dnsport* listen)
-{
-       struct listen_list *p;
-       log_assert(listen);
-       for(p = listen->cps; p; p = p->next)
-       {
-               if(p->com->type != comm_udp &&
-                       p->com->type != comm_tcp_accept)
-                       continue;
-               comm_point_stop_listening(p->com);
-       }
-}
-
-void listen_resume(struct listen_dnsport* listen)
-{
-       struct listen_list *p;
-       log_assert(listen);
-       for(p = listen->cps; p; p = p->next)
-       {
-               if(p->com->type != comm_udp &&
-                       p->com->type != comm_tcp_accept)
-                       continue;
-               comm_point_start_listening(p->com, -1, -1);
-       }
-}
-
 struct listen_port* 
 listening_ports_open(struct config_file* cfg)
 {
index 01de5e9d51286c3feef12d818ff5e8fc1e3f227b..728a562531061c7506e004567e6e2e6d044574ac 100644 (file)
@@ -130,25 +130,6 @@ struct listen_dnsport* listen_create(struct comm_base* base,
        struct listen_port* ports, size_t bufsize, int tcp_accept_count,
        comm_point_callback_t* cb, void* cb_arg);
 
-/**
- * Stop listening to the dnsports. Ports are still open but not checked
- * for readability - performs pushback of the load.
- * @param listen: the listening structs to stop listening on. Note that
- *    udp and tcp-accept handlers stop, but ongoing tcp-handlers are kept
- *    going, since its rude to 'reset connection by peer' them, instead,
- *    we keep them and the callback will be called when its ready. It can
- *    be dropped at that time. New tcp and udp queries can be served by 
- *    other threads.
- */
-void listen_pushback(struct listen_dnsport* listen);
-
-/**
- * Start listening again to the dnsports.
- * Call after the listen_pushback has been called.
- * @param listen: the listening structs to stop listening on.
- */
-void listen_resume(struct listen_dnsport* listen);
-
 /**
  * delete the listening structure
  * @param listen: listening structure.
index 35fdf450789b20dcbf0017dd11ef355792887d97..e59dd1cf99fad343e20d33e796358250cf312e8b 100644 (file)
@@ -1119,14 +1119,6 @@ void listening_ports_free(struct listen_port* list)
        free(list);
 }
 
-void listen_pushback(struct listen_dnsport* ATTR_UNUSED(listen))
-{
-}
-
-void listen_resume(struct listen_dnsport* ATTR_UNUSED(listen))
-{
-}
-
 struct comm_point* comm_point_create_local(struct comm_base* ATTR_UNUSED(base),
         int ATTR_UNUSED(fd), size_t ATTR_UNUSED(bufsize),
         comm_point_callback_t* ATTR_UNUSED(callback), 
index 22e146884cfc85b36ee5192ef6868ea24b171bfd..d0a4c3855051da1ef3b6235d2eb1924f0a9fe6ef 100644 (file)
@@ -516,18 +516,6 @@ query_info_parse(struct query_info* m, ldns_buffer* query)
        return 1;
 }
 
-int 
-query_info_allocqname(struct query_info* m)
-{
-       uint8_t* q = m->qname;
-       if(!(m->qname = (uint8_t*)malloc(m->qname_len))) {
-               log_err("query_info_allocqname: out of memory");
-               return 0; /* out of memory */
-       }
-       memcpy(m->qname, q, m->qname_len);
-       return 1;
-}
-
 /** tiny subroutine for msgreply_compare */
 #define COMPARE_IT(x, y) \
        if( (x) < (y) ) return -1; \
index a8d59181f5f9ac1571607675a4675a3841d809a5..a32f2b1edab49867600dbd27b518f85d0bfd0191 100644 (file)
@@ -263,13 +263,6 @@ void reply_info_set_ttls(struct reply_info* rep, uint32_t timenow);
  */
 void reply_info_parsedelete(struct reply_info* rep, struct alloc_cache* alloc);
 
-/**
- * Allocate and copy the qname (obtained from query_info_parse()).
- * @param m: the queryinfo structure.
- * @return: 0 on alloc failure.
- */
-int query_info_allocqname(struct query_info* m);
-
 /**
  * Compare two queryinfo structures, on query and type, class. 
  * It is _not_ sorted in canonical ordering.
index b247a6b7e5ff21100dbd6d780cd246b61fc1aa56..dd54b3145b905a182d550135e1a403a8ecc3d56c 100644 (file)
@@ -245,16 +245,6 @@ fptr_whitelist_hash_markdelfunc(lruhash_markdelfunc_t fptr)
        return 0;
 }
 
-int 
-fptr_whitelist_modenv_send_packet(int (*fptr)(ldns_buffer* pkt,
-        struct sockaddr_storage* addr, socklen_t addrlen, int timeout,
-        struct module_qstate* q, int use_tcp))
-{
-       if(fptr == &worker_send_packet) return 1;
-       else if(fptr == &libworker_send_packet) return 1;
-       return 0;
-}
-
 /** whitelist env->send_query callbacks */
 int 
 fptr_whitelist_modenv_send_query(struct outbound_entry* (*fptr)(
index 7301cbf3ae7895d2cbc297d386d3d03417b72c29..1b68627661e61535a3991f9ebdbe1ef9c108553d 100644 (file)
@@ -183,16 +183,6 @@ int fptr_whitelist_hash_deldatafunc(lruhash_deldatafunc_t fptr);
  */
 int fptr_whitelist_hash_markdelfunc(lruhash_markdelfunc_t fptr);
 
-/**
- * Check function pointer whitelist for module_env send_packet callback values.
- *
- * @param fptr: function pointer to check.
- * @return false if not in whitelist.
- */
-int fptr_whitelist_modenv_send_packet(int (*fptr)(ldns_buffer* pkt, 
-       struct sockaddr_storage* addr, socklen_t addrlen, int timeout, 
-       struct module_qstate* q, int use_tcp));
-
 /**
  * Check function pointer whitelist for module_env send_query callback values.
  *
index 89dfffd85e99ae80881508a02a74e9495806b2f8..3dac8b917fa807c0caf7f106f3210d0164b4f76c 100644 (file)
@@ -82,24 +82,6 @@ struct module_env {
        struct key_cache* key_cache;
 
        /* --- services --- */
-       /** 
-        * Direct access to the network, this packet gets sent to destination.
-        * Send DNS query to server. operate() should return with wait_reply.
-        * Later on a callback will cause operate() to be called with event
-        * timeout or reply. Replied packet is then in the query buffer.
-        * @param pkt: packet to send.
-        * @param addr: where to.
-        * @param addrlen: length of addr.
-        * @param timeout: seconds to wait until timeout.
-        * @param q: wich query state to reactivate upon return.
-        * @param use_tcp: set to true to send over TCP. 0 for UDP.
-        * @return: false on failure (memory or socket related). no query was
-        *      sent.
-        */
-       int (*send_packet)(ldns_buffer* pkt, struct sockaddr_storage* addr,
-               socklen_t addrlen, int timeout, struct module_qstate* q,
-               int use_tcp);
-
        /** 
         * Send serviced DNS query to server. UDP/TCP and EDNS is handled.
         * operate() should return with wait_reply. Later on a callback 
index 7b2a3f4009e79cbbf44f82f63ae250a3010ccf60..537577658fa36f29c112f7a22b78c015300eee16 100644 (file)
@@ -60,30 +60,6 @@ str_is_ip6(const char* str)
        else    return 0;
 }
 
-int 
-write_socket(int s, const void *buf, size_t size)
-{
-       const char* data = (const char*)buf;
-       size_t total_count = 0;
-
-       fd_set_block(s);
-       while (total_count < size) {
-               ssize_t count
-                       = write(s, data + total_count, size - total_count);
-               if (count == -1) {
-                       if (errno != EAGAIN && errno != EINTR) {
-                               fd_set_nonblock(s);
-                               return 0;
-                       } else {
-                               continue;
-                       }
-               }
-               total_count += count;
-       }
-       fd_set_nonblock(s);
-       return 1;
-}
-
 int 
 fd_set_nonblock(int s) 
 {
index b0d2ed499abdcab02a51c254083ce63e9fdc9eb2..142c909d600a7efd9c063838ae1f93d8c8d0bf87 100644 (file)
@@ -100,16 +100,6 @@ extern uint16_t EDNS_ADVERTISED_SIZE;
  */
 int str_is_ip6(const char* str);
 
-/**
- * Write (blocking) to a nonblocking socket.
- * @param s: fd. Is set to be nonblocking at exit.
- * @param buf: data buffer.
- * @param size: length of data to send.
- * @return: 0 on error. errno is set.
- */
-int
-write_socket(int s, const void *buf, size_t size);
-
 /**
  * Set fd nonblocking.
  * @param s: file descriptor.
index 8b479bfb1aa90b75a335a83b75c9872da6fd6ca9..a2dc1d786c6fd50787ed6dd67d911bcf6ebb86b5 100644 (file)
@@ -1489,13 +1489,6 @@ comm_point_delete(struct comm_point* c)
        free(c);
 }
 
-void 
-comm_point_set_cb_arg(struct comm_point* c, void *arg)
-{
-       log_assert(c);
-       c->cb_arg = arg;
-}
-
 void 
 comm_point_send_reply(struct comm_reply *repinfo)
 {
index 25ef6d15057d0490870a6324799f2335a310945a..de57ee36c69b5b152d9a37a754afb06c3d9fae83 100644 (file)
@@ -403,13 +403,6 @@ void comm_point_close(struct comm_point* c);
  */
 void comm_point_delete(struct comm_point* c);
 
-/**
- * Reset the callback argument for a comm point.
- * @param c: the comm point to change.
- * @param arg: the new callback user argument.
- */
-void comm_point_set_cb_arg(struct comm_point* c, void* arg);
-
 /**
  * Send reply. Put message into commpoint buffer.
  * @param repinfo: The reply info copied from a commpoint callback call.