[tor-bugs] #9958 [Tor]: We have two different places that calculate bootstrap progress

[Tor Bug Tracker & Wiki]

#9958: We have two different places that calculate bootstrap progress
--------------------+------------------------------------
 Reporter:  arma    |          Owner:
     Type:  defect  |         Status:  new
 Priority:  normal  |      Milestone:  Tor: 0.2.5.x-final
Component:  Tor     |        Version:
 Keywords:          |  Actual Points:
Parent ID:          |         Points:
--------------------+------------------------------------
 count_loading_descriptors_progress() calculates whether we've got enough
 descriptors using:
 {{{
   count_usable_descriptors(&num_present, &num_usable, NULL,
                            consensus, get_options(), now, NULL, 0);

   fraction = num_present / (num_usable/4.);
 }}}

 whereas update_router_have_minimum_dir_info() does it using
 {{{
     double paths = compute_frac_paths_available(consensus, options, now,
                                                 &num_present, &num_usable,
                                                 &status);

     if (paths < get_frac_paths_needed_for_circs(options,consensus)) {
 }}}

 We should do the second one in both places. (I have a patch.)

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/9958>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online