2015-05-04 230 views
1

我目前正在尝试实现一个类来创建和使用TCP连接。 (我在C++中很新,所以有可能我忘了一些明显的东西,尽管我一直在认真检查我的代码)10038 wsa socket error recv

请注意,稍后我将实现第二个类来处理新的连接,是代码的一些部分不完整/安排,我希望这不会打扰你,而试图理解它。

这里是我的问题:

服务器端和客户端似乎做工精细(我总是检查我听,捆绑,连接功能:没有错误),我显然能够与发送数据客户端,但是当我尝试从服务器读取它时,我得到WSA错误10038(这意味着我在一个无效的套接字上执行recv())。我一直在查找它几天,似乎无法找到我的错误。除了代码中的测试之外,我用“netstat -an -p tcp”来检查连接的状态,一切都很正常,但服务器不会收到数据。

注意:我正在使用Visual Studio 6(是的,它很旧,但它是强制性的,所以......)和winsock,而不是winsock2。

如果我忘记了一些问题,我想尽可能地完整,但忘记一些事情并不是不可能的。

这里是我的源代码,我试图把它切到必要的,但恐怕错误可以几乎任何地方:

CmTcpTransport.cpp

/** 
Initializes the TCP connexion (socket) in CLIENT or SERVER mode. 
It does not connect yet. 

@param strIPAddress: IP Adress to reach 
     nPort: Port to connect to 
     hSocket: client Socket if SERVER mode 
     nTcpMode: mode for this connexion SERVER(1) or CLIENT(2)  

@return an int to get the exit point of the function 
*/ 
int CmTcpTransport::initialize(std::string strIPAddress, unsigned int nPort,   SOCKET hSocket, int nTcpMode) 
{ 
m_nTcpMode = nTcpMode; 

cout << "Creating TCP connexion..." << endl << endl; 

if (nTcpMode == 1) // SERVER 
{ 

    serveraddr.sin_family = AF_INET; // address family Internet 
    serveraddr.sin_port = htons (nPort); // Port to connect on 
    serveraddr.sin_addr.s_addr = inet_addr (strIPAddress.c_str()); // Target IP 

    cout << "SERVER: Retreaving socket information..." << endl; 

    if (hSocket == INVALID_SOCKET) 
    { 
     return 102; // The received socket is invalid 
     cout << "SERVER: Socket creation failed" << endl; 
    } 

    serverSocket = hSocket; 

    cout << "SERVER: Listening socket received" << endl; 

    if (bind(serverSocket, (struct sockaddr *)&serveraddr, sizeof(serveraddr)) == SOCKET_ERROR) 
    { 
     return 103; // Couldn't bind 
     cout << "SERVER: Socket binding failed" << endl; 
    } 
    else { 
     cout << "SERVER: Binding successful" << endl; 
    } 

    if (listen(serverSocket, 1) == SOCKET_ERROR) 
    { 
     return 103; // Couldn't listen 
     cout << "SERVER: Socket listening failed" << endl; 
    } 
    else { 
     cout << "SERVER: Listening to socket" << endl; 
    } 

} 
else // CLIENT 
{ 
    serveraddr.sin_family = AF_INET; // address family Internet 
    serveraddr.sin_port = htons (nPort); // Port to connect on 
    serveraddr.sin_addr.s_addr = inet_addr (strIPAddress.c_str()); // Target IP 

    cout << "CLIENT: Creating client socket..." << endl; 

    clientSocket = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP); // CREATE CLIENT SOCKET 

    if (clientSocket == INVALID_SOCKET) 
    { 
     return 104; // Couldn't create the socket 
     cout << "CLIENT: Socket creation failed" << endl; 
    } 

    cout << "CLIENT: Client socket created" << endl; 


} 

cout << "Initialization succeeded." << endl << endl << endl; 
return 105; 
} 

/** 
Creates the connexion to TCP 

@param none   
@return an int to get the exit point of the function 
*/ 
int CmTcpTransport::connectTcp() 
{ 


if (m_nTcpMode == 1) // SERVER 
{ 
    cout << "Waiting for connexion..." << endl; 
    int clientLen(0); 
    clientLen = sizeof(clientaddr); 


    if ((clientSocket = accept(serverSocket, (SOCKADDR *)&clientaddr, &clientLen) == INVALID_SOCKET)) 
    { 
     cout << "SERVER: Connexion failed." << endl; 
     return 202;; // Couldn't listen 
    } 
    else { 
     cout << "SERVER: Connexion established." << endl << endl << endl; 
     if (clientSocket == SOCKET_ERROR) 
     { 
      printf("socket became invalid after connexion:: %d\n", WSAGetLastError()); 
     } 
     else { 
       cout << "SOCKET STILL VALID AND WORKING AFTER CONNEXION" << endl; 
     } 
     return 201; 
    } 


} 
else // CLIENT 
{ 
    cout << "Connecting..." << endl; 
    if (connect(clientSocket, (SOCKADDR *)&serveraddr, sizeof(serveraddr)) == SOCKET_ERROR) 
    { 
     cout << "CLIENT: Connexion failed." << endl << endl << endl; 
     return 202;; // Couldn't connect 
    } 
    else 
    { 
     cout << "CLIENT: Connexion to server succeeded." << endl << endl <<   endl; 
    } 
    return 203; 
} 
} 

