cancel_descriptor_fetches() uses wrong connection list function

cancel_descriptor_fetches() does:

  smartlist_t *conns =
    connection_list_by_type_state(CONN_TYPE_DIR, DIR_PURPOSE_FETCH_HSDESC);

when it should be using connection_list_by_type_purpose().

changed milestone to %Tor: 0.4.3.x-final

added backport? component::core tor/tor milestone::Tor: 0.4.3.x-final newnym owner::neel priority::medium resolution::fixed reviewer::asn severity::normal sponsor::27-can status::closed tor-hs type::defect labels

Trac:
Keywords: tor-hs newnym deleted, tor-hs newnym backport? added

Trac:
Owner: N/A to neel
Status: new to assigned

PR: https://github.com/torproject/tor/pull/1587

Trac:
Cc: gaba to gaba, neel

Trac:
Status: assigned to needs_review

The commit message for this ticket is wrong, it says cancel_descriptor_fetches() twice.

Trac:
Status: needs_review to needs_revision

Fixed it. Did a force push as there were no comments on the branch.

Trac:
Status: needs_revision to needs_review

Trac:
Reviewer: N/A to asn

The changes file has the same issue tho.

Trac:
Status: needs_review to needs_revision

Fixed it.

Trac:
Status: needs_revision to needs_review

LGTM! Merged! Sorry for the delay!

Trac:
Resolution: N/A to fixed
Status: needs_review to closed

closed

moved to tpo/core/tor#32639 (closed)