Age | Commit message (Collapse) | Author | |
---|---|---|---|
2016-06-13 | Merge branch 'maint-0.2.8' | Nick Mathewson | |
2016-06-13 | Merge branch 'maint-0.2.7' into maint-0.2.8 | Nick Mathewson | |
2016-06-13 | Merge branch 'maint-0.2.6' into maint-0.2.7 | Nick Mathewson | |
2016-06-13 | Merge branch 'maint-0.2.5' into maint-0.2.6 | Nick Mathewson | |
2016-06-13 | Merge branch 'maint-0.2.4' into maint-0.2.5 | Nick Mathewson | |
2016-06-12 | Keep make check-spaces happy | Andrea Shepard | |
2016-06-12 | typo/comment/log fixes i found in my sandbox from montreal | Roger Dingledine | |
2016-06-12 | Update geoip and geoip6 to the June 7 2016 database. | Karsten Loesing | |
2016-06-11 | Fix build on 32-bit systems. | Nick Mathewson | |
2016-06-11 | Merge remote-tracking branch 'public/bug19203_027' into maint-0.2.8 | Nick Mathewson | |
2016-06-11 | Merge branch 'bug19180_easy_squashed' | Nick Mathewson | |
2016-06-11 | Add clang's -Wstring-conversion, and fix the one place it hits | Nick Mathewson | |
2016-06-11 | Add -Wmissing-variable-declarations, with attendant fixes | Nick Mathewson | |
This is a big-ish patch, but it's very straightforward. Under this clang warning, we're not actually allowed to have a global variable without a previous extern declaration for it. The cases where we violated this rule fall into three roughly equal groups: * Stuff that should have been static. * Stuff that was global but where the extern was local to some other C file. * Stuff that was only global when built for the unit tests, that needed a conditional extern in the headers. The first two were IMO genuine problems; the last is a wart of how we build tests. | |||
2016-06-11 | Add the -Wextra-semi warning from clang, and fix the cases where it triggers | Nick Mathewson | |
2016-06-11 | Resolve some warnings from OSX clang. | Nick Mathewson | |
2016-06-11 | Use autoconf, not gcc version, to decide which warnings we have | Nick Mathewson | |
This gives more accurate results under Clang, which can only help us detect more warnings in more places. Fixes bug 19216; bugfix on 0.2.0.1-alpha | |||
2016-06-11 | Enable -Woverlength-strings for GCC>=4.6 on MOST of the code. | Nick Mathewson | |
IMO it's fine for us to make exceptions to this rule in the unit tests, but not in the code at large. | |||
2016-06-11 | Use -Wdouble-promotion in GCC >= 4.6 | Nick Mathewson | |
This warning triggers on silently promoting a float to a double. In our code, it's just a sign that somebody used a float by mistake, since we always prefer double. | |||
2016-06-11 | Add -Wfloat-conversion for GCC >= 4.9 | Nick Mathewson | |
This caught quite a few minor issues in our unit tests and elsewhere in our code. | |||
2016-06-11 | Add -Wunused-const-variable=2 on GCC >=6.1 | Nick Mathewson | |
This caused a trivial warning in curve25519-donna-64bit.h, which had two unused constants. I commented them out. | |||
2016-06-11 | Merge remote-tracking branch 'public/bug19203_027' into HEAD | Nick Mathewson | |
2016-06-11 | Enable -Wnull-dereference (GCC >=6.1), and fix the easy cases | Nick Mathewson | |
This warning, IIUC, means that the compiler doesn't like it when it sees a NULL check _after_ we've already dereferenced the variable. In such cases, it considers itself free to eliminate the NULL check. There are a couple of tricky cases: One was the case related to the fact that tor_addr_to_in6() can return NULL if it gets a non-AF_INET6 address. The fix was to create a variant which asserts on the address type, and never returns NULL. | |||
2016-06-11 | Merge remote-tracking branch 'asn/bug17688' | Nick Mathewson | |
2016-06-09 | Reduce make check-spaces noise | Andrea Shepard | |
2016-06-07 | Don't rely on consensus parameter to use a single guard. | George Kadianakis | |
2016-06-06 | Repair test_crypto_openssl_version with LibreSSL | Nick Mathewson | |
2016-06-06 | Merge branch 'maint-0.2.8' | Nick Mathewson | |
2016-06-06 | Check tor_sscanf return value in test_crypto.c | Nick Mathewson | |
Coverity noticed that we check tor_sscanf's return value everywhere else. | |||
2016-06-02 | Use directory_must_use_begindir to predict we'll surely use begindir | Nick Mathewson | |
Previously, we used !directory_fetches_from_authorities() to predict that we would tunnel connections. But the rules have changed somewhat over the course of 0.2.8 | |||
2016-06-02 | Use tor_sscanf, not sscanf, in test_crypto.c | Nick Mathewson | |
Fixes the 0.2.9 instance of bug #19213, which prevented mingw64 from working. This case wasn't in any released Tor. | |||
2016-06-02 | Merge branch 'maint-0.2.8' | Nick Mathewson | |
2016-06-02 | Merge branch 'maint-0.2.7' into maint-0.2.8 | Nick Mathewson | |
2016-06-02 | Use tor_sscanf, not sscanf, in test_util.c. | Nick Mathewson | |
Fixes the 0.2.7 case of bug #19213, which prevented mingw64 from working. | |||
2016-05-30 | Let's not even talk about those errors, ok? | Nick Mathewson | |
2016-05-30 | Wait, we had sprintf() in our unit tests?? FOR SHAME! | Nick Mathewson | |
2016-05-30 | Replace nearly all XXX0vv comments with smarter ones | Nick Mathewson | |
So, back long ago, XXX012 meant, "before Tor 0.1.2 is released, we had better revisit this comment and fix it!" But we have a huge pile of such comments accumulated for a large number of released versions! Not cool. So, here's what I tried to do: * 0.2.9 and 0.2.8 are retained, since those are not yet released. * XXX+ or XXX++ or XXX++++ or whatever means, "This one looks quite important!" * The others, after one-by-one examination, are downgraded to plain old XXX. Which doesn't mean they aren't a problem -- just that they cannot possibly be a release-blocking problem. | |||
2016-05-30 | We no longer generate v0 directories. Remove the code to do so | Nick Mathewson | |
2016-05-30 | Fix a warning on unnamed nodes in node_get_by_nickname(). | Nick Mathewson | |
There was a > that should have been an ==, and a missing !. These together prevented us from issuing a warning in the case that a nickname matched an Unnamed node only. Fixes bug 19203; bugfix on 0.2.3.1-alpha. | |||
2016-05-27 | Merge remote-tracking branch 'arma/task19035-fixedup' | Nick Mathewson | |
2016-05-27 | remove a now-unused section of or.h | Roger Dingledine | |
2016-05-27 | Enable the -Waggregate-return warning | Nick Mathewson | |
Suppress it in the one spot in the code where we actually do want to allow an aggregate return in order to call the mallinfo() API. | |||
2016-05-27 | Use ENABLE_GCC_WARNING and DISABLE_GCC_WARNING in tortls.c | Nick Mathewson | |
Previously we'd done this ad hoc. | |||
2016-05-27 | Add support for temporarily suppressing a warning | Nick Mathewson | |
There are a few places where we want to disable a warning: for example, when it's impossible to call a legacy API without triggering it, or when it's impossible to include an external header without triggering it. This pile of macros uses GCC's c99 _Pragma support, plus the usual macro trickery, to enable and disable warnings. | |||
2016-05-27 | remove an unneeded layer of indentation | Roger Dingledine | |
no actual behavior changes | |||
2016-05-27 | Disable GET /tor/bytes.txt and GETINFO dir-usage | Roger Dingledine | |
Remove support for "GET /tor/bytes.txt" DirPort request, and "GETINFO dir-usage" controller request, which were only available via a compile-time option in Tor anyway. Feature was added in 0.2.2.1-alpha. Resolves ticket 19035. | |||
2016-05-27 | Merge remote-tracking branch 'asn/feature19036' | Nick Mathewson | |
2016-05-27 | Merge remote-tracking branch 'arma/bug18840' | Nick Mathewson | |
2016-05-26 | Bump to 0.2.8.3-alpha-dev | Nick Mathewson | |
2016-05-26 | Bump to 0.2.8.3-alpha | Nick Mathewson | |
2016-05-26 | Merge branch 'maint-0.2.8' | Nick Mathewson | |