dns.c 17.9 KB
Newer Older
1
2
3
4
/* Copyright 2003 Roger Dingledine. */
/* See LICENSE for licensing information */
/* $Id$ */

5
6
7
8
9
/* See http://elvin.dstc.com/ListArchive/elvin-dev/archive/2001/09/msg00027.html
 * for some approaches to asynchronous dns. We will want to switch once one of
 * them becomes more commonly available.
 */

10
#include "or.h"
11
#include "tree.h"
12

13
14
#define MAX_ADDRESSLEN 256

15
16
#define MAX_DNSWORKERS 50
#define MIN_DNSWORKERS 3
17
#define MAX_IDLE_DNSWORKERS 10
18

19
20
21
22
#define DNS_RESOLVE_FAILED_TRANSIENT 1
#define DNS_RESOLVE_FAILED_PERMANENT 2
#define DNS_RESOLVE_SUCCEEDED 3

Roger Dingledine's avatar
Roger Dingledine committed
23
24
int num_dnsworkers=0;
int num_dnsworkers_busy=0;
25
26
27
28
29
30
31
32

struct pending_connection_t {
  struct connection_t *conn;
  struct pending_connection_t *next;
};

struct cached_resolve {
  SPLAY_ENTRY(cached_resolve) node;
33
34
  char address[MAX_ADDRESSLEN]; /* the hostname to be resolved */
  uint32_t addr; /* in host order. I know I'm horrible for assuming ipv4 */
35
36
37
38
39
40
41
42
43
  char state; /* 0 is pending; 1 means answer is valid; 2 means resolve failed */
#define CACHE_STATE_PENDING 0
#define CACHE_STATE_VALID 1
#define CACHE_STATE_FAILED 2
  uint32_t expire; /* remove untouched items from cache after some time? */
  struct pending_connection_t *pending_connections;
  struct cached_resolve *next;
};

44
45
46
47
48
49
50
51
static void purge_expired_resolves(uint32_t now);
static int assign_to_dnsworker(connection_t *exitconn);
static void dns_purge_resolve(struct cached_resolve *resolve);
static void dns_found_answer(char *address, uint32_t addr, char outcome);
int dnsworker_main(void *data);
static int spawn_dnsworker(void);
static void spawn_enough_dnsworkers(void);

52
static SPLAY_HEAD(cache_tree, cached_resolve) cache_root;
53

54
55
static int compare_cached_resolves(struct cached_resolve *a,
                                   struct cached_resolve *b) {
56
  /* make this smarter one day? */
57
  return strncasecmp(a->address, b->address, MAX_ADDRESSLEN);
58
59
60
61
62
}

SPLAY_PROTOTYPE(cache_tree, cached_resolve, node, compare_cached_resolves);
SPLAY_GENERATE(cache_tree, cached_resolve, node, compare_cached_resolves);

63
static void init_cache_tree(void) {
64
65
66
  SPLAY_INIT(&cache_root);
}

67
68
void dns_init(void) {
  init_cache_tree();
Roger Dingledine's avatar
Roger Dingledine committed
69
  spawn_enough_dnsworkers();
70
}
71

72
73
74
static struct cached_resolve *oldest_cached_resolve = NULL; /* linked list, */
static struct cached_resolve *newest_cached_resolve = NULL; /* oldest to newest */

75
76
77
78
79
80
81
82
static void purge_expired_resolves(uint32_t now) {
  struct cached_resolve *resolve;

  /* this is fast because the linked list
   * oldest_cached_resolve is ordered by when they came in.
   */
  while(oldest_cached_resolve && (oldest_cached_resolve->expire < now)) {
    resolve = oldest_cached_resolve;
83
    log(LOG_DEBUG,"Forgetting old cached resolve (expires %lu)", (unsigned long)resolve->expire);
84
85
86
87
88
89
    if(resolve->state == CACHE_STATE_PENDING) {
      log_fn(LOG_WARN,"Expiring a dns resolve that's still pending. Forgot to cull it?");
      /* XXX if resolve->pending_connections is used, then we're probably
       * introducing bugs by closing resolve without notifying those streams.
       */
    }
90
91
92
93
    oldest_cached_resolve = resolve->next;
    if(!oldest_cached_resolve) /* if there are no more, */
      newest_cached_resolve = NULL; /* then make sure the list's tail knows that too */
    SPLAY_REMOVE(cache_tree, &cache_root, resolve);
94
    tor_free(resolve);
95
96
97
  }
}