/** 
Receives a buffer of bytes from the TCP 

@param buffer: buffer to store the data 
     bufLen: size of the buffer 

@return nbChar: the result of the recv function 
if no error occurs, recv returns the number of bytes received 
else, it returns a value of SOCKET_ERROR 
*/ 
int CmTcpTransport::recvTcp(char *buffer, int bufLen) 
{ 
if (clientSocket == SOCKET_ERROR) 
     { 
      printf("socket became invalid before receive:: %d\n", WSAGetLastError()); 
     } 
     else { 
       cout << "SOCKET STILL VALID AND WORKING BEFORE RECEIVE" <<   endl; 
     } 

int nbChar = recv(clientSocket, buffer, bufLen, 0); 
if (nbChar == SOCKET_ERROR) 
{ 
    printf("recv failed: %d\n", WSAGetLastError()); 
    cout << "No data received." << endl << endl << endl; 
    return nbChar; 
} 
else { 
    buffer[nbChar]=0; // Gestion de la taille du buffer 
    cout << "Data received: " << buffer << endl << endl << endl; 
    return nbChar; 
} 
} 
/** 
Sends a buffer of bytes to the TCP 

@param buffer: buffer containing the data 
     bufLen: size of the buffer 

@return nbChar: the result of the send function 
if no error occurs, recv returns the number of bytes sent 
else, it returns a value of SOCKET_ERROR 
*/ 
int CmTcpTransport::sendTcp(char *buffer, int bufLen) 
{ 

int nbChar = send(clientSocket, buffer, bufLen,0); 
if (nbChar == SOCKET_ERROR) 
{ 

    cout << "No data sent." << endl << endl << endl; 
    return nbChar; 
} 
else { 
    cout << "Data sent: " << buffer << endl << endl << endl; 
    return nbChar; 
} 
} 

服务器main.cpp中

int main(int argc, char* argv[]) 
{ 
// ----------------------- WINSOCK ---------------------------- 

cout << "Initializing winsock library..." << endl; 

// Start up Winsock 
WSADATA wsadata; 

int error = WSAStartup(0x101, &wsadata); 

// Error ? 
if (error) 0; 

// Try Winsock Version ? 
if (wsadata.wVersion != 0x101) 
{ 
    WSACleanup(); // Clean up Winsock 
    return 101; 
} 

cout << "Initialization successful" << endl << endl << endl; 


// ----------------------- SERVER --------------------------- 

char buffer[3000]; 

int testRecv; // return of the recv function 

cout << "SERVER" << endl << endl << endl; 

// Simulate the activity of the TcpListener transfering a listening Socket for the client connexion 
SOCKET testSocket; 
testSocket = socket(AF_INET, SOCK_STREAM,IPPROTO_TCP); 

// Creating a new TCP transport 
CmTcpTransport server; 

// Initialize and connect the TCP transport 
server.initialize("127.0.0.1", 10000, testSocket, TTCP_MODE_SERVER); 

server.connectTcp(); 

// Try to receive data 
testRecv = server.recvTcp(buffer, sizeof(buffer)); 

cout << testRecv << endl; 

// Disconnect 
server.disconnectTcp(); 


return 0; 
} 

客户的main.c第

int main(int argc, char* argv[]) 
{ 

// ----------------------- WINSOCK ---------------------------- 

cout << "Initializing winsock library..." << endl; 

// Start up Winsock 
WSADATA wsadata; 

int error = WSAStartup(0x0202, &wsadata); 

// Error ? 
if (error) 0; 

// Try Winsock Version ? 
if (wsadata.wVersion != 0x0202) 
{ 
    WSACleanup(); // Clean up Winsock 
    return 101; 
} 

cout << "Initialization successful" << endl << endl << endl; 


// ----------------------- CLIENT --------------------------- 

char buffer[10000]; 

cout << "CLIENT" << endl << endl << endl; 

// this Socket won't be used as the TCP transport is in client mode 
SOCKET test; 
test = socket(AF_INET, SOCK_STREAM,IPPROTO_TCP); 

// Create, initialize and connect the client TCP transport 
CmTcpTransport client; 

client.initialize("127.0.0.1", 10000, test, TTCP_MODE_CLIENT); 

client.connectTcp(); 

// Copy the data in the buffer and send it 
strcpy(buffer, "Test donnees"); 

client.sendTcp(buffer, sizeof(buffer)); 

client.disconnectTcp(); 

return 0; 
} 

这里是我的控制台的屏幕截图,当我运行两个电源:

http://www.hostingpics.net/viewer.php?id=892752Consoles.jpg

感谢你的帮助,我意识到这是很多读/过程,我很欣赏努力!

+1

这通常是一个调试器是什么,不是关于SO的问题。 –

+0

Smal编辑:我已经纠正了事实,即我的“返回”的错误情况之前,我的“错误...”,这是一个旧的,但我的代码工作版本。而通过工作,我的意思是我达到了recv()的错误10038。 –

回答

2

我认为你是不正确的支架上accept无效的新袜子ID:

if ((clientSocket = accept(serverSocket, (SOCKADDR *)&clientaddr, &clientLen) == INVALID_SOCKET))

尝试将其更改为:

if ((clientSocket = accept(serverSocket, (SOCKADDR *)&clientaddr, &clientLen)) == INVALID_SOCKET)

+0

谢谢!有效。我确定这是荒谬的,我想这是一个很好的教训。 –

+0

谢谢,Prabhu,我用同样的问题撕掉了我的头发! –