Merge bitcoin/bitcoin#21929: fuzz: Remove incorrect float round-trip serialization test

fae814c9a6c8ce4822f1fc6b88cfbbde7cc2d49c fuzz: Remove incorrect float round-trip serialization test (MarcoFalke)

Pull request description:

  It tests the wrong way of the round-trip: `int -> float -> int`, but only `float -> int -> float` is allowed and used. See also `src/test/fuzz/float.cpp`.

  Hopefully fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=34118

ACKs for top commit:
  laanwj:
    Anyhow, ACK fae814c9a6c8ce4822f1fc6b88cfbbde7cc2d49c

Tree-SHA512: 8412a7985be2225109f382b7c7ea6d6fcfbea15711671fdf2f41dd1a9adbb3b4489592863751d78bedaff98e9b0b13571d9cae06ffd92db8fbf7ce0f47874a41
This commit is contained in:
W. J. van der Laan 2021-05-14 12:14:01 +02:00 committed by pasta
parent cbbe801a3c
commit d045b0d849

View File

@ -95,10 +95,6 @@ void test_one_input(const std::vector<uint8_t>& buffer)
const unsigned char uch = static_cast<unsigned char>(u8); const unsigned char uch = static_cast<unsigned char>(u8);
(void)memusage::DynamicUsage(uch); (void)memusage::DynamicUsage(uch);
(void)MillisToTimeval(i64); (void)MillisToTimeval(i64);
const double d = ser_uint64_to_double(u64);
assert(ser_double_to_uint64(d) == u64);
const float f = ser_uint32_to_float(u32);
assert(ser_float_to_uint32(f) == u32);
(void)SighashToStr(uch); (void)SighashToStr(uch);
(void)SipHashUint256(u64, u64, u256); (void)SipHashUint256(u64, u64, u256);
(void)SipHashUint256Extra(u64, u64, u256, u32); (void)SipHashUint256Extra(u64, u64, u256, u32);