98
/* See if we have a cache entry for 'exitconn->address'. if so,
99
100
 * if resolve valid, put it into exitconn->addr and return 1.
 * If resolve failed, return -1.
101
102
103
104
105
106
107
108
109
 *
 * Else, if seen before and pending, add conn to the pending list,
 * and return 0.
 *
 * Else, if not seen before, add conn to pending list, hand to
 * dns farm, and return 0.
 */
int dns_resolve(connection_t *exitconn) {
  struct cached_resolve *resolve;
Roger Dingledine's avatar
   
Roger Dingledine committed
110
  struct cached_resolve search;
111
  struct pending_connection_t *pending_connection;
112
  uint32_t now = time(NULL);
113
  assert_connection_ok(exitconn, 0);
114

115
  /* first take this opportunity to see if there are any expired
116
117
     resolves in the tree.*/
  purge_expired_resolves(now);
118

119
120
  /* now check the tree to see if 'address' is already there. */
  strncpy(search.address, exitconn->address, MAX_ADDRESSLEN);
121
  search.address[MAX_ADDRESSLEN-1] = 0;
Roger Dingledine's avatar
   
Roger Dingledine committed
122
  resolve = SPLAY_FIND(cache_tree, &cache_root, &search);
123
  if(resolve) { /* already there */
124
125
126
    switch(resolve->state) {
      case CACHE_STATE_PENDING:
        /* add us to the pending list */
127
        pending_connection = tor_malloc(sizeof(struct pending_connection_t));
128
        pending_connection->conn = exitconn;
Roger Dingledine's avatar
   
Roger Dingledine committed
129
130
        pending_connection->next = resolve->pending_connections;
        resolve->pending_connections = pending_connection;
131
132
        log_fn(LOG_DEBUG,"Connection (fd %d) waiting for pending DNS resolve of '%s'",
               exitconn->s, exitconn->address);
Roger Dingledine's avatar
   
Roger Dingledine committed
133
        return 0;
134
      case CACHE_STATE_VALID:
135
        exitconn->addr = resolve->addr;
136
137
        log_fn(LOG_DEBUG,"Connection (fd %d) found cached answer for '%s'",
               exitconn->s, exitconn->address);
138
        return 1;
139
140
141
      case CACHE_STATE_FAILED:
        return -1;
    }
Roger Dingledine's avatar
Roger Dingledine committed
142
143
144
145
146
    assert(0);
  }
  /* not there, need to add it */
  resolve = tor_malloc_zero(sizeof(struct cached_resolve));
  resolve->state = CACHE_STATE_PENDING;
147
148
  resolve->expire = now + MAX_DNS_ENTRY_AGE;
  strncpy(resolve->address, exitconn->address, MAX_ADDRESSLEN);
149
  resolve->address[MAX_ADDRESSLEN-1] = 0;
Roger Dingledine's avatar
Roger Dingledine committed
150
151
152
153

  /* add us to the pending list */
  pending_connection = tor_malloc(sizeof(struct pending_connection_t));
  pending_connection->conn = exitconn;
154
  pending_connection->next = NULL;
Roger Dingledine's avatar
Roger Dingledine committed
155
156
157
158
159
160
161
  resolve->pending_connections = pending_connection;

  /* add us to the linked list of resolves */
  if (!oldest_cached_resolve) {
    oldest_cached_resolve = resolve;
  } else {
    newest_cached_resolve->next = resolve;
162
  }
Roger Dingledine's avatar
Roger Dingledine committed
163
  newest_cached_resolve = resolve;
164

Roger Dingledine's avatar
Roger Dingledine committed
165
166
  SPLAY_INSERT(cache_tree, &cache_root, resolve);
  return assign_to_dnsworker(exitconn);
167
168
}

