letsencrypt / ct-woodpecker

A tool to monitor a certificate transparency log for operational problems
Mozilla Public License 2.0
182 stars 19 forks source link

inclusion-checker: fix off-by-one error. #49

Closed cpu closed 6 years ago

cpu commented 6 years ago

The second argument to getEntries is inclusive. When fetching to the current treesize we need to fetch the entries up to treesize - 1.

Similarly, when looping we need to exit when start == end, meaning the loop should run when start < end, not start <= end.

coveralls commented 6 years ago

Pull Request Test Coverage Report for Build 261


Totals Coverage Status
Change from base Build 257: 0.0%
Covered Lines: 644
Relevant Lines: 969

💛 - Coveralls