Paths

Table of Contentst

Differential D12812

refactor: Fix implicit-signed-integer-truncation in cuckoocache.h
ClosedPublic
Actions

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

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC9a1608daf553: refactor: Fix implicit-signed-integer-truncation in cuckoocache.h

Summary

This is a backport of core#23626

Depends on D12799

Test Plan

With UBSAN
ninja && ninja check check-functional

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

PiRK created this revision.Dec 8 2022, 12:27

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 8 2022, 12:27

PiRK requested review of this revision.Dec 8 2022, 12:27

teamcity edited the summary of this revision. (Show Details)Dec 8 2022, 12:27

PiRK edited the summary of this revision. (Show Details)Dec 8 2022, 12:27

PiRK added a parent revision: D12799: refactor: Remove implicit-integer-sign-change suppressions in validation.cpp.

PiRK added a child revision: D12813: Fix implicit integer sign changes in strencodings and reduce size of strencodings decode tables.Dec 8 2022, 12:28

Harbormaster completed remote builds in B21467: Diff 37116.Dec 8 2022, 12:39

Fabien accepted this revision.Dec 8 2022, 14:30

This revision is now accepted and ready to land.Dec 8 2022, 14:30

Closed by commit rABC9a1608daf553: refactor: Fix implicit-signed-integer-truncation in cuckoocache.h (authored by MarcoFalke <falke.marco@gmail.com>, committed by PiRK). · Explain WhyDec 9 2022, 07:24

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABC9a1608daf553: refactor: Fix implicit-signed-integer-truncation in cuckoocache.h.

Revision Contents
Changeset List

Path

Size

src/

cuckoocache.h

5 lines

test/

sanitizer_suppressions/

ubsan

1 line

Diff 37145

View Options

src/cuckoocache.h