| Commit message (Collapse) | Author | Age |
| ... | |
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Another dereference-then-NULL-check sequence. No reports of this bug
triggered in the wild. Fixes bugreport 1256.
Thanks to ekir for discovering and reporting this bug.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fix a dereference-then-NULL-check sequence. This bug wasn't triggered
in the wild, but we should fix it anyways in case it ever happens.
Also make sure users get a note about this being a bug when they
see it in their log.
Thanks to ekir for discovering and reporting this bug.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
We used to only zero the first ptrsize bytes of the cipher. Since
cipher is large enough, we didn't zero too many bytes. Discovered
and fixed by ekir. Fixes bug 1254.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Don't allow anything but directory-signature tokens in a consensus after
the first directory-signature token. Fixes bug in bandwidth-weights branch.
Found by "outofwords."
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This means that "if (E<G) {abc} else if (E>=G) {def}" can be replaced with
"if (E<G) {abc} else {def}"
Doing the second test explicitly made my mingw gcc nervous that we might
never be initializing casename.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
For my 64-bit Linux system running with GCC 4.4.3-fc12-whatever, you
can't do 'printf("%lld", (int64_t)x);' Instead you need to tell the
compiler 'printf("%lld", (long long int)x);' or else it doesn't
believe the types match. This is why we added U64_PRINTF_ARG; it
looks like we needed an I64_PRINTF_ARG too.
|
| |\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | | |
Conflicts:
ChangeLog
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Maybe this is what parakeep was complaining about? Really wish he
would stick around more. Playing these guessing games is not fun :(
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
They are capped to be between 0 and weight_scale (10000) by the code
just before the snprintf.
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Still not sure why they generate an empty consensus document..
Too much frobbing going on there.
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
Also fix a spacing issue.
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Also always predict that we need a high capacity circuit or internal
circuit.
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Also add bwweightscale consensus param. Use it as our
fixed-point calculation width.
|
| | | |/
| |/| |
|
| |\ \ \ |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
All other bandwidthrate settings are restricted to INT32_MAX, but
this check was forgotten for PerConnBWRate and PerConnBWBurst. Also
update the manpage to reflect the fact that specifying a bandwidth
in terabytes does not make sense, because that value will be too
large.
|
| |\ \ \ \ |
|
| | |/ / / |
|
| | | | | |
|
| |/ / /
| | |
| | |
| | |
| | |
| | | |
Patch by Christian Kujau to fix some links in the exit notice file
(the file you'd use for your DirPortFrontPage), as well as making
the file xhtml compatible. Thanks!
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Fix a dereference-then-NULL-check sequence. This bug wasn't triggered
in the wild, but we should fix it anyways in case it ever happens.
Also make sure users get a note about this being a bug when they
see it in their log.
Thanks to ekir for discovering and reporting this bug.
|
| | | | |
|
| | | | |
|
| |/ / |
|
| | | |
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | | |
Don't continue handling EXTENDCIRCUIT messages when we get an
unknown purpose.
|
| | | | |
|
| | | |
| | |
| | |
| | | |
Found by coverity
|
| |\ \ \ |
|
| | |/ /
| | |
| | |
| | |
| | |
| | | |
We used to only zero the first ptrsize bytes of the cipher. Since
cipher is large enough, we didn't zero too many bytes. Discovered
and fixed by ekir. Fixes bug 1254.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
On Windows, we don't have a notion of ~ meaning "our homedir", so we
were deliberately using an #ifdef to avoid calling expand_filename()
in multiple places. This is silly: The right place to turn a function
into a no-op on a single platform is in the function itself, not in
every single call-site.
|
| | | |
| | |
| | |
| | |
| | | |
get_torrc_fname() does the same thing we did in this code, so let's
replace it.
|
| |/ /
| |
| |
| |
| | |
This is so that coverity stops complaining about using a user-supplied
string with the open() syscall. Let's see if it works.
|
| |\ \ |
|
| | | |
| | |
| | |
| | |
| | | |
Make sure we follow basic whitespace sanity principles, update a
few links
|
| |/ /
| |
| |
| |
| |
| |
| | |
Spec conformance issue: The code didn't force the network-status-version
token to be the first token in a v3 vote or consensus.
Problem discovered by Parakeep.
|
| | | |
|