Page MenuHomePhabricator
Differential D12813

Fix implicit integer sign changes in strencodings and reduce size of strencodings decode tables
ClosedPublic
Actions

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

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC9df212fb7485: Fix implicit integer sign changes in strencodings and reduce size of…
Summary

This is a backport of core#23756

Depends on D12812

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:28
Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 8 2022, 12:28
PiRK requested review of this revision.Dec 8 2022, 12:28
teamcity edited the summary of this revision. (Show Details)Dec 8 2022, 12:28
PiRK added a child revision: D12814: Fix implicit-integer-sign-change in arith_uint256.Dec 8 2022, 12:29
Harbormaster completed remote builds in B21468: Diff 37117.Dec 8 2022, 12:46
Fabien accepted this revision.Dec 8 2022, 14:34
This revision is now accepted and ready to land.Dec 8 2022, 14:34
Closed by commit rABC9df212fb7485: Fix implicit integer sign changes in strencodings and reduce size of… (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: rABC9df212fb7485: Fix implicit integer sign changes in strencodings and reduce size of….
PiRK mentioned this in D12829: fuzz: Avoid -fsanitize=integer warnings in fuzzing harnesses, add UBSan suppressions.Dec 9 2022, 08:27
PiRK mentioned this in rABC7cf480a0cc20: fuzz: Avoid -fsanitize=integer warnings in fuzzing harnesses, add UBSan….Dec 12 2022, 07:50

Revision Contents
Changeset List

PathSize
src/
util/
9 lines
10 lines
test/
sanitizer_suppressions/
1 line

Diff 37146

View Options

src/util/strencodings.h

Loading...
View Options

src/util/strencodings.cpp

Loading...
View Options

test/sanitizer_suppressions/ubsan

Loading...