Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ticket28668 040 #651

Merged
9 commits merged into from Feb 13, 2019
Merged

Ticket28668 040 #651

9 commits merged into from Feb 13, 2019

Conversation

nmathewson
Copy link
Contributor

No description provided.

nmathewson and others added 7 commits January 16, 2019 15:47
Fixes an instance of ticket28668.
Fixes 29160, and allows 28668 (treating ERR logs as test failures)
to procede.
These are ones that happen on windows only.

Fixes bug 29161.
Because the test is adding entries to the "rend_cache" directly, the
rend_cache_increment_allocation() was never called which made the
rend_cache_clean() call trigger that underflow warning:

rend_cache/clean: [forking] Nov 29 09:55:04.024 [warn] rend_cache_decrement_allocation(): Bug: Underflow in rend_cache_decrement_allocation (on Tor 0.4.0.0-alpha-dev 2240fe6)

The test is still good and valid.

Fixes #28660

Signed-off-by: David Goulet <dgoulet@torproject.org>
@coveralls
Copy link

Pull Request Test Coverage Report for Build 3683

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 1210 unchanged lines in 38 files lost coverage.
  • Overall coverage decreased (-1.6%) to 61.742%

Files with Coverage Reduction New Missed Lines %
src/feature/hs/hs_service.c 1 73.85%
src/feature/dirclient/dirclient.c 1 18.37%
src/feature/nodelist/nodelist.c 2 71.88%
src/core/or/policies.c 2 84.52%
src/feature/nodelist/microdesc.c 2 72.75%
src/core/or/circuitpadding.c 2 97.27%
src/lib/buf/buffers.c 2 90.93%
src/feature/client/addressmap.c 2 60.54%
src/feature/rend/rendcache.c 2 93.02%
src/lib/evloop/token_bucket.h 2 66.67%
Totals Coverage Status
Change from base Build 3595: -1.6%
Covered Lines: 45256
Relevant Lines: 73298

💛 - Coveralls

Copy link
Contributor

@juga0 juga0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The commit message could explain better what it solves and why

@ghost ghost merged commit bbe417a into torproject:master Feb 13, 2019
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants