Page MenuHomePhabricator
Differential D12820

test: Avoid testing negative block heights
ClosedPublic
Actions

Authored by PiRK on Dec 8 2022, 12:36.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC32d5b6c9cfa2: test: Avoid testing negative block heights
Summary

This is a backport of core#24237

Depends on D12819

Test Plan

With UBSAN
ninja && ninja check check-functional bitcoin-fuzzers

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

PiRK created this revision.Dec 8 2022, 12:36
Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 8 2022, 12:36
PiRK requested review of this revision.Dec 8 2022, 12:36
teamcity edited the summary of this revision. (Show Details)Dec 8 2022, 12:36
Harbormaster completed remote builds in B21475: Diff 37124.Dec 8 2022, 13:24
Fabien requested changes to this revision.Dec 8 2022, 14:53
Fabien added a subscriber: Fabien.

Despite very funny, it's not the right PR

This revision now requires changes to proceed.Dec 8 2022, 14:53
PiRK requested review of this revision.Dec 8 2022, 15:03
PiRK edited the summary of this revision. (Show Details)

fixed PR url to a less funny one

Fabien accepted this revision.Dec 8 2022, 15:08
This revision is now accepted and ready to land.Dec 8 2022, 15:08
Closed by commit rABC32d5b6c9cfa2: test: Avoid testing negative block heights (authored by MarcoFalke <falke.marco@gmail.com>, committed by PiRK). · Explain WhyDec 9 2022, 07:25
This revision was automatically updated to reflect the committed changes.
PiRK added a commit: rABC32d5b6c9cfa2: test: Avoid testing negative block heights.

Revision Contents
Changeset List

PathSize
src/
test/
6 lines
fuzz/
3 lines
test/
sanitizer_suppressions/
1 line

Diff 37153

View Options

src/test/coins_tests.cpp

Loading...
View Options

src/test/fuzz/coins_view.cpp

Loading...
View Options

test/sanitizer_suppressions/ubsan

Loading...