summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | | | | | | | | Merge remote-tracking branch 'origin/maint-0.2.3'Nick Mathewson2012-08-17
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / /
| * | | | | | | | Merge remote-tracking branch 'public/bug6507' into maint-0.2.3Nick Mathewson2012-08-17
| |\ \ \ \ \ \ \ \
| | * | | | | | | | Reject attempts to say FooPort and FooPort 0 in the same cfg domainNick Mathewson2012-08-09
| | | | | | | | | |
| | * | | | | | | | Fix spaces from last patchNick Mathewson2012-08-09
| | | | | | | | | |
| | * | | | | | | | Don't infer we have a FooPort from the presence of a FooPort lineNick Mathewson2012-08-09
| | |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Thanks to the changes we started making with SocksPort and friends in 0.2.3.3-alpha, any of our code that did "if (options->Sockport)" became wrong, since "SocksPort 0" would make that test true whereas using the default SocksPort value would make it false. (We didn't actually do "if (options->SockPort)" but we did have tests for TransPort. When we moved DirPort, ORPort, and ControlPort over to the same system in 0.2.3.9-alpha, the problem got worse, since our code is littered with checks for DirPort and ORPort as booleans. This code renames the current linelist-based FooPort options to FooPort_lines, and adds new FooPort_set options which get set at parse-and-validate time on the or_options_t. FooPort_set is true iff we will actually try to open a listener of the given type. (I renamed the FooPort options rather than leave them alone so that every previous user of a FooPort would need to get inspected, and so that any new code that forgetfully uses FooPort will need fail to compile.) Fix for bug 6507.
* | | | | | | | | Merge remote-tracking branch 'origin/maint-0.2.3'Nick Mathewson2012-08-17
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / /
| * | | | | | | | Whitespace and build fixes on 6475 patchNick Mathewson2012-08-17
| | | | | | | | |
| * | | | | | | | Merge remote-tracking branch 'mikeperry/bug6475' into maint-0.2.3Nick Mathewson2012-08-17
| |\ \ \ \ \ \ \ \
| | * | | | | | | | Address Nick's comments from code review.Mike Perry2012-08-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also promote log messages to notice and rate-limit them.
| | * | | | | | | | Bug 6475: Explicitly track our path bias state.Mike Perry2012-08-15
| | |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is done to avoid spurious warns. Additional log lines are also added to try to track down the codepaths where we are somehow overcounting success counts.
* | | | | | | | | Merge remote-tracking branch 'origin/maint-0.2.3'Nick Mathewson2012-08-17
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / | | | | / / / / / | |_|_|/ / / / / |/| | | | | | |
| * | | | | | | Merge remote-tracking branch 'public/bug6514' into maint-0.2.3Nick Mathewson2012-08-17
| |\ \ \ \ \ \ \ | | |/ / / / / / | |/| | | | | |
| | * | | | | | Defensive programming: clear rs_out between iterations.Nick Mathewson2012-08-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I can't currently find a bug here, but there are a couple of near-misses. Addresses ticket 6514; reported pseudonymously on IRC.
* | | | | | | | Check for stream_id, not conn, on extend cells.Nick Mathewson2012-08-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Extend cells aren't allowed to have a stream_id, but we were only blocking them when they had a stream_id that corresponded to a connection. As far as I can tell, this change is harmless: it will make some kinds of broken clients not work any more, but afaik nobody actually make a client that was broken in that way. Found while hunting for other places where we made the same mistake as in 6271. Bugfix on d7f50337c14c back from May 2003, which introduced telescoping circuit construction into 0.0.2pre8.
* | | | | | | | Clarify the last changes messageNick Mathewson2012-08-13
| | | | | | | |
* | | | | | | | oops: changes file for f45cde05f934ee95e0002ff7d72f7006ce156a4cNick Mathewson2012-08-13
| | | | | | | |
* | | | | | | | Remove tor_malloc_roundup().Nick Mathewson2012-08-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This function never actually did us any good, and it added a little complexity. See the changes file for more info.
* | | | | | | | forward-port the 0.2.2.38 changelogRoger Dingledine2012-08-13
| |_|_|_|/ / / |/| | | | | |
* | | | | | | Temporarily make spurious sendmes warn louder at arma's suggestion.Nick Mathewson2012-08-09
| | | | | | |
* | | | | | | Merge remote-tracking branch 'origin/maint-0.2.3'Nick Mathewson2012-08-09
|\ \ \ \ \ \ \ | |/ / / / / /
| * | | | | | Merge remote-tracking branch 'public/bug6252_again' into maint-0.2.3Nick Mathewson2012-08-09
| |\ \ \ \ \ \
| | * | | | | | Have the bug6252 commit message reflect its on-again/off-again statusNick Mathewson2012-07-18
| | | | | | | |
| | * | | | | | Detect bug 6252 (unexpected sendme cell)Roger Dingledine2012-07-18
| | | |_|_|_|/ | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I only check on circuits, not streams, since bloating your stream window past the initial circuit window can't help you much. Also, I compare to CIRCWINDOW_START_MAX so we don't have surprising races if we lower CIRCWINDOW_START for an experiment.
* | | | | | | forward-port the 0.2.3.20-rc changelogRoger Dingledine2012-08-07
| | | | | | |
* | | | | | | Speak not the name of INT_MIN; it can upset older compilersNick Mathewson2012-08-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | And more to the point, some GCCs will warn that you can't say it before C90. Bug not in any released version of Tor.
* | | | | | | Merge remote-tracking branch 'origin/maint-0.2.3'Nick Mathewson2012-08-03
|\ \ \ \ \ \ \ | |/ / / / / /
| * | | | | | Merge remote-tracking branch 'origin/maint-0.2.2' into maint-0.2.3Nick Mathewson2012-08-03
| |\ \ \ \ \ \ | | | |_|_|_|/ | | |/| | | | | | | | | | | | | | | | | | Conflicts: src/or/routerlist.c
| | * | | | | Try to clarify impact of bug 6537Nick Mathewson2012-08-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I don't personally agree that this is likely to be easy to exploit, and some initial experimention I've done suggests that cache-miss times are just plain too fast to get useful info out of when they're mixed up with the rest of Tor's timing noise. Nevertheless, I'm leaving Robert's initial changelog entry in the git history so that he can be the voice of reason if I'm wrong. :)
| | * | | | | Mitigate a side-channel leak of which relays Tor chooses for a circuitRobert Ransom2012-08-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tor's and OpenSSL's current design guarantee that there are other leaks, but this one is likely to be more easily exploitable, and is easy to fix.
* | | | | | | Merge remote-tracking branch 'origin/maint-0.2.3'Nick Mathewson2012-08-03
|\ \ \ \ \ \ \ | |/ / / / / /
| * | | | | | Remove bogus comment claiming that an assertion is triggerable by consensusRobert Ransom2012-08-03
| | | | | | |
* | | | | | | Fix a bunch of "implicit 64->32" warnings from introduce refactoringNick Mathewson2012-08-03
| | | | | | |
* | | | | | | Merge remote-tracking branch 'origin/maint-0.2.3'Nick Mathewson2012-08-03
|\ \ \ \ \ \ \ | |/ / / / / /
| * | | | | | Merge remote-tracking branch 'origin/maint-0.2.2' into maint-0.2.3Nick Mathewson2012-08-03
| |\ \ \ \ \ \ | | |/ / / / / | | | | / / / | | |_|/ / / | |/| | | |
| | * | | | Clarify security impact of bug 6530Nick Mathewson2012-08-03
| | | | | |
| | * | | | Avoid possible segfault when handling networkstatus vote with bad flavorNick Mathewson2012-08-03
| | | | | | | | | | | | | | | | | | | | | | | | Fix for 6530; fix on 0.2.2.6-alpha.
* | | | | | Updated docs for new connections.Matthew Finkel2012-08-02
| | | | | |
* | | | | | Merge remote-tracking branch 'sysrqb/bug6518'Nick Mathewson2012-08-02
|\ \ \ \ \ \
| * | | | | | Constify struct sockaddr *sa parameter for checkMatthew Finkel2012-08-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The values are only being checked, not modified.
* | | | | | | Removed redundant check_sockaddr_family_match callMatthew Finkel2012-08-02
|/ / / / / /
* | | | | | add $(AM_V_GEN) to documentation generation Makefile targets.Stewart Smith2012-08-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes the V=1 or V=0 automake silent build options display (or hide) the full command line used. GEN foo.bar will be seen rather than the full command. As with all automake silent rules, "make V=1" will output the full command.
* | | | | | Enable the automake silent rules.Stewart Smith2012-08-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | $ make V=1 # will temporarily disable them otherwise you see: CC foo.c rather than the giant long bulid line. This makes it significantly easier to spot compiler warnings etc. Additionally, make them conditional, so we won't error on automake < 1.11 (commits squashed by nickm.)
* | | | | | Merge remote-tracking branch 'origin/maint-0.2.3'Nick Mathewson2012-07-31
|\ \ \ \ \ \ | |/ / / / /
| * | | | | Merge branch 'bug6480_squashed' into maint-0.2.3Nick Mathewson2012-07-31
| |\ \ \ \ \
| | * | | | | Avoid hard (impossible?)-to-trigger double-free in dns_resolve()Nick Mathewson2012-07-31
| |/ / / / / | | | | | | | | | | | | | | | | | | Fixes 6480; fix on 0.2.0.1-alpha; based on pseudonymous patch.
* | | | | | Merge branch 'bug6177_squashed'Nick Mathewson2012-07-31
|\ \ \ \ \ \
| * | | | | | new changes entry for rend_service_introduce refactoringNick Mathewson2012-07-31
| | | | | | |
| * | | | | | test_assert is always in affect: no need for extra gotosNick Mathewson2012-07-31
| | | | | | |
| * | | | | | Remove duplicate code in test_introduce.cNick Mathewson2012-07-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Two of the do_*_test functions were actually prefixes of the third, which suggests a trivial code elimination step
| * | | | | | Whitespace tweaksNick Mathewson2012-07-31
| | | | | | |