| Commit message (Collapse) | Author | Age |
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
A new set of unit test cases are provided, as well as introducing
an alternative paradigm and macros to support it. Primarily, each test
case is given its own namespace, in order to isolate tests from each
other. We do this by in the usual fashion, by appending module and
submodule names to our symbols. New macros assist by reducing friction
for this and other tasks, like overriding a function in the global
namespace with one in the current namespace, or declaring integer
variables to assist tracking how many times a mock has been called.
A set of tests for a small-scale module has been included in this
commit, in order to highlight how the paradigm can be used. This
suite gives 100% coverage to status.c in test execution.
|
|\
| |
| |
| |
| |
| | |
Conflicts:
src/test/include.am
src/test/test.c
|
| | |
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | | |
Conflicts:
src/or/router.c
src/test/test_dir.c
|
| | |
| | |
| | |
| | | |
resolves ticket 5528.
|
| | |
| | |
| | |
| | | |
(rather than me->address)
|
| | |
| | |
| | |
| | | |
(rather than router->address)
|
| | |
| | |
| | |
| | | |
since it was always just the string version of "addr" anyway
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
Implements #10884.
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
(There is no longer meaningfully any such thing as a HS authority,
since we stopped uploading or downloading v0 hs descriptors in
0.2.2.1-alpha.)
Implements #10881, and part of #10841.
|
| | |
| | |
| | |
| | | |
This function is not used anymore
|
|\ \ \ |
|
| | | | |
|
|/ / /
| | |
| | |
| | |
| | |
| | | |
The remaining vestige is that we continue to publish the V2dir flag,
and that, for the controller, we continue to emit v2 directory
formats when requested.
|
|\ \ \ |
|
| | | | |
|
| | | | |
|
|\| | |
| |_|/
|/| | |
|
| |\ \ |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The problem was that the server_identity_key_is_set() function could
return true under conditions where we don't really have an identity
key -- specifically, where we used to have one, but we stopped being a
server.
This is a fix for 6979; bugfix on 0.2.2.18-alpha where we added that
assertion to get_server_identity_key().
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
We previously used FILENAME_PRIVATE identifiers mostly for
identifiers exposed only to the unit tests... but also for
identifiers exposed to the benchmarker, and sometimes for
identifiers exposed to a similar module, and occasionally for no
really good reason at all.
Now, we use FILENAME_PRIVATE identifiers for identifiers shared by
Tor and the unit tests. They should be defined static when we
aren't building the unit test, and globally visible otherwise. (The
STATIC macro will keep us honest here.)
For identifiers used only by the unit tests and never by Tor at all,
on the other hand, we wrap them in #ifdef TOR_UNIT_TESTS.
This is not the motivating use case for the split test/non-test
build system; it's just a test example to see how it works, and to
take a chance to clean up the code a little.
|
|\ \ \ |
|
| |/ /
| | |
| | |
| | | |
from requesting by ID digest, signing key pair; fixes bug 5595
|
| | |
| | |
| | |
| | | |
(Cherry-picked from fc35ee4910326dc1ae718482b30e57666a71df85)
|
|\ \ \ |
|
| | | |
| | | |
| | | |
| | | | |
Fix for bug 6026
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Conflicts:
src/or/dirserv.c
src/or/dirserv.h
src/test/test_dir.c
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Looks like I turned this off in 6ac42f5e back in 2003 and never got
around to making it work again. There has been no small amount of
code drift.
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Now we can compute the hash and signature of a dirobj before
concatenating the smartlist, and we don't need to play silly games
with sigbuf and realloc any more.
|
| | | | | |
|
|\ \ \ \ \
| |_|/ / /
|/| | | | |
|
| |/ / / |
|
| | | |
| | | |
| | | |
| | | |
| | | | |
also, snap the start time and end time to a day boundary, since most
certs in the wild seem to do this.
|
| | | |
| | | |
| | | |
| | | | |
resolves ticket 8443.
|
|/ / /
| | |
| | |
| | | |
no actual changes in behavior yet
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Avoid the phrase "legally valid" (as opposed to valid but not legal)?
And document what functions really do.
Fix for bug 6935.
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Conflicts:
src/or/channel.h
src/or/connection_or.c
src/or/cpuworker.c
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Implements proposal 214.
Needs testing.
|
| | | |
| | | |
| | | |
| | | | |
and replace it with the good old-fashioned two functions approach
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
For example, we were doing a resolve every time we think about doing a
directory fetch. Now we reuse the cached answer in some cases.
Fixes bugs 1992 (bugfix on 0.2.0.20-rc) and 2410 (bugfix on
0.1.2.2-alpha).
|
| | | |
| | | |
| | | |
| | | | |
I didn't make any of the callers use this feature yet.
|
| | | | |
|
|\ \ \ \
| | |/ /
| |/| | |
|