Skip to content

fuzz: avoid returning non-conforming results from FuzzedSock::GetSock…

Cirrus CI / lint failed Mar 21, 2025 in 1m 4s

Task Summary

Instruction lint failed in 00:25

Details

✅ 00:03 clone
✅ 00:01 merge_base
✅ 00:33 unshallow
❌ 00:25 lint

[10:08:29.693] + export LINT_RUNNER_PATH=/lint_test_runner
[10:08:29.693] + LINT_RUNNER_PATH=/lint_test_runner
[10:08:29.693] + ./ci/lint/06_script.sh
[10:08:29.695] + '[' -n 32109 ']'
[10:08:29.695] + export COMMIT_RANGE=HEAD~..HEAD
[10:08:29.695] + COMMIT_RANGE=HEAD~..HEAD
[10:08:29.696] ++ git rev-list -1 HEAD
[10:08:29.698] ++ git rev-list -1 --merges HEAD
[10:08:29.700] + '[' 96382bfb2c48d283af28ffa60e86b54322dd0912 '!=' 96382bfb2c48d283af28ffa60e86b54322dd0912 ']'
[10:08:29.700] + RUST_BACKTRACE=1
[10:08:29.700] + /lint_test_runner/test_runner
[10:08:29.705] Checking commit range (HEAD~..HEAD):
[10:08:29.705] 9c3298eb04 fuzz: avoid returning non-conforming results from FuzzedSock::GetSockName()
[10:08:29.705] 
[10:08:29.999] Args used        : 215
[10:08:29.999] Args documented  : 227
[10:08:29.999] Args undocumented: 0
[10:08:29.999] set()
[10:08:29.999] Args unknown     : 12
[10:08:29.999] {'-includeconf', '-zmqpubhashtxhwm', '-zmqpubhashblock', '-zmqpubrawtx', '-zmqpubsequence', '-zmqpubrawblock', '-zmqpubrawtxhwm', '-zmqpubhashblockhwm', '-zmqpubsequencehwm', '-zmqpubrawblockhwm', '-testdatadir', '-zmqpubhashtx'}
[10:08:31.800] src/test/fuzz/util/net.cpp
[10:08:31.801] ^^^
[10:08:31.801] One or more files use a symbol declared in the bitcoin-build-config.h header. However, they are not
[10:08:31.801] including the header. This is problematic, because the header may or may not be indirectly
[10:08:31.801] included. If the indirect include were to be intentionally or accidentally removed, the build could
[10:08:31.801] still succeed, but silently be buggy. For example, a slower fallback algorithm could be picked,
[10:08:31.801] even though bitcoin-build-config.h indicates that a faster feature is available and should be used.
[10:08:31.801] 
[10:08:31.801] If you are unsure which symbol is used, you can find it with this command:
[10:08:31.801] git grep --perl-regexp '^\s*(?!//).*(BITCOIN_CONFIG_H|CLIENT_VERSION_BUILD|CLIENT_VERSION_IS_RELEASE|CLIENT_VERSION_MAJOR|CLIENT_VERSION_MINOR|COPYRIGHT_HOLDERS|COPYRIGHT_HOLDERS_FINAL|COPYRIGHT_HOLDERS_SUBSTITUTION|COPYRIGHT_YEAR|ENABLE_ARM_SHANI|ENABLE_AVX2|ENABLE_EXTERNAL_SIGNER|ENABLE_SSE41|ENABLE_TRACING|ENABLE_WALLET|ENABLE_X86_SHANI|HAVE_DECL_FORK|HAVE_DECL_FREEIFADDRS|HAVE_DECL_GETIFADDRS|HAVE_DECL_PIPE2|HAVE_DECL_SETSID|HAVE_FDATASYNC|HAVE_GETENTROPY_RAND|HAVE_GETRANDOM|HAVE_MALLOC_INFO|HAVE_MALLOPT_ARENA_MAX|HAVE_O_CLOEXEC|HAVE_POSIX_FALLOCATE|HAVE_SOCKADDR_UN|HAVE_STRONG_GETAUXVAL|HAVE_SYSCTL|HAVE_SYSCTL_ARND|HAVE_SYSTEM|HAVE_SYS_PRCTL_H|HAVE_SYS_RESOURCES_H|HAVE_SYS_VMMETER_H|HAVE_VM_VM_PARAM_H|CLIENT_BUGREPORT|CLIENT_NAME|CLIENT_URL|CLIENT_VERSION_STRING|STRERROR_R_CHAR_P|USE_BDB|USE_DBUS|USE_QRCODE)' -- file_name
[10:08:31.801] 
[10:08:31.801] Make sure to include it with the IWYU pragma. Otherwise, IWYU may falsely instruct to remove the
[10:08:31.801] include again.
[10:08:31.801] 
[10:08:31.801] #include <bitcoin-build-config.h> // IWYU pragma: keep
[10:08:31.801] ^---- ⚠️ Failure generated from lint check 'includes_build_config' (Check that no symbol from bitcoin-build-config.h is used without the header being included)!
[10:08:31.801] 
[10:08:31.801] 
[10:08:32.203] All checks passed!
[10:08:32.463] src/crc32c in HEAD currently refers to tree 454691a9b89ee8b9e1f71a48a7398edba49c3805
[10:08:32.463] src/crc32c in HEAD was last updated in commit 5d45552fd4303f8d668ffbc50cce1053485aeead (tree 454691a9b89ee8b9e1f71a48a7398edba49c3805)
[10:08:32.463] GOOD
[10:08:32.956] src/crypto/ctaes in HEAD currently refers to tree 1b6c31139a71f80245c09597c343936a8e41d021
[10:08:32.958] src/crypto/ctaes in HEAD was last updated in commit 8501bedd7508ac514385806e191aec21ee978891 (tree 1b6c31139a71f80245c09597c343936a8e41d021)
[10:08:32.958] GOOD
[10:08:32.998] src/leveldb in HEAD currently refers to tree 5cc97a9cb91c01ec5fc38e2588ef4b315efdbb48
[10:08:33.000] src/leveldb in HEAD was last updated in commit a130bbd154d535b80fe9f602a13bf2e322817881 (tree 5cc97a9cb91c01ec5fc38e2588ef4b315efdbb48)
[10:08:33.000] GOOD
[10:08:33.060] src/minisketch in HEAD currently refers to tree 87710f57550ac964d39e1245baa179f0800586e6
[10:08:33.062] src/minisketch in HEAD was last updated in commit f5d8b66a8cf23f9ccc51fb9702943c8a5f755f43 (tree 87710f57550ac964d39e1245baa179f0800586e6)
[10:08:33.062] GOOD
[10:08:33.094] src/secp256k1 in HEAD currently refers to tree a6dd5562b35bdbf5f959a18a8af51d526527b9b1
[10:08:33.096] src/secp256k1 in HEAD was last updated in commit 2d46a89386d34d72edf93a24b67e44b82fe6e390 (tree a6dd5562b35bdbf5f959a18a8af51d526527b9b1)
[10:08:33.096] GOOD
[10:08:55.322] Success: no issues found in 305 source files
����