Roger Dingledine's avatar
Roger Dingledine committed
169
static int assign_to_dnsworker(connection_t *exitconn) {
170
171
  connection_t *dnsconn;
  unsigned char len;
172

Roger Dingledine's avatar
Roger Dingledine committed
173
  spawn_enough_dnsworkers(); /* respawn here, to be sure there are enough */
174

175
  dnsconn = connection_get_by_type_state(CONN_TYPE_DNSWORKER, DNSWORKER_STATE_IDLE);
176
177

  if(!dnsconn) {
Roger Dingledine's avatar
Roger Dingledine committed
178
    log_fn(LOG_WARN,"no idle dns workers. Failing.");
179
    dns_cancel_pending_resolve(exitconn->address);
180
    return -1;
181
182
  }

183
184
185
  log_fn(LOG_DEBUG, "Connection (fd %d) needs to resolve '%s'; assigning to DNSWorker (fd %d)",
         exitconn->s, exitconn->address, dnsconn->s);

186
  tor_free(dnsconn->address);
187
  dnsconn->address = tor_strdup(exitconn->address);
188
  dnsconn->state = DNSWORKER_STATE_BUSY;
Roger Dingledine's avatar
Roger Dingledine committed
189
  num_dnsworkers_busy++;
190

191
  len = strlen(dnsconn->address);
192
193
  connection_write_to_buf(&len, 1, dnsconn);
  connection_write_to_buf(dnsconn->address, len, dnsconn);
194

Roger Dingledine's avatar
Roger Dingledine committed
195
//  log_fn(LOG_DEBUG,"submitted '%s'", exitconn->address);
196
197
198
  return 0;
}

199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
void connection_dns_remove(connection_t *conn)
{
  struct pending_connection_t *pend, *victim;
  struct cached_resolve search;
  struct cached_resolve *resolve;

  strncpy(search.address, conn->address, MAX_ADDRESSLEN);
  search.address[MAX_ADDRESSLEN-1] = 0;

  resolve = SPLAY_FIND(cache_tree, &cache_root, &search);
  if(!resolve) {
    log_fn(LOG_WARN,"Address '%s' is not pending. Dropping.", conn->address);
    return;
  }

  assert(resolve->pending_connections);
  assert_connection_ok(conn,0);

  pend = resolve->pending_connections;

  if(pend->conn == conn) {
    resolve->pending_connections = pend->next;
221
    tor_free(pend);
222
    log_fn(LOG_DEBUG, "First connection (fd %d) no longer waiting for resolve of '%s'",
223
224
225
226
227
228
229
           conn->s, conn->address);
    return;
  } else {
    for( ; pend->next; pend = pend->next) {
      if(pend->next->conn == conn) {
        victim = pend->next;
        pend->next = victim->next;
230
        tor_free(victim);
231
232
233
234
235
236
237
238
239
        log_fn(LOG_DEBUG, "Connection (fd %d) no longer waiting for resolve of '%s'",
               conn->s, conn->address);
        return; /* more are pending */
      }
    }
    assert(0); /* not reachable unless onlyconn not in pending list */
  }
}

240
241
242
243
244
245
246
247
248
249
250
251
252
void assert_connection_edge_not_dns_pending(connection_t *conn) {
  struct pending_connection_t *pend;
  struct cached_resolve *resolve;

  SPLAY_FOREACH(resolve, cache_tree, &cache_root) {
    for(pend = resolve->pending_connections;
        pend;
        pend = pend->next) {
      assert(pend->conn != conn);
    }
  }
}

Roger Dingledine's avatar
Roger Dingledine committed
253
254
/* Cancel all pending connections. Then cancel the resolve itself,
 * and remove the 'struct cached_resolve' from the cache.
255
 */
256
257
void dns_cancel_pending_resolve(char *address) {
  struct pending_connection_t *pend;
258
  struct cached_resolve search;
259
  struct cached_resolve *resolve;
260
  connection_t *pendconn;
261

262
  strncpy(search.address, address, MAX_ADDRESSLEN);
263
  search.address[MAX_ADDRESSLEN-1] = 0;
264
265
266

  resolve = SPLAY_FIND(cache_tree, &cache_root, &search);
  if(!resolve) {
267
    log_fn(LOG_WARN,"Address '%s' is not pending. Dropping.", address);
268
269
270
    return;
  }

271
  assert(resolve->pending_connections);
272

273
274
275
276
  /* mark all pending connections to fail */
  log_fn(LOG_DEBUG, "Failing all connections waiting on DNS resolve of '%s'",
         address);
  while(resolve->pending_connections) {
277
    pend = resolve->pending_connections;
278
279
    /* So that mark_for_close doesn't double-remove the connection. */
    pend->conn->state = EXIT_CONN_STATE_RESOLVEFAILED;
280
281
282
    pendconn = pend->conn; /* don't pass complex things to the
                              connection_mark_for_close macro */
    connection_mark_for_close(pendconn, END_STREAM_REASON_MISC);
283
284
    resolve->pending_connections = pend->next;
    tor_free(pend);
285
286
  }

287
288
289
290
291
292
  dns_purge_resolve(resolve);
}

