HomePhabricator

zmq: Fix implicit-integer-sign-change

Description

zmq: Fix implicit-integer-sign-change

Summary:
uint256::begin() returns unsigned data, so there is no reason to make it signed.

Fix that and remove the sanitizer suppression.

This is a backport of core#24218

Depends on D12817

Test Plan:
With UBSAN
ninja && ninja check check-functional

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D12818

Details

Provenance
MarcoFalke <falke.marco@gmail.com>Authored on Jan 31 2022, 16:03
PiRKCommitted on Dec 9 2022, 07:24
PiRKPushed on Dec 9 2022, 07:24
Reviewer
Restricted Project
Differential Revision
D12818: zmq: Fix implicit-integer-sign-change
Parents
rABC54363e9dbc46: refactor: Make MessageBoxFlags enum underlying type unsigned
Branches
Unknown
Tags
Unknown