|
@@ -864,7 +864,8 @@ networkstatus_copy_old_consensus_info(networkstatus_vote_t *new_c,
|
|
|
SMARTLIST_FOREACH(new_c->routerstatus_list, routerstatus_t *, rs_new,
|
|
|
{
|
|
|
int r;
|
|
|
- while ((r = memcmp(rs_old->identity_digest, rs_new->identity_digest,
|
|
|
+ while (rs_old &&
|
|
|
+ (r = memcmp(rs_old->identity_digest, rs_new->identity_digest,
|
|
|
DIGEST_LEN))<0) {
|
|
|
if (idx == smartlist_len(old_c->routerstatus_list))
|
|
|
goto done;
|
|
@@ -872,6 +873,9 @@ networkstatus_copy_old_consensus_info(networkstatus_vote_t *new_c,
|
|
|
}
|
|
|
if (r>0)
|
|
|
continue;
|
|
|
+ if (r<0)
|
|
|
+ * match your comment below. */
|
|
|
+ continue;
|
|
|
|
|
|
rs_new->name_lookup_warned = rs_old->name_lookup_warned;
|
|
|
rs_new->last_dir_503_at = rs_old->last_dir_503_at;
|