diff options
author | Vasil Dimov <vd@FreeBSD.org> | 2021-04-13 16:31:04 +0200 |
---|---|---|
committer | Vasil Dimov <vd@FreeBSD.org> | 2022-05-18 16:40:12 +0200 |
commit | 3ad7de225efce3e76530f56bee8a8f7a75ea0f3c (patch) | |
tree | 0bf931f272414bd820a22acb203e73accb9d4563 /src/test/fuzz | |
parent | 7164e00e1bc4e30e69b38a7ba9c557d4fc5d5f87 (diff) |
net: add new method Sock::Bind() that wraps bind()
This will help to increase `Sock` usage and make more code mockable.
Diffstat (limited to 'src/test/fuzz')
-rw-r--r-- | src/test/fuzz/util.cpp | 20 | ||||
-rw-r--r-- | src/test/fuzz/util.h | 2 |
2 files changed, 22 insertions, 0 deletions
diff --git a/src/test/fuzz/util.cpp b/src/test/fuzz/util.cpp index 2514636d6e..c5694b0b06 100644 --- a/src/test/fuzz/util.cpp +++ b/src/test/fuzz/util.cpp @@ -160,6 +160,26 @@ int FuzzedSock::Connect(const sockaddr*, socklen_t) const return 0; } +int FuzzedSock::Bind(const sockaddr*, socklen_t) const +{ + // Have a permanent error at bind_errnos[0] because when the fuzzed data is exhausted + // SetFuzzedErrNo() will always set the global errno to bind_errnos[0]. We want to + // avoid this method returning -1 and setting errno to a temporary error (like EAGAIN) + // repeatedly because proper code should retry on temporary errors, leading to an + // infinite loop. + constexpr std::array bind_errnos{ + EACCES, + EADDRINUSE, + EADDRNOTAVAIL, + EAGAIN, + }; + if (m_fuzzed_data_provider.ConsumeBool()) { + SetFuzzedErrNo(m_fuzzed_data_provider, bind_errnos); + return -1; + } + return 0; +} + std::unique_ptr<Sock> FuzzedSock::Accept(sockaddr* addr, socklen_t* addr_len) const { constexpr std::array accept_errnos{ diff --git a/src/test/fuzz/util.h b/src/test/fuzz/util.h index 6c91844633..f0f9e50698 100644 --- a/src/test/fuzz/util.h +++ b/src/test/fuzz/util.h @@ -64,6 +64,8 @@ public: int Connect(const sockaddr*, socklen_t) const override; + int Bind(const sockaddr*, socklen_t) const override; + std::unique_ptr<Sock> Accept(sockaddr* addr, socklen_t* addr_len) const override; int GetSockOpt(int level, int opt_name, void* opt_val, socklen_t* opt_len) const override; |