Skip to content

Commit b82c3a0

Browse files
committed
Merge bitcoin#21929: fuzz: Remove incorrect float round-trip serialization test
fae814c 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 fae814c Tree-SHA512: 8412a7985be2225109f382b7c7ea6d6fcfbea15711671fdf2f41dd1a9adbb3b4489592863751d78bedaff98e9b0b13571d9cae06ffd92db8fbf7ce0f47874a41
2 parents 61fea52 + fae814c commit b82c3a0

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

src/test/fuzz/integer.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -122,10 +122,6 @@ FUZZ_TARGET_INIT(integer, initialize_integer)
122122
assert(dynamic_usage == incremental_dynamic_usage * i64s.size());
123123
}
124124
(void)MillisToTimeval(i64);
125-
const double d = ser_uint64_to_double(u64);
126-
assert(ser_double_to_uint64(d) == u64);
127-
const float f = ser_uint32_to_float(u32);
128-
assert(ser_float_to_uint32(f) == u32);
129125
(void)SighashToStr(uch);
130126
(void)SipHashUint256(u64, u64, u256);
131127
(void)SipHashUint256Extra(u64, u64, u256, u32);

0 commit comments

Comments
 (0)