| Commit message (Expand) | Author | Age |
* | Merge remote-tracking branch 'origin/maint-0.2.2'•••This merge was a bit nontrivial, since I had to write a new
node_is_a_configured_bridge to parallel router_is_a_configured_bridge.
Conflicts:
src/or/circuitbuild.c
| Nick Mathewson | 2011-06-02 |
|\ |
|
* | | Merge remote-tracking branch 'origin/maint-0.2.2'•••Conflicts:
src/or/circuitbuild.h
| Nick Mathewson | 2011-05-15 |
|\| |
|
| * | Preserve bridge download status across SETCONF, HUP•••This code changes it so that we don't remove bridges immediately when
we start re-parsing our configuration. Instead, we mark them all, and
remove all the marked ones after re-parsing our bridge lines. As we
add a bridge, we see if it's already in the list. If so, we just
unmark it.
This new behavior will lose the property we used to have that bridges
were in bridge_list in the same order in which they appeared in the
torrc. I took a quick look through the code, and I'm pretty sure we
didn't actually depend on that anywhere.
This is for bug 3019; it's a fix on 0.2.0.3-alpha.
| Nick Mathewson | 2011-05-15 |
* | | Merge branch 'maint-0.2.2' | Roger Dingledine | 2011-01-15 |
|\| |
|
| * | Sanity-check consensus param values•••We need to make sure that the worst thing that a weird consensus param
can do to us is to break our Tor (and only if the other Tors are
reliably broken in the same way) so that the majority of directory
authorities can't pull any attacks that are worse than the DoS that
they can trigger by simply shutting down.
One of these worse things was the cbtnummodes parameter, which could
lead to heap corruption on some systems if the value was sufficiently
large.
This commit fixes this particular issue and also introduces sanity
checking for all consensus parameters.
| Sebastian Hahn | 2011-01-15 |
* | | Merge remote branch 'origin/maint-0.2.2' | Nick Mathewson | 2011-01-03 |
|\| |
|
| * | Bump copyright statements to 2011 (0.2.2) | Nick Mathewson | 2011-01-03 |
* | | Merge branch 'maint-0.2.2' | Roger Dingledine | 2010-12-19 |
|\| |
|
| * | Merge remote branch fix_security_bug_021 into fix_security_bug_022•••Conflicts:
src/common/memarea.c
src/or/or.h
src/or/rendclient.c
| Nick Mathewson | 2010-12-15 |
* | | Initial conversion to use node_t throughout our codebase.•••A node_t is an abstraction over routerstatus_t, routerinfo_t, and
microdesc_t. It should try to present a consistent interface to all
of them. There should be a node_t for a server whenever there is
* A routerinfo_t for it in the routerlist
* A routerstatus_t in the current_consensus.
(note that a microdesc_t alone isn't enough to make a node_t exist,
since microdescriptors aren't usable on their own.)
There are three ways to get a node_t right now: looking it up by ID,
looking it up by nickname, and iterating over the whole list of
microdescriptors.
All (or nearly all) functions that are supposed to return "a router"
-- especially those used in building connections and circuits --
should return a node_t, not a routerinfo_t or a routerstatus_t.
A node_t should hold all the *mutable* flags about a node. This
patch moves the is_foo flags from routerinfo_t into node_t. The
flags in routerstatus_t remain, but they get set from the consensus
and should not change.
Some other highlights of this patch are:
* Looking up routerinfo and routerstatus by nickname is now
unified and based on the "look up a node by nickname" function.
This tries to look only at the values from current consensus,
and not get confused by the routerinfo_t->is_named flag, which
could get set for other weird reasons. This changes the
behavior of how authorities (when acting as clients) deal with
nodes that have been listed by nickname.
* I tried not to artificially increase the size of the diff here
by moving functions around. As a result, some functions that
now operate on nodes are now in the wrong file -- they should
get moved to nodelist.c once this refactoring settles down.
This moving should happen as part of a patch that moves
functions AND NOTHING ELSE.
* Some old code is now left around inside #if 0/1 blocks, and
should get removed once I've verified that I don't want it
sitting around to see how we used to do things.
There are still some unimplemented functions: these are flagged
with "UNIMPLEMENTED_NODELIST()." I'll work on filling in the
implementation here, piece by piece.
I wish this patch could have been smaller, but there did not seem to
be any piece of it that was independent from the rest. Moving flags
forces many functions that once returned routerinfo_t * to return
node_t *, which forces their friends to change, and so on.
| Nick Mathewson | 2010-10-01 |
* | | Try to make most routerinfo_t interfaces const | Nick Mathewson | 2010-10-01 |
|/ |
|
* | Merge branch 'bug1772' into maint-0.2.2 | Roger Dingledine | 2010-09-29 |
|\ |
|
| * | refactor and recomment; no actual changes | Roger Dingledine | 2010-09-29 |
* | | Merge commit 'mikeperry/bug1740' into maint-0.2.2 | Roger Dingledine | 2010-09-29 |
|\| |
|
| * | Only count timeout data for 3 hop circuits.•••Use 4/3 of this timeout value for 4 hop circuits, and use half of it for
canabalized circuits.
| Mike Perry | 2010-09-29 |
* | | optimistically retry EntryNodes on socks request•••We used to mark all our known bridges up when they're all down and we
get a new socks request. Now do that when we've set EntryNodes too.
| Roger Dingledine | 2010-09-28 |
* | | refactor; no actual changes | Roger Dingledine | 2010-09-28 |
|/ |
|
* | Remove unused function declarations•••Also remove some #if 0'd code from the unit tests for buffers. The
code was killed in e6794e58081af773073c266e23fe3ab2ebecdb7e (5 years
ago), and is now broken anyways.
| Sebastian Hahn | 2010-08-17 |
* | Refactor circuit_build_times_parse_state•••Remove the msg parameter to pass an error message out. This
wasn't needed and made it harder to detect a memory leak.
| Sebastian Hahn | 2010-08-16 |
* | Fix typos, make all \brief's conformant, end sentences with a period. | Linus Nordberg | 2010-07-28 |
* | Create circuitbuild.h | Sebastian Hahn | 2010-07-27 |