Function problem (Winsock)

I having a problem to start winsock2, check it out (It's dosn't connect):

Code:

#include <winsock2.h>
#include <windows.h>
#include <stdio.h>


int connect()
{

	int sockfd;
	struct sockaddr_in dest_addr;

		WSADATA info;
	    if (WSAStartup(MAKEWORD(1,1), &info) != 0)
  		  MessageBox(NULL, "Cannot initialize WinSock!", "WSAStartup", MB_OK);
		else
			{
			MessageBox(NULL, "Cannot initialize WinSock!", "ERROR 216", MB_OK);
			return(0);
		}
			
	    sockfd = socket(AF_INET,SOCK_STREAM,0);
		dest_addr.sin_family = AF_INET;
		dest_addr.sin_port = htons(6667);
    		dest_addr.sin_addr.s_addr = inet_addr("bombers.no-ip.org");
		if (connect(sockfd, (struct sockaddr *)&dest_addr, sizeof(struct sockaddr)) >= 0)
			printf("\n%d", "Connected succesfully"); 
		else
		{
			MessageBox(NULL, "Cannot initialize WinSock!", "ERROR 219", MB_OK);
			return(0);
		}
}


int main()
{
connect();
}

Read this and edit your post. While you're editting, specify the problem you are having with the posted code.

gg

Here is the full source again, It's dosn't connect :\ Anyone?

Code:

#include <winsock2.h>
#include <windows.h>
#include <stdio.h>


int IRCconnect()
{
	int sockfd;
	struct sockaddr_in dest_addr;

		WSADATA info;
	    if (WSAStartup(MAKEWORD(1,1), &info) != 0)
  		  MessageBox(NULL, "Cannot initialize WinSock!", "WSAStartup", MB_OK);
		else
			{
			//MessageBox(NULL, "Cannot initialize WinSock!", "ERROR 216", MB_OK);
			return(0);
		}
			
	    sockfd = socket(AF_INET,SOCK_STREAM,0);
		dest_addr.sin_family = AF_INET;
		dest_addr.sin_port = htons(6667);
    		dest_addr.sin_addr.s_addr = inet_addr("80.105.107.170");
		if (connect(sockfd, (struct sockaddr *)&dest_addr, sizeof(struct sockaddr)) >= 0)
			printf("\n%d", "Connected succesfully"); 
		else
		{
			printf("%d",sockfd);
			return(0);
		}
}



int main()
{
	IRCconnect();
}

If you say that is working you, can you please write here your code?

Well, i think it's working:


--------------------------------------------------------------------
Microsoft Windows 2000 [Version 5.00.2195]
(C) Copyright 1985-2000 Microsoft Corp.

C:\>sock

4325424
C:\>

--------------------------------------------------------------------


The source code, can you tell me if the
if (WSAStartup(MAKEWORD(1,1), &info) != 0)
and the else are correct?


Source code:

Code:

#include <stdio.h>
#include <winsock2.h>
#include <windows.h>

int IRCconnect()
{
	int sockfd;
	struct sockaddr_in dest_addr;

		WSADATA info;
	    if (WSAStartup(MAKEWORD(1,1), &info) != 0) {
	MessageBox(NULL, "Cannot initialize WinSock!", "WSAStartup", MB_OK);
	return(1);
	}
		else
			{
			MessageBox(NULL, "Winsock started!", "OK", MB_OK);
		}
			
	    sockfd = socket(AF_INET,SOCK_STREAM,0);
		dest_addr.sin_family = AF_INET;
		dest_addr.sin_port = htons(6667);
    		dest_addr.sin_addr.s_addr = inet_addr("80.105.107.170");
		if (connect(sockfd, (struct sockaddr *)&dest_addr, sizeof(struct sockaddr)) >= 0)
			printf("\n%d", "Connected succesfully"); 
		else
		{
			//MessageBox(NULL, "Cannot initialize WinSock!", "ERROR 219", MB_OK);
			printf("\n%d", "Connected...");
		}
}



int main()
{
	IRCconnect();
}

Pandora, USE CODE TAGS when posting code.

gg

NP

Just say if it's ok... :\

Code:

#include <stdio.h>
#include <winsock2.h>
#include <windows.h>

int IRCconnect()
{
int sockfd;
struct sockaddr_in dest_addr;

WSADATA info;
if (WSAStartup(MAKEWORD(1,1), &info) != 0) {
MessageBox(NULL, "Cannot initialize WinSock!", "WSAStartup", MB_OK);
return(1);
}
else
{
MessageBox(NULL, "Winsock started!", "OK", MB_OK);
}

sockfd = socket(AF_INET,SOCK_STREAM,0);
dest_addr.sin_family = AF_INET;
dest_addr.sin_port = htons(6667);
dest_addr.sin_addr.s_addr = inet_addr("80.105.107.170");
if (connect(sockfd, (struct sockaddr *)&dest_addr, sizeof(struct sockaddr)) >= 0)
printf("\n%d", "Connected succesfully"); 
else
{
//MessageBox(NULL, "Cannot initialize WinSock!", "ERROR 219", MB_OK);
printf("\n%d", "Connected...");
}
}



int main()
{
IRCconnect();
}

Shouldn't you tell your printf()'s to output strings instead of integers?

No, he don't say anything about printf & string insted integer...

>>printf("\n%d", "Connected succesfully");
This should be
>>printf("\n%s", "Connected succesfully");

or better still

>>printf("\nConnected succesfully");

or maybe even

>>puts("Connected succesfully");

But you missed my point, when i run it, it's tell me that the winsock culd not start...

>>> it's tell me that the winsock culd not start...

Then it is also telling you why. From MSDN...

>>>

Code:

Return Values
The WSAStartup function returns zero if successful. Otherwise, it returns one of the error codes listed below.

*** clipped ***

Error Codes
WSASYSNOTREADY Indicates that the underlying network subsystem is not ready for network communication. 
WSAVERNOTSUPPORTED The version of Windows Sockets support requested is not provided by this particular Windows Sockets implementation. 
WSAEINPROGRESS A blocking Windows Sockets 1.1 operation is in progress. 
WSAEPROCLIM Limit on the number of tasks supported by the Windows Sockets implementation has been reached. 
WSAEFAULT The lpWSAData is not a valid pointer.

<<<

... see which it is and fix it!

Originally posted by Pandora
But you missed my point, when i run it, it's tell me that the winsock culd not start...

I didn't miss it, you did state that before, you said:

It's dosn't connect

Accurate questions get accurate answers (in most cases )