static void dns_purge_resolve(struct cached_resolve *resolve) {
  struct cached_resolve *tmp;

293
294
295
296
297
  /* remove resolve from the linked list */
  if(resolve == oldest_cached_resolve) {
    oldest_cached_resolve = resolve->next;
    if(oldest_cached_resolve == NULL)
      newest_cached_resolve = NULL;
298
  } else {
299
300
301
302
303
304
305
    /* FFFF make it a doubly linked list if this becomes too slow */
    for(tmp=oldest_cached_resolve; tmp && tmp->next != resolve; tmp=tmp->next) ;
    assert(tmp); /* it's got to be in the list, or we screwed up somewhere else */
    tmp->next = resolve->next; /* unlink it */

    if(newest_cached_resolve == resolve)
      newest_cached_resolve = tmp;
306
  }
307
308
309
310

  /* remove resolve from the tree */
  SPLAY_REMOVE(cache_tree, &cache_root, resolve);

311
  tor_free(resolve);
312
313
}

314
static void dns_found_answer(char *address, uint32_t addr, char outcome) {
315
316
317
  struct pending_connection_t *pend;
  struct cached_resolve search;
  struct cached_resolve *resolve;
318
  connection_t *pendconn;
319

320
  strncpy(search.address, address, MAX_ADDRESSLEN);
321
  search.address[MAX_ADDRESSLEN-1] = 0;
322
323
324

  resolve = SPLAY_FIND(cache_tree, &cache_root, &search);
  if(!resolve) {
325
    log_fn(LOG_INFO,"Resolved unasked address '%s'? Dropping.", address);
326
327
    /* XXX Why drop?  Just because we don't care now doesn't mean we shouldn't
     * XXX cache the result for later. */
328
    return;
329
330
  }

331
  if (resolve->state != CACHE_STATE_PENDING) {
332
333
    log_fn(LOG_WARN, "Resolved '%s' which was already resolved; ignoring",
           address);
334
335
336
337
338
339
340
    return;
  }
  /* Removed this assertion: in fact, we'll sometimes get a double answer
   * to the same question.  This can happen when we ask one worker to resolve
   * X.Y.Z., then we cancel the request, and then we ask another worker to
   * resolve X.Y.Z. */
  /* assert(resolve->state == CACHE_STATE_PENDING); */
341

342
  resolve->addr = ntohl(addr);
343
  if(outcome == DNS_RESOLVE_SUCCEEDED)
344
345
346
347
348
349
    resolve->state = CACHE_STATE_VALID;
  else
    resolve->state = CACHE_STATE_FAILED;

  while(resolve->pending_connections) {
    pend = resolve->pending_connections;
350
    assert_connection_ok(pend->conn,time(NULL));
351
    pend->conn->addr = resolve->addr;
352
    if(resolve->state == CACHE_STATE_FAILED) {
353
354
      /* prevent double-remove */
      pend->conn->state = EXIT_CONN_STATE_RESOLVEFAILED; 
355
356
357
      pendconn = pend->conn; /* don't pass complex things to the
                                connection_mark_for_close macro */
      connection_mark_for_close(pendconn, END_STREAM_REASON_RESOLVEFAILED);
Roger Dingledine's avatar
Roger Dingledine committed
358
    } else {
359
      connection_exit_connect(pend->conn);
Roger Dingledine's avatar
Roger Dingledine committed
360
    }
361
362
    resolve->pending_connections = pend->next;
    tor_free(pend);
363
  }
364
365
366
367

  if(outcome == DNS_RESOLVE_FAILED_TRANSIENT) { /* remove from cache */
    dns_purge_resolve(resolve);
  }
368
369
}

370
371
372
373
374
375
376
377
378
/******************************************************************/

int connection_dns_finished_flushing(connection_t *conn) {
  assert(conn && conn->type == CONN_TYPE_DNSWORKER);
  connection_stop_writing(conn);
  return 0;
}

