diff options
author | fanquake <fanquake@gmail.com> | 2024-02-13 11:46:56 -0300 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2024-02-13 11:47:10 -0300 |
commit | d7dabdbfcd4de560bb7a2ba1d326999c0e3c249f (patch) | |
tree | 8d1b9b21d4e60a95051681fa93f7ab01aca277ef /src/test/fuzz | |
parent | f83565db45dece755848d6eba2c921973e9db81e (diff) | |
parent | 864e2e9097de8f1fda63137f803687dd5cc96c03 (diff) | |
download | bitcoin-d7dabdbfcd4de560bb7a2ba1d326999c0e3c249f.tar.xz |
Merge bitcoin/bitcoin#29413: fuzz: increase length of string used for `NetWhitelist{bind}Permissions::TryParse`
864e2e9097de8f1fda63137f803687dd5cc96c03 fuzz: increase length of string used for `NetWhitelist{bind}Permissions::TryParse` (brunoerg)
Pull request description:
The string `s` represents the value from `-whitelist`/`-whitebind` (e.g. "bloom,forcerelay,noban@1.2.3.4:32") and it is used in `NetWhitelistPermissions::TryParse` and `NetWhitebindPermissions::TryParse`. However, a max length of 32 is not enough to cover a lot of cases. Even disconsidering the permissions, 32 would not be enough to cover a lot of addresses. This PR fixes it.
ACKs for top commit:
maflcko:
lgtm ACK 864e2e9097de8f1fda63137f803687dd5cc96c03
epiccurious:
utACK 864e2e9097de8f1fda63137f803687dd5cc96c03.
vasild:
ACK 864e2e9097de8f1fda63137f803687dd5cc96c03
Tree-SHA512: 2b89031b9f2ea92d636f05fd167b1e5ac726742a7e7c1af8ddaeaf90236e659731aaa6b7c23f65ec16ce52ac1b9e68e7b16e23c59e355312d057e001976d172a
Diffstat (limited to 'src/test/fuzz')
-rw-r--r-- | src/test/fuzz/net_permissions.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/fuzz/net_permissions.cpp b/src/test/fuzz/net_permissions.cpp index ae343602e9..6ea2139c46 100644 --- a/src/test/fuzz/net_permissions.cpp +++ b/src/test/fuzz/net_permissions.cpp @@ -17,7 +17,7 @@ FUZZ_TARGET(net_permissions) { FuzzedDataProvider fuzzed_data_provider(buffer.data(), buffer.size()); - const std::string s = fuzzed_data_provider.ConsumeRandomLengthString(32); + const std::string s = fuzzed_data_provider.ConsumeRandomLengthString(1000); const NetPermissionFlags net_permission_flags = ConsumeWeakEnum(fuzzed_data_provider, ALL_NET_PERMISSION_FLAGS); NetWhitebindPermissions net_whitebind_permissions; |