Better fix for 997.
This commit is contained in:
parent
2378e37a5d
commit
2bcf2cb099
|
@ -22,6 +22,11 @@ Changes in version 0.2.0.35 - 2009-??-??
|
|||
- When starting with a cache over a few days old, do not leak
|
||||
memory for the obsolete router descriptors in it. Bugfix on
|
||||
0.2.0.33; fixes bug 672.
|
||||
- Hidden service clients didn't use a cached service descriptor that
|
||||
was older than 15 minutes, but wouldn't fetch a new one either,
|
||||
because there was already one in the cache. Now, fetch a v2
|
||||
descriptor unless the same descriptor was added to the cache within
|
||||
the last 15 minutes. Fixes bug 997; reported by Marcus Griep.
|
||||
|
||||
|
||||
Changes in version 0.2.0.34 - 2009-02-08
|
||||
|
|
|
@ -1610,9 +1610,6 @@ connection_ap_handshake_rewrite_and_attach(edge_connection_t *conn,
|
|||
rend_client_refetch_v2_renddesc(conn->rend_query);
|
||||
rend_client_refetch_renddesc(conn->rend_query);
|
||||
} else { /* r > 0 */
|
||||
/** How long after we receive a hidden service descriptor do we consider
|
||||
* it valid? */
|
||||
#define NUM_SECONDS_BEFORE_HS_REFETCH (60*15)
|
||||
if (now - entry->received < NUM_SECONDS_BEFORE_HS_REFETCH) {
|
||||
conn->_base.state = AP_CONN_STATE_CIRCUIT_WAIT;
|
||||
log_info(LD_REND, "Descriptor is here and fresh enough. Great.");
|
||||
|
|
|
@ -613,6 +613,10 @@ typedef enum {
|
|||
/** Length of a binary-encoded rendezvous service ID. */
|
||||
#define REND_SERVICE_ID_LEN 10
|
||||
|
||||
/** How long after we receive a hidden service descriptor do we consider
|
||||
* it fresh? */
|
||||
#define NUM_SECONDS_BEFORE_HS_REFETCH (60*15)
|
||||
|
||||
/** Time period for which a v2 descriptor will be valid. */
|
||||
#define REND_TIME_PERIOD_V2_DESC_VALIDITY (24*60*60)
|
||||
|
||||
|
|
|
@ -421,8 +421,9 @@ rend_client_refetch_v2_renddesc(const char *query)
|
|||
{
|
||||
char descriptor_id[DIGEST_LEN];
|
||||
int replicas_left_to_try[REND_NUMBER_OF_NON_CONSECUTIVE_REPLICAS];
|
||||
int i, tries_left;
|
||||
int i, tries_left, r;
|
||||
rend_cache_entry_t *e = NULL;
|
||||
time_t now = time(NULL);
|
||||
tor_assert(query);
|
||||
tor_assert(strlen(query) == REND_SERVICE_ID_LEN_BASE32);
|
||||
/* Are we configured to fetch descriptors? */
|
||||
|
@ -432,9 +433,11 @@ rend_client_refetch_v2_renddesc(const char *query)
|
|||
return;
|
||||
}
|
||||
/* Before fetching, check if we already have the descriptor here. */
|
||||
if (rend_cache_lookup_entry(query, -1, &e) > 0) {
|
||||
r = rend_cache_lookup_entry(query, -1, &e);
|
||||
if (r > 0 && now - e->received < NUM_SECONDS_BEFORE_HS_REFETCH) {
|
||||
log_info(LD_REND, "We would fetch a v2 rendezvous descriptor, but we "
|
||||
"already have that descriptor here. Not fetching.");
|
||||
"already have a fresh copy of that descriptor here. "
|
||||
"Not fetching.");
|
||||
return;
|
||||
}
|
||||
log_debug(LD_REND, "Fetching v2 rendezvous descriptor for service %s",
|
||||
|
|
Loading…
Reference in New Issue