[ubsan] tentatively remove suppressions for txmempool and serialize
ClosedPublic
Actions

Authored by PiRK on Dec 9 2022, 08:32.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC794a943d2225: [ubsan] tentatively remove suppressions for txmempool and serialize

Summary

If we never see any failure it will be much harder to fix the problem, and file-wide suppressions can hide introduction of new problems.

Test Plan

With UBSAN
ninja && ninja check check-functional

Diff Detail

Repository

rABC Bitcoin ABC

Branch

pr21000

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 21488
Build 42624: Build Diff	build-without-wallet · build-diff · build-debug · build-clang-tidy · build-clang
Build 42623: arc lint + arc unit

Event Timeline

PiRK created this revision.Dec 9 2022, 08:32

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 9 2022, 08:32

@bot build-ubsan

teamcity awarded a token.Dec 9 2022, 08:33

Harbormaster completed remote builds in B21488: Diff 37156.Dec 9 2022, 08:57

PiRK published this revision for review.Dec 9 2022, 10:46

Fabien accepted this revision.Dec 9 2022, 13:54

This revision is now accepted and ready to land.Dec 9 2022, 13:54

Closed by commit rABC794a943d2225: [ubsan] tentatively remove suppressions for txmempool and serialize (authored by PiRK). · Explain WhyDec 12 2022, 07:50

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABC794a943d2225: [ubsan] tentatively remove suppressions for txmempool and serialize.

Revision Contents
Changeset List

Path

Size

test/

sanitizer_suppressions/

ubsan

3 lines

Diff 37156

View Options

test/sanitizer_suppressions/ubsan

[ubsan] tentatively remove suppressions for txmempool and serializeClosedPublicActions