int connection_dns_process_inbuf(connection_t *conn) {
379
  char success;
380
  uint32_t addr;
381
382
383
384

  assert(conn && conn->type == CONN_TYPE_DNSWORKER);

  if(conn->inbuf_reached_eof) {
385
    log_fn(LOG_WARN,"Read eof. Worker died unexpectedly.");
386
    if(conn->state == DNSWORKER_STATE_BUSY) {
387
      dns_cancel_pending_resolve(conn->address);
Roger Dingledine's avatar
Roger Dingledine committed
388
      num_dnsworkers_busy--;
389
    }
Roger Dingledine's avatar
Roger Dingledine committed
390
    num_dnsworkers--;
391
392
    connection_mark_for_close(conn,0);
    return 0;
393
394
395
  }

  assert(conn->state == DNSWORKER_STATE_BUSY);
396
  if(buf_datalen(conn->inbuf) < 5) /* entire answer available? */
397
    return 0; /* not yet */
398
  assert(buf_datalen(conn->inbuf) == 5);
399

400
401
  connection_fetch_from_buf(&success,1,conn);
  connection_fetch_from_buf((char *)&addr,sizeof(uint32_t),conn);
402

403
404
405
  log_fn(LOG_DEBUG, "DNSWorker (fd %d) returned answer for '%s'",
         conn->s, conn->address);

406
407
408
  assert(success >= DNS_RESOLVE_FAILED_TRANSIENT);
  assert(success <= DNS_RESOLVE_SUCCEEDED);
  dns_found_answer(conn->address, addr, success);
409

410
  tor_free(conn->address);
411
  conn->address = tor_strdup("<idle>");
412
  conn->state = DNSWORKER_STATE_IDLE;
Roger Dingledine's avatar
Roger Dingledine committed
413
  num_dnsworkers_busy--;
414
415
416
417

  return 0;
}

418
int dnsworker_main(void *data) {
419
420
  char address[MAX_ADDRESSLEN];
  unsigned char address_len;
421
  char answer[5];
422
  struct hostent *rent;
423
  int *fdarray = data;
Roger Dingledine's avatar
Roger Dingledine committed
424
  int fd;
425
426
427

  close(fdarray[0]); /* this is the side of the socketpair the parent uses */
  fd = fdarray[1]; /* this side is ours */
428
#ifndef MS_WINDOWS
429
  connection_free_all(); /* so the child doesn't hold the parent's fd's open */
430
#endif
431
432
433

  for(;;) {

434
    if(recv(fd, &address_len, 1, 0) != 1) {
435
      log_fn(LOG_INFO,"dnsworker exiting because tor process died.");
436
      spawn_exit();
437
    }
438
    assert(address_len > 0);
439

440
    if(read_all(fd, address, address_len, 1) != address_len) {
441
      log_fn(LOG_ERR,"read hostname failed. Child exiting.");
442
      spawn_exit();
443
    }
444
    address[address_len] = 0; /* null terminate it */
445

446
    rent = gethostbyname(address);
447
    if (!rent) {
448
449
450
451
452
453
      if(h_errno == TRY_AGAIN) { /* transient error -- don't cache it */
        log_fn(LOG_INFO,"Could not resolve dest addr %s (transient).",address);
        answer[0] = DNS_RESOLVE_FAILED_TRANSIENT;
      } else { /* permanent error, can be cached */
        log_fn(LOG_INFO,"Could not resolve dest addr %s (permanent).",address);
        answer[0] = DNS_RESOLVE_FAILED_PERMANENT;
454
      }
455
      memset(answer+1,0,4);
456
457
    } else {
      assert(rent->h_length == 4); /* break to remind us if we move away from ipv4 */
458
459
      answer[0] = DNS_RESOLVE_SUCCEEDED;
      memcpy(answer+1, rent->h_addr, 4);
460
      log_fn(LOG_INFO,"Resolved address '%s'.",address);
461
    }
462
463
464
465
    if(write_all(fd, answer, 5, 1) != 5) {
      log_fn(LOG_ERR,"writing answer failed. Child exiting.");
      spawn_exit();
    }
466
  }
467
  return 0; /* windows wants this function to return an int */
468
469
}

