Page MenuHomePhabricator

Fix undefined behaviour errors due to long int overflow in Amount
ClosedPublic

Authored by Fabien on Dec 7 2018, 14:53.

Details

Summary

Running the units tests with the undefined behaviour sanitizer throws
runtime errors: signed integer overflow in file amount.h.
This is due to the INF_PRIORITY constant, which is no longer used in the
software. It can be safely removed.

Test Plan

Build with the undefined sanitizer enabled then

./src/test/test_bitcoin

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.