diff --git a/ChangeLog b/ChangeLog index 773aa8bfec..4d3ab05657 100644 --- a/ChangeLog +++ b/ChangeLog @@ -143,6 +143,8 @@ Changes in version 0.1.2.5-xxxx - 200?-??-?? reported by Mike Perry.) - Do not report bizarre values for results of accounting GETINFOs when the last second's write or read exceeds the alloted bandwidth. + - Report "unrecognized key" rather than an empty string when the + controller tries to fetch a networkstatus that doesn't exist. Changes in version 0.1.2.4-alpha - 2006-12-03 diff --git a/src/or/routerlist.c b/src/or/routerlist.c index fe462e3ec4..92ae7da9e9 100644 --- a/src/or/routerlist.c +++ b/src/or/routerlist.c @@ -1031,7 +1031,7 @@ router_choose_random_node(const char *preferred, smartlist_subtract(sl,excludedsmartlist); routerlist_sl_remove_unreliable_routers(sl, need_uptime, need_capacity, need_guard); - if (need_capacity) /* XXXX012 Is this documented in path spec. -NM */ + if (need_capacity) choice = routerlist_sl_choose_by_bandwidth(sl, weight_for_exit); else choice = smartlist_choose(sl); @@ -4208,10 +4208,6 @@ getinfo_helper_networkstatus(control_connection_t *conn, if (status) { *answer = networkstatus_getinfo_helper_single(&status->status); - } else { - *answer = tor_strdup(""); - /* XXXX012 this should return a 552, not a 250; but handle_getinfo_helper() - * isn't set up to handle that. That should be fixed too. :) -RD */ } return 0; }