Recvfrom tidy and store laste error.

This commit is contained in:
Matthew Parlane 2013-09-01 15:15:55 +12:00
parent 1c106abf13
commit ffe710b4e8
3 changed files with 55 additions and 34 deletions

View File

@ -755,6 +755,13 @@ bool CWII_IPC_HLE_Device_net_ip_top::IOCtl(u32 _CommandAddress)
s32 errorcode = Memory::Read_U32(BufferOut + 0x10); s32 errorcode = Memory::Read_U32(BufferOut + 0x10);
INFO_LOG(WII_IPC_NET,"IOCTL_SO_GETSOCKOPT error code = %i", errorcode); INFO_LOG(WII_IPC_NET,"IOCTL_SO_GETSOCKOPT error code = %i", errorcode);
} }
else if (optname == SO_ERROR)
{
s32 last_error = WiiSockMan::getInstance().getLastNetError();
Memory::Write_U32(sizeof(s32), BufferOut + 0xC);
Memory::Write_U32(last_error, BufferOut + 0x10);
}
break; break;
} }

View File

@ -37,21 +37,9 @@ char* WiiSockMan::DecodeError(s32 ErrorCode)
#endif #endif
} }
s32 translateErrorCode(s32 native_error, bool isRW)
s32 WiiSockMan::getNetErrorCode(s32 ret, std::string caller, bool isRW)
{ {
#ifdef _WIN32 switch (native_error)
s32 errorCode = WSAGetLastError();
#else
s32 errorCode = errno;
#endif
if (ret >= 0)
return ret;
DEBUG_LOG(WII_IPC_NET, "%s failed with error %d: %s, ret= %d",
caller.c_str(), errorCode, DecodeError(errorCode), ret);
switch (errorCode)
{ {
case ERRORCODE(EMSGSIZE): case ERRORCODE(EMSGSIZE):
ERROR_LOG(WII_IPC_NET, "Find out why this happened, looks like PEEK failure?"); ERROR_LOG(WII_IPC_NET, "Find out why this happened, looks like PEEK failure?");
@ -87,7 +75,29 @@ s32 WiiSockMan::getNetErrorCode(s32 ret, std::string caller, bool isRW)
default: default:
return -1; return -1;
} }
}
s32 WiiSockMan::getNetErrorCode(s32 ret, std::string caller, bool isRW)
{
#ifdef _WIN32
s32 errorCode = WSAGetLastError();
#else
s32 errorCode = errno;
#endif
if (ret >= 0)
{
WiiSockMan::getInstance().setLastNetError(ret);
return ret;
}
INFO_LOG(WII_IPC_NET, "%s failed with error %d: %s, ret= %d",
caller.c_str(), errorCode, DecodeError(errorCode), ret);
s32 ReturnValue = translateErrorCode(errorCode, isRW);
WiiSockMan::getInstance().setLastNetError(ReturnValue);
return ReturnValue;
} }
WiiSocket::~WiiSocket() WiiSocket::~WiiSocket()
@ -413,7 +423,7 @@ void WiiSocket::update(bool read, bool write, bool except)
case IOCTLV_SO_SENDTO: case IOCTLV_SO_SENDTO:
{ {
u32 flags = Memory::Read_U32(BufferIn2 + 4); u32 flags = Memory::Read_U32(BufferIn2 + 0x04);
u32 has_destaddr = Memory::Read_U32(BufferIn2 + 0x08); u32 has_destaddr = Memory::Read_U32(BufferIn2 + 0x08);
char * data = (char*)Memory::GetPointer(BufferIn); char * data = (char*)Memory::GetPointer(BufferIn);
@ -448,50 +458,51 @@ void WiiSocket::update(bool read, bool write, bool except)
} }
case IOCTLV_SO_RECVFROM: case IOCTLV_SO_RECVFROM:
{ {
u32 flags = Memory::Read_U32(BufferIn + 4); u32 flags = Memory::Read_U32(BufferIn + 0x04);
char * data = (char *)Memory::GetPointer(BufferOut);
int data_len = BufferOutSize;
char *buf = (char *)Memory::GetPointer(BufferOut); sockaddr_in local_name;
int len = BufferOutSize; memset(&local_name, 0, sizeof(sockaddr_in));
struct sockaddr_in addr;
memset(&addr, 0, sizeof(sockaddr_in));
socklen_t fromlen = 0;
if (BufferOutSize2 != 0) if (BufferOutSize2 != 0)
{ {
fromlen = BufferOutSize2 >= sizeof(struct sockaddr) ? BufferOutSize2 : sizeof(struct sockaddr); WiiSockAddrIn* wii_name = (WiiSockAddrIn*)Memory::GetPointer(BufferOut2);
WiiSockMan::Convert(*wii_name, local_name);
} }
// Act as non blocking when SO_MSG_NONBLOCK is specified // Act as non blocking when SO_MSG_NONBLOCK is specified
forceNonBlock = ((flags & SO_MSG_NONBLOCK) == SO_MSG_NONBLOCK); forceNonBlock = ((flags & SO_MSG_NONBLOCK) == SO_MSG_NONBLOCK);
// recv/recvfrom only handles PEEK // recv/recvfrom only handles PEEK/OOB
flags &= SO_MSG_PEEK | SO_MSG_OOB; flags &= SO_MSG_PEEK | SO_MSG_OOB;
#ifdef _WIN32 #ifdef _WIN32
if (flags & MSG_PEEK){ if (flags & SO_MSG_PEEK){
unsigned long totallen = 0; unsigned long totallen = 0;
ioctlsocket(fd, FIONREAD, &totallen); ioctlsocket(fd, FIONREAD, &totallen);
ReturnValue = totallen; ReturnValue = totallen;
break; break;
} }
#endif #endif
int ret = recvfrom(fd, buf, len, flags, socklen_t addrlen = sizeof(sockaddr_in);
fromlen ? (struct sockaddr*) &addr : NULL, int ret = recvfrom(fd, data, data_len, flags,
fromlen ? &fromlen : 0); BufferOutSize2 ? (struct sockaddr*) &local_name : NULL,
ReturnValue = WiiSockMan::getNetErrorCode(ret, fromlen ? "SO_RECVFROM" : "SO_RECV", true); BufferOutSize2 ? &addrlen : 0);
ReturnValue = WiiSockMan::getNetErrorCode(ret, BufferOutSize2 ? "SO_RECVFROM" : "SO_RECV", true);
INFO_LOG(WII_IPC_NET, "%s(%d, %p) Socket: %08X, Flags: %08X, " INFO_LOG(WII_IPC_NET, "%s(%d, %p) Socket: %08X, Flags: %08X, "
"BufferIn: (%08x, %i), BufferIn2: (%08x, %i), " "BufferIn: (%08x, %i), BufferIn2: (%08x, %i), "
"BufferOut: (%08x, %i), BufferOut2: (%08x, %i)", "BufferOut: (%08x, %i), BufferOut2: (%08x, %i)",
fromlen ? "IOCTLV_SO_RECVFROM " : "IOCTLV_SO_RECV ", BufferOutSize2 ? "IOCTLV_SO_RECVFROM " : "IOCTLV_SO_RECV ",
ReturnValue, buf, fd, flags, ReturnValue, data, fd, flags,
BufferIn, BufferInSize, BufferIn2, BufferInSize2, BufferIn, BufferInSize, BufferIn2, BufferInSize2,
BufferOut, BufferOutSize, BufferOut2, BufferOutSize2); BufferOut, BufferOutSize, BufferOut2, BufferOutSize2);
if (BufferOutSize2 != 0) if (BufferOutSize2 != 0)
{ {
addr.sin_family = (addr.sin_family << 8) | (BufferOutSize2&0xFF); WiiSockAddrIn* wii_name = (WiiSockAddrIn*)Memory::GetPointer(BufferOut2);
Memory::WriteBigEData((u8*)&addr, BufferOut2, BufferOutSize2); WiiSockMan::Convert(local_name, *wii_name, addrlen);
} }
break; break;
} }

View File

@ -212,6 +212,8 @@ public:
s32 newSocket(s32 af, s32 type, s32 protocol); s32 newSocket(s32 af, s32 type, s32 protocol);
void addSocket(s32 fd); void addSocket(s32 fd);
s32 delSocket(s32 s); s32 delSocket(s32 s);
s32 getLastNetError() {return errono_last;}
void setLastNetError(s32 error) {errono_last = error;}
void clean() void clean()
{ {
@ -238,8 +240,9 @@ private:
WiiSockMan() {}; // Constructor? (the {} brackets) are needed here. WiiSockMan() {}; // Constructor? (the {} brackets) are needed here.
WiiSockMan(WiiSockMan const&); // Don't Implement WiiSockMan(WiiSockMan const&); // Don't Implement
void operator=(WiiSockMan const&); // Don't implement void operator=(WiiSockMan const&); // Don't implement
std::unordered_map<s32, WiiSocket> WiiSockets; std::unordered_map<s32, WiiSocket> WiiSockets;
s32 errono_last;
}; };
#endif #endif