Extex101
/
minetest
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix unit tests failing if IPv6 not available 

See:
https://github.com/minetest/minetest/issues/1526
https://github.com/minetest/minetest/issues/793
master
Craig Robbins 2014-11-02 17:21:42 +10:00 committed by kwolekr
parent dfd15fd1d9
commit 209c0dd1af
3 changed files with 124 additions and 134 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

159
src/socket.cpp
View File

@ -62,9 +62,9 @@ typedef int socket_t;
#endif #endif
// Set to true to enable verbose debug output // Set to true to enable verbose debug output
bool socket_enable_debug_output = false; bool socket_enable_debug_output = false; // yuck
bool g_sockets_initialized = false; static bool g_sockets_initialized = false;
// Initialize sockets // Initialize sockets
void sockets_init() void sockets_init()
@ -111,7 +111,7 @@ Address::Address(u8 a, u8 b, u8 c, u8 d, u16 port)
setPort(port); setPort(port);
} }
Address::Address(const IPv6AddressBytes * ipv6_bytes, u16 port) Address::Address(const IPv6AddressBytes *ipv6_bytes, u16 port)
{ {
memset(&m_address, 0, sizeof(m_address)); memset(&m_address, 0, sizeof(m_address));
setAddress(ipv6_bytes); setAddress(ipv6_bytes);
@ -119,7 +119,7 @@ Address::Address(const IPv6AddressBytes * ipv6_bytes, u16 port)
} }
// Equality (address family, address and port must be equal) // Equality (address family, address and port must be equal)
bool Address::operator==(Address &address) bool Address::operator==(const Address &address)
{ {
if(address.m_addr_family != m_addr_family || address.m_port != m_port) if(address.m_addr_family != m_addr_family || address.m_port != m_port)
return false; return false;
@ -137,7 +137,7 @@ bool Address::operator==(Address &address)
return false; return false;
} }
bool Address::operator!=(Address &address) bool Address::operator!=(const Address &address)
{ {
return !(*this == address); return !(*this == address);
} }
@ -289,11 +289,11 @@ void Address::setAddress(u8 a, u8 b, u8 c, u8 d)
m_address.ipv4.sin_addr.s_addr = addr; m_address.ipv4.sin_addr.s_addr = addr;
} }
void Address::setAddress(const IPv6AddressBytes * ipv6_bytes) void Address::setAddress(const IPv6AddressBytes *ipv6_bytes)
{ {
m_addr_family = AF_INET6; m_addr_family = AF_INET6;
m_address.ipv6.sin6_family = AF_INET6; m_address.ipv6.sin6_family = AF_INET6;
if(ipv6_bytes) if (ipv6_bytes)
memcpy(m_address.ipv6.sin6_addr.s6_addr, ipv6_bytes->bytes, 16); memcpy(m_address.ipv6.sin6_addr.s6_addr, ipv6_bytes->bytes, 16);
else else
memset(m_address.ipv6.sin6_addr.s6_addr, 0, 16); memset(m_address.ipv6.sin6_addr.s6_addr, 0, 16);
@ -307,13 +307,9 @@ void Address::setPort(u16 port)
void Address::print(std::ostream *s) const void Address::print(std::ostream *s) const
{ {
if(m_addr_family == AF_INET6) if(m_addr_family == AF_INET6)
{ *s << "[" << serializeString() << "]:" << m_port;
(*s) << "[" << serializeString() << "]:" << m_port;
}
else else
{ *s << serializeString() << ":" << m_port;
(*s) << serializeString() << ":" << m_port;
}
} }
/* /*
@ -322,33 +318,44 @@ void Address::print(std::ostream *s) const
UDPSocket::UDPSocket(bool ipv6) UDPSocket::UDPSocket(bool ipv6)
{ {
if(g_sockets_initialized == false) init(ipv6, false);
throw SocketException("Sockets not initialized"); }
bool UDPSocket::init(bool ipv6, bool noExceptions)
{
if (g_sockets_initialized == false) {
dstream << "Sockets not initialized" << std::endl;
return false;
}
// Use IPv6 if specified // Use IPv6 if specified
m_addr_family = ipv6 ? AF_INET6 : AF_INET; m_addr_family = ipv6 ? AF_INET6 : AF_INET;
m_handle = socket(m_addr_family, SOCK_DGRAM, IPPROTO_UDP); m_handle = socket(m_addr_family, SOCK_DGRAM, IPPROTO_UDP);
if(socket_enable_debug_output) if (socket_enable_debug_output) {
{
dstream << "UDPSocket(" << (int) m_handle dstream << "UDPSocket(" << (int) m_handle
<< ")::UDPSocket(): ipv6 = " << ")::UDPSocket(): ipv6 = "
<< (ipv6 ? "true" : "false") << (ipv6 ? "true" : "false")
<< std::endl; << std::endl;
} }
if(m_handle <= 0) if (m_handle <= 0) {
{ if (noExceptions) {
throw SocketException("Failed to create socket"); return false;
} else {
throw SocketException("Failed to create socket");
}
} }
setTimeoutMs(0); setTimeoutMs(0);
return true;
} }
UDPSocket::~UDPSocket() UDPSocket::~UDPSocket()
{ {
if(socket_enable_debug_output) if (socket_enable_debug_output) {
{
dstream << "UDPSocket( " << (int) m_handle << ")::~UDPSocket()" dstream << "UDPSocket( " << (int) m_handle << ")::~UDPSocket()"
<< std::endl; << std::endl;
} }
@ -362,22 +369,19 @@ UDPSocket::~UDPSocket()
void UDPSocket::Bind(Address addr) void UDPSocket::Bind(Address addr)
{ {
if(socket_enable_debug_output) if(socket_enable_debug_output) {
{
dstream << "UDPSocket(" << (int) m_handle << ")::Bind(): " dstream << "UDPSocket(" << (int) m_handle << ")::Bind(): "
<< addr.serializeString() << ":" << addr.serializeString() << ":"
<< addr.getPort() << std::endl; << addr.getPort() << std::endl;
} }
if (addr.getFamily() != m_addr_family) if (addr.getFamily() != m_addr_family) {
{ static const char *errmsg = "Socket and bind address families do not match";
char errmsg[] = "Socket and bind address families do not match";
errorstream << "Bind failed: " << errmsg << std::endl; errorstream << "Bind failed: " << errmsg << std::endl;
throw SocketException(errmsg); throw SocketException(errmsg);
} }
if(m_addr_family == AF_INET6) if(m_addr_family == AF_INET6) {
{
struct sockaddr_in6 address; struct sockaddr_in6 address;
memset(&address, 0, sizeof(address)); memset(&address, 0, sizeof(address));
@ -386,15 +390,12 @@ void UDPSocket::Bind(Address addr)
address.sin6_port = htons(addr.getPort()); address.sin6_port = htons(addr.getPort());
if(bind(m_handle, (const struct sockaddr *) &address, if(bind(m_handle, (const struct sockaddr *) &address,
sizeof(struct sockaddr_in6)) < 0) sizeof(struct sockaddr_in6)) < 0) {
{
dstream << (int) m_handle << ": Bind failed: " dstream << (int) m_handle << ": Bind failed: "
<< strerror(errno) << std::endl; << strerror(errno) << std::endl;
throw SocketException("Failed to bind socket"); throw SocketException("Failed to bind socket");
} }
} } else {
else
{
struct sockaddr_in address; struct sockaddr_in address;
memset(&address, 0, sizeof(address)); memset(&address, 0, sizeof(address));
@ -402,10 +403,9 @@ void UDPSocket::Bind(Address addr)
address.sin_family = AF_INET; address.sin_family = AF_INET;
address.sin_port = htons(addr.getPort()); address.sin_port = htons(addr.getPort());
if(bind(m_handle, (const struct sockaddr *) &address, if (bind(m_handle, (const struct sockaddr *) &address,
sizeof(struct sockaddr_in)) < 0) sizeof(struct sockaddr_in)) < 0) {
{ dstream << (int)m_handle << ": Bind failed: "
dstream << (int) m_handle << ": Bind failed: "
<< strerror(errno) << std::endl; << strerror(errno) << std::endl;
throw SocketException("Failed to bind socket"); throw SocketException("Failed to bind socket");
} }
@ -417,40 +417,35 @@ void UDPSocket::Send(const Address & destination, const void * data, int size)
bool dumping_packet = false; // for INTERNET_SIMULATOR bool dumping_packet = false; // for INTERNET_SIMULATOR
if(INTERNET_SIMULATOR) if(INTERNET_SIMULATOR)
dumping_packet = (myrand() % INTERNET_SIMULATOR_PACKET_LOSS == 0); dumping_packet = myrand() % INTERNET_SIMULATOR_PACKET_LOSS == 0;
if(socket_enable_debug_output) if(socket_enable_debug_output) {
{
// Print packet destination and size // Print packet destination and size
dstream << (int) m_handle << " -> "; dstream << (int)m_handle << " -> ";
destination.print(&dstream); destination.print(&dstream);
dstream << ", size=" << size; dstream << ", size=" << size;
// Print packet contents // Print packet contents
dstream << ", data="; dstream << ", data=";
for(int i = 0; i < size && i < 20; i++) for(int i = 0; i < size && i < 20; i++) {
{
if(i % 2 == 0) if(i % 2 == 0)
dstream << " "; dstream << " ";
unsigned int a = ((const unsigned char *) data)[i]; unsigned int a = ((const unsigned char *)data)[i];
dstream << std::hex << std::setw(2) << std::setfill('0') dstream << std::hex << std::setw(2) << std::setfill('0') << a;
<< a;
} }
if(size > 20) if(size > 20)
dstream << "..."; dstream << "...";
if(dumping_packet) if(dumping_packet)
dstream << " (DUMPED BY INTERNET_SIMULATOR)"; dstream << " (DUMPED BY INTERNET_SIMULATOR)";
dstream << std::endl; dstream << std::endl;
} }
if(dumping_packet) if(dumping_packet) {
{
// Lol let's forget it // Lol let's forget it
dstream << "UDPSocket::Send(): " dstream << "UDPSocket::Send(): INTERNET_SIMULATOR: dumping packet."
"INTERNET_SIMULATOR: dumping packet."
<< std::endl; << std::endl;
return; return;
} }
@ -459,38 +454,30 @@ void UDPSocket::Send(const Address & destination, const void * data, int size)
throw SendFailedException("Address family mismatch"); throw SendFailedException("Address family mismatch");
int sent; int sent;
if(m_addr_family == AF_INET6) if(m_addr_family == AF_INET6) {
{
struct sockaddr_in6 address = destination.getAddress6(); struct sockaddr_in6 address = destination.getAddress6();
address.sin6_port = htons(destination.getPort()); address.sin6_port = htons(destination.getPort());
sent = sendto(m_handle, (const char *) data, size, sent = sendto(m_handle, (const char *)data, size,
0, (struct sockaddr *) &address, sizeof(struct sockaddr_in6)); 0, (struct sockaddr *)&address, sizeof(struct sockaddr_in6));
} } else {
else
{
struct sockaddr_in address = destination.getAddress(); struct sockaddr_in address = destination.getAddress();
address.sin_port = htons(destination.getPort()); address.sin_port = htons(destination.getPort());
sent = sendto(m_handle, (const char *) data, size, sent = sendto(m_handle, (const char *)data, size,
0, (struct sockaddr *) &address, sizeof(struct sockaddr_in)); 0, (struct sockaddr *)&address, sizeof(struct sockaddr_in));
} }
if(sent != size) if(sent != size)
{
throw SendFailedException("Failed to send packet"); throw SendFailedException("Failed to send packet");
}
} }
int UDPSocket::Receive(Address & sender, void * data, int size) int UDPSocket::Receive(Address & sender, void *data, int size)
{ {
// Return on timeout // Return on timeout
if(WaitData(m_timeout_ms) == false) if(WaitData(m_timeout_ms) == false)
{
return -1; return -1;
}
int received; int received;
if(m_addr_family == AF_INET6) if (m_addr_family == AF_INET6) {
{
struct sockaddr_in6 address; struct sockaddr_in6 address;
memset(&address, 0, sizeof(address)); memset(&address, 0, sizeof(address));
socklen_t address_len = sizeof(address); socklen_t address_len = sizeof(address);
@ -505,16 +492,14 @@ int UDPSocket::Receive(Address & sender, void * data, int size)
IPv6AddressBytes bytes; IPv6AddressBytes bytes;
memcpy(bytes.bytes, address.sin6_addr.s6_addr, 16); memcpy(bytes.bytes, address.sin6_addr.s6_addr, 16);
sender = Address(&bytes, address_port); sender = Address(&bytes, address_port);
} } else {
else
{
struct sockaddr_in address; struct sockaddr_in address;
memset(&address, 0, sizeof(address)); memset(&address, 0, sizeof(address));
socklen_t address_len = sizeof(address); socklen_t address_len = sizeof(address);
received = recvfrom(m_handle, (char *) data, received = recvfrom(m_handle, (char *)data,
size, 0, (struct sockaddr *) &address, &address_len); size, 0, (struct sockaddr *)&address, &address_len);
if(received < 0) if(received < 0)
return -1; return -1;
@ -525,8 +510,7 @@ int UDPSocket::Receive(Address & sender, void * data, int size)
sender = Address(address_ip, address_port); sender = Address(address_ip, address_port);
} }
if(socket_enable_debug_output) if (socket_enable_debug_output) {
{
// Print packet sender and size // Print packet sender and size
dstream << (int) m_handle << " <- "; dstream << (int) m_handle << " <- ";
sender.print(&dstream); sender.print(&dstream);
@ -534,13 +518,11 @@ int UDPSocket::Receive(Address & sender, void * data, int size)
// Print packet contents // Print packet contents
dstream << ", data="; dstream << ", data=";
for(int i = 0; i < received && i < 20; i++) for(int i = 0; i < received && i < 20; i++) {
{
if(i % 2 == 0) if(i % 2 == 0)
dstream << " "; dstream << " ";
unsigned int a = ((const unsigned char *) data)[i]; unsigned int a = ((const unsigned char *) data)[i];
dstream << std::hex << std::setw(2) << std::setfill('0') dstream << std::hex << std::setw(2) << std::setfill('0') << a;
<< a;
} }
if(received > 20) if(received > 20)
dstream << "..."; dstream << "...";
@ -578,15 +560,14 @@ bool UDPSocket::WaitData(int timeout_ms)
// select() // select()
result = select(m_handle+1, &readset, NULL, NULL, &tv); result = select(m_handle+1, &readset, NULL, NULL, &tv);
if(result == 0) if (result == 0)
return false; return false;
else if(result < 0 && (errno == EINTR || errno == EBADF)) else if (result < 0 && (errno == EINTR || errno == EBADF)) {
// N.B. select() fails when sockets are destroyed on Connection's dtor // N.B. select() fails when sockets are destroyed on Connection's dtor
// with EBADF. Instead of doing tricky synchronization, allow this // with EBADF. Instead of doing tricky synchronization, allow this
// thread to exit but don't throw an exception. // thread to exit but don't throw an exception.
return false; return false;
else if(result < 0) } else if (result < 0) {
{
dstream << (int) m_handle << ": Select failed: " dstream << (int) m_handle << ": Select failed: "
<< strerror(errno) << std::endl; << strerror(errno) << std::endl;
@ -602,9 +583,7 @@ bool UDPSocket::WaitData(int timeout_ms)
#endif #endif
throw SocketException("Select failed"); throw SocketException("Select failed");
} } else if(FD_ISSET(m_handle, &readset) == false) {
else if(FD_ISSET(m_handle, &readset) == false)
{
// No data // No data
return false; return false;
} }

12
src/socket.h
View File

@ -85,16 +85,16 @@ public:
Address(); Address();
Address(u32 address, u16 port); Address(u32 address, u16 port);
Address(u8 a, u8 b, u8 c, u8 d, u16 port); Address(u8 a, u8 b, u8 c, u8 d, u16 port);
Address(const IPv6AddressBytes * ipv6_bytes, u16 port); Address(const IPv6AddressBytes *ipv6_bytes, u16 port);
bool operator==(Address &address); bool operator==(const Address &address);
bool operator!=(Address &address); bool operator!=(const Address &address);
// Resolve() may throw ResolveError (address is unchanged in this case) // Resolve() may throw ResolveError (address is unchanged in this case)
void Resolve(const char *name); void Resolve(const char *name);
struct sockaddr_in getAddress() const; struct sockaddr_in getAddress() const;
unsigned short getPort() const; unsigned short getPort() const;
void setAddress(u32 address); void setAddress(u32 address);
void setAddress(u8 a, u8 b, u8 c, u8 d); void setAddress(u8 a, u8 b, u8 c, u8 d);
void setAddress(const IPv6AddressBytes * ipv6_bytes); void setAddress(const IPv6AddressBytes *ipv6_bytes);
struct sockaddr_in6 getAddress6() const; struct sockaddr_in6 getAddress6() const;
int getFamily() const; int getFamily() const;
bool isIPv6() const; bool isIPv6() const;
@ -115,9 +115,13 @@ private:
class UDPSocket class UDPSocket
{ {
public: public:
UDPSocket() { }
UDPSocket(bool ipv6); UDPSocket(bool ipv6);
~UDPSocket(); ~UDPSocket();
void Bind(Address addr); void Bind(Address addr);
bool init(bool ipv6, bool noExceptions = false);
//void Close(); //void Close();
//bool IsOpen(); //bool IsOpen();
void Send(const Address & destination, const void * data, int size); void Send(const Address & destination, const void * data, int size);

87
src/test.cpp
View File

@ -1513,35 +1513,46 @@ struct TestSocket: public TestBase
// IPv6 socket test // IPv6 socket test
{ {
UDPSocket socket6(true); UDPSocket socket6;
socket6.Bind(address6);
const char sendbuffer[] = "hello world!"; if (!socket6.init(true, true)) {
IPv6AddressBytes bytes; /* Note: Failing to create an IPv6 socket is not technically an
bytes.bytes[15] = 1; error because the OS may not support IPv6 or it may
have been disabled. IPv6 is not /required/ by
try { minetest and therefore this should not cause the unit
socket6.Send(Address(&bytes, port), sendbuffer, sizeof(sendbuffer)); test to fail
*/
dstream << "WARNING: IPv6 socket creation failed (unit test)"
<< std::endl;
} else {
const char sendbuffer[] = "hello world!";
IPv6AddressBytes bytes;
bytes.bytes[15] = 1;
sleep_ms(50); socket6.Bind(address6);
char rcvbuffer[256]; try {
memset(rcvbuffer, 0, sizeof(rcvbuffer)); socket6.Send(Address(&bytes, port), sendbuffer, sizeof(sendbuffer));
Address sender;
for(;;) sleep_ms(50);
{
int bytes_read = socket6.Receive(sender, rcvbuffer, sizeof(rcvbuffer)); char rcvbuffer[256] = { 0 };
if(bytes_read < 0) Address sender;
break;
for(;;) {
if (socket6.Receive(sender, rcvbuffer, sizeof(rcvbuffer )) < 0)
break;
}
//FIXME: This fails on some systems
UASSERT(strncmp(sendbuffer, rcvbuffer, sizeof(sendbuffer)) == 0);
UASSERT(memcmp(sender.getAddress6().sin6_addr.s6_addr,
Address(&bytes, 0).getAddress6().sin6_addr.s6_addr, 16) == 0);
}
catch (SendFailedException e) {
errorstream << "IPv6 support enabled but not available!"
<< std::endl;
} }
//FIXME: This fails on some systems
UASSERT(strncmp(sendbuffer, rcvbuffer, sizeof(sendbuffer))==0);
UASSERT(memcmp(sender.getAddress6().sin6_addr.s6_addr, Address(&bytes, 0).getAddress6().sin6_addr.s6_addr, 16) == 0);
} }
catch (SendFailedException e) {
errorstream << "IPv6 support enabled but not available!" << std::endl;
}
} }
// IPv4 socket test // IPv4 socket test
@ -1550,22 +1561,20 @@ struct TestSocket: public TestBase
socket.Bind(address); socket.Bind(address);
const char sendbuffer[] = "hello world!"; const char sendbuffer[] = "hello world!";
socket.Send(Address(127,0,0,1,port), sendbuffer, sizeof(sendbuffer)); socket.Send(Address(127, 0, 0 ,1, port), sendbuffer, sizeof(sendbuffer));
sleep_ms(50); sleep_ms(50);
char rcvbuffer[256]; char rcvbuffer[256] = { 0 };
memset(rcvbuffer, 0, sizeof(rcvbuffer));
Address sender; Address sender;
for(;;) for(;;) {
{ if (socket.Receive(sender, rcvbuffer, sizeof(rcvbuffer)) < 0)
int bytes_read = socket.Receive(sender, rcvbuffer, sizeof(rcvbuffer));
if(bytes_read < 0)
break; break;
} }
//FIXME: This fails on some systems //FIXME: This fails on some systems
UASSERT(strncmp(sendbuffer, rcvbuffer, sizeof(sendbuffer))==0); UASSERT(strncmp(sendbuffer, rcvbuffer, sizeof(sendbuffer)) == 0);
UASSERT(sender.getAddress().sin_addr.s_addr == Address(127,0,0,1, 0).getAddress().sin_addr.s_addr); UASSERT(sender.getAddress().sin_addr.s_addr ==
Address(127, 0, 0, 1, 0).getAddress().sin_addr.s_addr);
} }
} }
}; };
@ -1585,7 +1594,7 @@ struct TestConnection: public TestBase
SharedBuffer<u8> data1(1); SharedBuffer<u8> data1(1);
data1[0] = 100; data1[0] = 100;
Address a(127,0,0,1, 10); Address a(127,0,0,1, 10);
u16 seqnum = 34352; const u16 seqnum = 34352;
con::BufferedPacket p1 = con::makePacket(a, data1, con::BufferedPacket p1 = con::makePacket(a, data1,
proto_id, peer_id, channel); proto_id, peer_id, channel);
@ -1978,23 +1987,21 @@ struct TestConnection: public TestBase
} }
}; };
#define TEST(X)\ #define TEST(X) do {\
{\
X x;\ X x;\
infostream<<"Running " #X <<std::endl;\ infostream<<"Running " #X <<std::endl;\
x.Run();\ x.Run();\
tests_run++;\ tests_run++;\
tests_failed += x.test_failed ? 1 : 0;\ tests_failed += x.test_failed ? 1 : 0;\
} } while (0)
#define TESTPARAMS(X, ...)\ #define TESTPARAMS(X, ...) do {\
{\
X x;\ X x;\
infostream<<"Running " #X <<std::endl;\ infostream<<"Running " #X <<std::endl;\
x.Run(__VA_ARGS__);\ x.Run(__VA_ARGS__);\
tests_run++;\ tests_run++;\
tests_failed += x.test_failed ? 1 : 0;\ tests_failed += x.test_failed ? 1 : 0;\
} } while (0)
void run_tests() void run_tests()
{ {