From 9b05c49ade729311a0f4388a109530ff8d0ed1f9 Mon Sep 17 00:00:00 2001 From: Vasil Dimov Date: Thu, 4 Mar 2021 16:05:33 +0100 Subject: fuzz: implement unimplemented FuzzedSock methods We want `Get()` to always return the same value, otherwise it will look like the `FuzzedSock` implementation itself is broken. So assign `m_socket` a random number in the `FuzzedSock` constructor. There is nothing to fuzz about the `Get()` and `Release()` methods, so use the ones from the base class `Sock`. `Reset()` is just setting our socket to `INVALID_SOCKET`. We don't want to use the base `Reset()` because it will close `m_socket` and given that our `m_socket` is just a random number it may end up closing a real opened file descriptor if it coincides with our random `m_socket`. --- src/util/sock.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/util') diff --git a/src/util/sock.h b/src/util/sock.h index 4b0618dcff..d923de531f 100644 --- a/src/util/sock.h +++ b/src/util/sock.h @@ -153,7 +153,7 @@ public: */ virtual bool IsConnected(std::string& errmsg) const; -private: +protected: /** * Contained socket. `INVALID_SOCKET` designates the object is empty. */ -- cgit v1.2.3