Roger Dingledine's avatar
Roger Dingledine committed
470
static int spawn_dnsworker(void) {
471
472
473
  int fd[2];
  connection_t *conn;

474
  if(tor_socketpair(AF_UNIX, SOCK_STREAM, 0, fd) < 0) {
475
    log(LOG_ERR, "Couldn't construct socketpair: %s", strerror(errno));
476
477
478
    exit(1);
  }

479
  spawn_func(dnsworker_main, (void*)fd);
Roger Dingledine's avatar
Roger Dingledine committed
480
  log_fn(LOG_DEBUG,"just spawned a worker.");
481
  close(fd[1]); /* we don't need the worker's side of the pipe */
482
483
484

  conn = connection_new(CONN_TYPE_DNSWORKER);

485
  set_socket_nonblocking(fd[0]);
486
487
488

  /* set up conn so it's got all the data we need to remember */
  conn->s = fd[0];
Roger Dingledine's avatar
Roger Dingledine committed
489
  conn->address = tor_strdup("<unused>");
490
491

  if(connection_add(conn) < 0) { /* no space, forget it */
Roger Dingledine's avatar
Roger Dingledine committed
492
    log_fn(LOG_WARN,"connection_add failed. Giving up.");
493
494
495
496
497
498
499
500
501
502
    connection_free(conn); /* this closes fd[0] */
    return -1;
  }

  conn->state = DNSWORKER_STATE_IDLE;
  connection_start_reading(conn);

  return 0; /* success */
}

Roger Dingledine's avatar
Roger Dingledine committed
503
504
static void spawn_enough_dnsworkers(void) {
  int num_dnsworkers_needed; /* aim to have 1 more than needed,
505
                           * but no less than min and no more than max */
506
507
  connection_t *dnsconn;

508
  /* XXX This may not be the best strategy. Maybe we should queue pending
509
510
511
512
513
514
   *     requests until the old ones finish or time out: otherwise, if
   *     the connection requests come fast enough, we never get any DNS done. -NM
   * XXX But if we queue them, then the adversary can pile even more
   *     queries onto us, blocking legitimate requests for even longer.
   *     Maybe we should compromise and only kill if it's been at it for
   *     more than, e.g., 2 seconds. -RD
515
   */
Roger Dingledine's avatar
Roger Dingledine committed
516
  if(num_dnsworkers_busy == MAX_DNSWORKERS) {
517
518
519
    /* We always want at least one worker idle.
     * So find the oldest busy worker and kill it.
     */
520
521
    dnsconn = connection_get_by_type_state_lastwritten(CONN_TYPE_DNSWORKER,
                                                       DNSWORKER_STATE_BUSY);
522
523
    assert(dnsconn);

524
525
    log_fn(LOG_WARN, "%d DNS workers are spawned; all are busy. Killing one.",
           MAX_DNSWORKERS);
526

527
    connection_mark_for_close(dnsconn,0);
Roger Dingledine's avatar
Roger Dingledine committed
528
    num_dnsworkers_busy--;
529
    num_dnsworkers--;
530
  }
531

Roger Dingledine's avatar
Roger Dingledine committed
532
533
  if(num_dnsworkers_busy >= MIN_DNSWORKERS)
    num_dnsworkers_needed = num_dnsworkers_busy+1;
534
  else
Roger Dingledine's avatar
Roger Dingledine committed
535
    num_dnsworkers_needed = MIN_DNSWORKERS;
536

Roger Dingledine's avatar
Roger Dingledine committed
537
538
  while(num_dnsworkers < num_dnsworkers_needed) {
    if(spawn_dnsworker() < 0) {
Roger Dingledine's avatar
Roger Dingledine committed
539
      log(LOG_WARN,"spawn_enough_dnsworkers(): spawn failed!");
540
541
      return;
    }
Roger Dingledine's avatar
Roger Dingledine committed
542
    num_dnsworkers++;
543
544
  }

545
  while(num_dnsworkers > num_dnsworkers_busy+MAX_IDLE_DNSWORKERS) { /* too many idle? */
546
    /* cull excess workers */
547
548
    log_fn(LOG_WARN,"%d of %d dnsworkers are idle. Killing one.",
           num_dnsworkers-num_dnsworkers_needed, num_dnsworkers);
549
550
    dnsconn = connection_get_by_type_state(CONN_TYPE_DNSWORKER, DNSWORKER_STATE_IDLE);
    assert(dnsconn);
551
    connection_mark_for_close(dnsconn,0);
Roger Dingledine's avatar
Roger Dingledine committed
552
    num_dnsworkers--;
553
  }
554
555
}

556
557
558
559
560
561
562
/*
  Local Variables:
  mode:c
  indent-tabs-mode:nil
  c-basic-offset:2
  End:
*/