fanquake
d09071da5b
[MOVEONLY] consensus: move amount.h into consensus
...
Move amount.h to consensus/amount.h.
Renames, adds missing and removes uneeded includes.
2021-09-30 07:41:57 +08:00
MarcoFalke
fa7f77b7d1
Fix addrdb includes
2021-09-07 10:44:48 +02:00
fanquake
5ef2738089
util: make ParseMoney return a std::optional<CAmount>
2021-08-04 19:48:24 +08:00
MarcoFalke
fabf17056c
fuzz: Move CTxDestination fuzzing to script fuzz target
...
No need to split it over several targets
2021-07-04 21:30:50 +02:00
MarcoFalke
fae814c9a6
fuzz: Remove incorrect float round-trip serialization test
2021-05-12 14:42:41 +02:00
practicalswift
1f05dbd06d
util: Avoid invalid integer negation in ValueFromAmount: make ValueFromAmount(const CAmount& n) well-defined also when n is std::numeric_limits<CAmount>::min()
2021-03-02 16:05:28 +00:00
practicalswift
7cc75c9ba3
util: Avoid invalid integer negation in FormatMoney: make FormatMoney(const CAmount& n) well-defined also when n is std::numeric_limits<CAmount>::min()
2021-03-02 16:05:28 +00:00
MarcoFalke
44444ba759
fuzz: Link all targets once
2020-12-10 07:15:42 +01:00
fanquake
68ef9523d1
Merge #18413 : script: prevent UB when computing abs value for num opcode serialize
...
2748e87932
script: prevent UB when computing abs value for num opcode serialize (pierrenn)
Pull request description:
This was reported by practicalswift here #18046
It seems that the original author of the line used a reference to glibc `abs`: https://github.com/lattera/glibc/blob/master/stdlib/abs.c
However depending on some implementation details this can be undefined behavior for unusual values.
A detailed explanation of the UB is provided here : https://stackoverflow.com/questions/17313579/is-there-a-safe-way-to-get-the-unsigned-absolute-value-of-a-signed-integer-with (by [Billy O'Neal](https://twitter.com/malwareminigun ))
Simple relevant godbolt example : https://godbolt.org/z/yRwtCG
Thanks!
ACKs for top commit:
sipa:
ACK 2748e87932
MarcoFalke:
ACK 2748e87932
, only checked that the bitcoind binary does not change with clang -O2 🎓
practicalswift:
ACK 2748e87932
Tree-SHA512: 539a34c636c2674c66cb6e707d9d0dfdce63f59b5525610ed88da10c9a8d59d81466b111ad63b850660cef3750d732fc7755530c81a2d61f396be0707cd86dec
2020-05-02 21:24:05 +08:00
practicalswift
90b635e84e
tests: Add fuzzing coverage for CHECK_NONFATAL(...)
2020-04-26 20:23:56 +00:00
MarcoFalke
fa488f131f
scripted-diff: Bump copyright headers
...
-BEGIN VERIFY SCRIPT-
./contrib/devtools/copyright_header.py update ./
-END VERIFY SCRIPT-
2020-04-16 13:33:09 -04:00
pierrenn
2748e87932
script: prevent UB when computing abs value for num opcode serialize
2020-04-09 08:32:00 +09:00
practicalswift
283bd72156
tests: Add coverage of {,Incremental}DynamicUsage(const std::set<X, Y>& s) to existing fuzzer
2020-04-08 14:45:27 +00:00
practicalswift
2df5701e90
tests: Add coverage of GetVirtualTransactionSize(...) to existing fuzzer
2020-04-08 14:45:27 +00:00
MarcoFalke
faa64af960
fuzz: Add CScriptNum::getint coverage
2020-04-03 09:02:34 +08:00
MarcoFalke
faaf1cb5b9
util: Replace i64tostr with ToString
2020-03-27 10:14:08 -04:00
MarcoFalke
fac96fff62
util: Remove unused itostr
2020-03-27 08:59:06 -04:00
practicalswift
87d24e67bb
tests: Add integer serialization/deserialization fuzzing harness
2020-03-24 16:48:28 +00:00
practicalswift
cb4eec13c0
tests: Add fuzzing harness for count_seconds(...)
2020-03-15 15:19:50 +00:00
practicalswift
7a861a62c1
tests: Fuzz HasAllDesirableServiceFlags(...) and MayHaveUsefulAddressDB(...)
2020-03-11 12:51:26 +00:00
practicalswift
1d324ce922
tests: Fuzz additional functions in the integer fuzzing harness
2020-03-07 13:40:19 +00:00
MarcoFalke
a2b5aae9f3
Merge #17996 : tests: Add fuzzing harness for serialization/deserialization of floating-points and integrals
...
9ff41f6419
tests: Add float to FUZZERS_MISSING_CORPORA (temporarily) (practicalswift)
8f6fb0a85a
tests: Add serialization/deserialization fuzzing for integral types (practicalswift)
3c82b92d2e
tests: Add fuzzing harness for functions taking floating-point types as input (practicalswift)
c2bd588860
Add missing includes (practicalswift)
Pull request description:
Add simple fuzzing harness for functions with floating-point parameters (such as `ser_double_to_uint64(double)`, etc.).
Add serialization/deserialization fuzzing for integral types.
Add missing includes.
To test this PR:
```
$ make distclean
$ ./autogen.sh
$ CC=clang CXX=clang++ ./configure --enable-fuzz \
--with-sanitizers=address,fuzzer,undefined
$ make
$ src/test/fuzz/float
…
```
Top commit has no ACKs.
Tree-SHA512: 9b5a0c4838ad18d715c7398e557d2a6d0fcc03aa842f76d7a8ed716170a28f17f249eaede4256998aa3417afe2935e0ffdfaa883727d71ae2d2d18a41ced24b5
2020-03-05 15:41:30 -05:00
practicalswift
8f6fb0a85a
tests: Add serialization/deserialization fuzzing for integral types
2020-03-05 20:35:26 +00:00
practicalswift
4a7fd7a712
tests: Add amount compression/decompression fuzzing to existing fuzzing harness: test compression round-trip
2020-01-22 13:06:52 +00:00
practicalswift
597d10ceb9
tests: Add fuzzing harness for various functions consuming only integrals
2019-12-06 09:14:17 +00:00