Skip to content

Commit b807d05

Browse files
practicalswiftsidhujag
authored and
sidhujag
committed
tests: Add serialization/deserialization fuzzing for integral types
1 parent 82b27ae commit b807d05

File tree

1 file changed

+67
-0
lines changed

1 file changed

+67
-0
lines changed

src/test/fuzz/integer.cpp

+67
Original file line numberDiff line numberDiff line change
@@ -19,12 +19,14 @@
1919
#include <script/signingprovider.h>
2020
#include <script/standard.h>
2121
#include <serialize.h>
22+
#include <streams.h>
2223
#include <test/fuzz/FuzzedDataProvider.h>
2324
#include <test/fuzz/fuzz.h>
2425
#include <uint256.h>
2526
#include <util/strencodings.h>
2627
#include <util/system.h>
2728
#include <util/time.h>
29+
#include <version.h>
2830

2931
#include <cassert>
3032
#include <limits>
@@ -54,6 +56,7 @@ void test_one_input(const std::vector<uint8_t>& buffer)
5456
// We cannot assume a specific value of std::is_signed<char>::value:
5557
// ConsumeIntegral<char>() instead of casting from {u,}int8_t.
5658
const char ch = fuzzed_data_provider.ConsumeIntegral<char>();
59+
const bool b = fuzzed_data_provider.ConsumeBool();
5760

5861
const Consensus::Params& consensus_params = Params().GetConsensus();
5962
(void)CheckProofOfWork(u256, u32, consensus_params);
@@ -132,4 +135,68 @@ void test_one_input(const std::vector<uint8_t>& buffer)
132135
(void)GetScriptForDestination(destination);
133136
(void)IsValidDestination(destination);
134137
}
138+
139+
{
140+
CDataStream stream(SER_NETWORK, INIT_PROTO_VERSION);
141+
142+
uint256 deserialized_u256;
143+
stream << u256;
144+
stream >> deserialized_u256;
145+
assert(u256 == deserialized_u256 && stream.empty());
146+
147+
uint160 deserialized_u160;
148+
stream << u160;
149+
stream >> deserialized_u160;
150+
assert(u160 == deserialized_u160 && stream.empty());
151+
152+
uint64_t deserialized_u64;
153+
stream << u64;
154+
stream >> deserialized_u64;
155+
assert(u64 == deserialized_u64 && stream.empty());
156+
157+
int64_t deserialized_i64;
158+
stream << i64;
159+
stream >> deserialized_i64;
160+
assert(i64 == deserialized_i64 && stream.empty());
161+
162+
uint32_t deserialized_u32;
163+
stream << u32;
164+
stream >> deserialized_u32;
165+
assert(u32 == deserialized_u32 && stream.empty());
166+
167+
int32_t deserialized_i32;
168+
stream << i32;
169+
stream >> deserialized_i32;
170+
assert(i32 == deserialized_i32 && stream.empty());
171+
172+
uint16_t deserialized_u16;
173+
stream << u16;
174+
stream >> deserialized_u16;
175+
assert(u16 == deserialized_u16 && stream.empty());
176+
177+
int16_t deserialized_i16;
178+
stream << i16;
179+
stream >> deserialized_i16;
180+
assert(i16 == deserialized_i16 && stream.empty());
181+
182+
uint8_t deserialized_u8;
183+
stream << u8;
184+
stream >> deserialized_u8;
185+
assert(u8 == deserialized_u8 && stream.empty());
186+
187+
int8_t deserialized_i8;
188+
stream << i8;
189+
stream >> deserialized_i8;
190+
assert(i8 == deserialized_i8 && stream.empty());
191+
192+
char deserialized_ch;
193+
stream << ch;
194+
stream >> deserialized_ch;
195+
assert(ch == deserialized_ch && stream.empty());
196+
197+
bool deserialized_b;
198+
stream << b;
199+
stream >> deserialized_b;
200+
assert(b == deserialized_b && stream.empty());
201+
}
135202
}

0 commit comments

Comments
 (0)