| Age | Commit message (Collapse) | Author |
|---|
|
|
|
Signed-off-by: David Goulet <dgoulet@torproject.org>
|
|
Part of #15516
Signed-off-by: David Goulet <dgoulet@torproject.org>
|
|
|
|
Padding circuits were regular cells that got closed before their padding
machine could finish. This means that they can still receive regular cells from
their past life, but they have no way or reason to answer them anymore. Hence
let's ignore them before they even get to the proper subsystems.
|
|
The required change is to ignore trailing CRs when diffing files.
|
|
This makes all of the practracker tests get run by make check, and
hence by our CI.
Closes ticket 31304.
|
|
|
|
|
|
|
|
|
|
I had the logic reversed here.
|
|
Since we sometimes call practracker directly, that's where we should
check the TOR_DISABLE_PRACTRACKER envvar.
|
|
This test runs practracker with a set of 0 thresholds, to make sure
that it enumerates all its values right. It tries running with an
empty exceptions file, and with an exceptions file that covers
_some_ of the data, and it makes sure that the outputs are as expected.
|
|
Now that there is only one toplevel place where we print problems,
we can redirect just that one print to a file when we are
regenerating the exceptions.txt file. Previously we redirected
sys.stdout, which is naughty, and forced us to send warnings (and
warnings alone) to stderr.
|
|
These flags let you suppress the message about the number of
problems and warnings, and let you control the thresholds above
which something counts as a problem.
I need this for testing.
|
|
|
|
Previously warnings were generated by magic inside ProblemVault; now
they're printed on demand.
|
|
Instead of having "consider" functions that have to call a global
ProblemVault, we can now generate all the metrics for the code
separately from the decision about what to do for them.
|
|
I'm about to refactor the code into a set of iterators that yield
*all* the metrics for the code, and then add a filter on top of that
to return the problems.
|
|
Signed-off-by: David Goulet <dgoulet@torproject.org>
|
|
|
|
|
|
|
|
|
|
Fixes part of bug 30752
|
|
When an exception is present, we can now violate the limit by a little
bit and only produce a warning. The strict flag overrides this
behavior.
I've given file sizes a 2% tolerances and function sizes/include
counts a 10% tolerance.
Part of 30752
|
|
This option lists every exception that is stricter than it needs to
be.
Part of 30752
|
|
Instead of excluding directories at the last minute if they happen
to appear in our filenames, we exclude them early, before recursing
into all their subdirectories.
Part of 29746.
|
|
Part of 29746.
|
|
This isn't actually something that Tor does, but it's cleaner to do
it this way. Part of 29746.
|
|
The practracker_tests.py unit test file called a function by its old
name.
Also, practracker counted functions as starting one line after the
function name, and ending with the closing brace. Now they start
with the open brace and end with the closing brace.
|
|
|
|
|
|
|
|
Coverity doesn't understand that if begin_cell_parse() returns 0 and
sets is_begindir to 0, its address field will always be set.
Fixes bug 30126; bugfix on 0.2.4.7-alpha; Fixes CID 1447296.
|
|
|
|
And remove the practracker exception for tor_addr_lookup().
Cleanup after 30721.
|
|
|
|
These comments should prevent future instances of 30958.
And allow a larger file in practracker.
Follow up after 30958.
|
|
Right now, this has been done at a high level by confparse.c, but it
makes more sense to lower it.
This API is radically un-typesafe as it stands; we'll be wrapping it
in a safer API as we do #30914 and lower the struct manipulation
code as well.
Closes ticket 30864.
|
|
We need this so we can declare function pointers returning bool
without upsetting our style checker. :/
|
|
Closes ticket 30956.
|
|
Signed-off-by: David Goulet <dgoulet@torproject.org>
|
|
|
|
|
|
|
|
Part of 30799.
|
|
|
|
|
