C / C++ / MFC

led mike11-Jan-07 11:49

led mike

11-Jan-07 11:49

Are you behind a firewall?

led mike

11-Jan-07 11:51

11-Jan-07 11:51

Message Removed

led mike11-Jan-07 12:07

led mike

11-Jan-07 12:07

What does WSAaGetLastError report?

led mike

Mark Salsbery11-Jan-07 11:50

11-Jan-07 11:50

inet_addr() expects a dotted IP address. Is that what your getAddr() func returns?

If not, maybe you can use getaddrinfo() to resolve the name (google.com) to an address.

Mark

11-Jan-07 11:52

11-Jan-07 11:52

Message Removed

Mark Salsbery11-Jan-07 11:58

11-Jan-07 11:58

But what is the string value? An IP address "xxx.xxx.xxx.xxx" or a name "google.com"?

Mark

11-Jan-07 11:59

11-Jan-07 11:59

Message Removed

Mark Salsbery11-Jan-07 12:10

11-Jan-07 12:10

That won't work. You need to resolve google.com to an ip address (DNS works Smile | :)

)

getaddrinfo() will use DNS if you have it configured (if you can post here I imagine you do)

Here's some sample code ripped right from MSDN...

char* ip = "google.com";
char* port = "80";
struct addrinfo aiHints;
struct addrinfo *aiList = NULL;
int retVal;
 
memset(&aiHints, 0, sizeof(aiHints));
aiHints.ai_family = AF_INET;
aiHints.ai_socktype = SOCK_STREAM;
aiHints.ai_protocol = IPPROTO_TCP;
 
//--------------------------------
// Call getaddrinfo(). If the call succeeds,
// the aiList variable will hold a linked list
// of addrinfo structures containing response
// information about the host
if ((retVal = getaddrinfo(ip, port, &aiHints, &aiList)) != 0) 
{
  printf("getaddrinfo() failed.\n");
}
else
{
  printf("getaddrinfo() succeeded.\n");
}

On success, aiList will have all the info you need to make a connection.

Mark

11-Jan-07 12:22

11-Jan-07 12:22

Message Removed

Mark Salsbery11-Jan-07 12:43

11-Jan-07 12:43

Anywhere before you call connect(), in place of the code you posted that builds the address
(my_addr).

Where I used
char* ip = "google.com";
you can use your getAddr()

Where I used
char* port = "80";
you can either convert your returned getPort() to a string or use the service name instead
char* port = "http";

Mark Salsbery11-Jan-07 13:47

11-Jan-07 13:47

A deprecated solution...

   sockaddr_in my_addr;
   memset(&my_addr, 0, sizeof(sockaddr_in));
...
   HOSTENT *pHostent = ::gethostbyname(getAddr( ));
   if (pHostent && pHostent->h_addrtype == AF_INET)
   {
      my_addr.sin_family = AF_INET;
      my_addr.sin_port = htons( getPort( ) );
      my_addr.sin_addr = *(in_addr *)(pHostent->h_addr_list[0]);
 
      if ( connect( hSocket, ( const struct sockaddr* ) &my_addr, sizeof( my_addr ) ) == SOCKET_ERROR ) 
      { 
         return( false ); 
      }
   }
   else
   {
      return( false ); 
   }

11-Jan-07 14:37

11-Jan-07 14:37

Message Removed

11-Jan-07 15:05

11-Jan-07 15:05

Message Removed

11-Jan-07 15:12

11-Jan-07 15:12

Message Removed

Mark Salsbery11-Jan-07 15:29

11-Jan-07 15:29

if ( connect( hSocket, aiList[0].ai_addr, sizeof( aiList ) ) == -1 ) { return( false ); }

11-Jan-07 15:35

11-Jan-07 15:35

Message Removed

Mark Salsbery11-Jan-07 15:45

11-Jan-07 15:45

ack sorry

if ( connect( hSocket, aiList[0].ai_addr, sizeof( sockaddr ) ) == -1 ) { return( false ); }

If that doesn't work:

check aiList[0].ai_addr in debugger - is it valid address/port?
hSocket valid? Created as TCP socket?
WSAGetLastError() returns what after failed connect()?

11-Jan-07 15:50

11-Jan-07 15:50

Message Removed

Mark Salsbery11-Jan-07 15:54

11-Jan-07 15:54

10038

Socket operation on nonsocket.
An operation was attempted on something that is not a socket. Either the socket handle parameter
did not reference a valid socket, or for select, a member of an fd_set was not valid

11-Jan-07 15:56

11-Jan-07 15:56

Message Removed

11-Jan-07 16:00

11-Jan-07 16:00

Message Removed

Mark Salsbery11-Jan-07 16:35

11-Jan-07 16:35

Armond Sarkisian wrote:
Can I zip up all four files and send it to you?

Before we do that, try this...I just connected to google.com - 3 different ip addresses returned
by DNS. step through this in the debugger. Notice I'm using ALL the info returned by getaddrinfo
to do the socket() and connect() calls. Does this all succeed for you?

char* ip = "google.com";
char* port = "80";
struct addrinfo aiHints;
struct addrinfo *aiList = NULL;
int retVal;
sockaddr addr;
 
memset(&aiHints, 0, sizeof(aiHints));
aiHints.ai_family = AF_INET;
aiHints.ai_socktype = SOCK_STREAM;
aiHints.ai_protocol = IPPROTO_TCP;
 
if ((retVal = getaddrinfo(ip, port, &aiHints, &aiList)) != 0) 
{
  printf("getaddrinfo() failed.\n");
}
else
{
	int SockErr;
 
	addrinfo *pAddrInfo = aiList;
	while (pAddrInfo)
	{
		addr = *pAddrInfo->ai_addr;
 
		SOCKET s = socket(pAddrInfo->ai_family, pAddrInfo->ai_socktype, pAddrInfo->ai_protocol);
 
		if (s != INVALID_SOCKET)
		{
			if (SOCKET_ERROR == connect(s, &addr, sizeof(sockaddr)))
			{
				SockErr = WSAGetLastError();
			}
			closesocket(s);
		}
		else
		{
			SockErr = WSAGetLastError();
		}
 
		pAddrInfo = pAddrInfo->ai_next;
	}
}

Same code without /pre tags...

char* ip = "google.com";
char* port = "80";
struct addrinfo aiHints;
struct addrinfo *aiList = NULL;
int retVal;
sockaddr addr;

memset(&aiHints, 0, sizeof(aiHints));
aiHints.ai_family = AF_INET;
aiHints.ai_socktype = SOCK_STREAM;
aiHints.ai_protocol = IPPROTO_TCP;

if ((retVal = getaddrinfo(ip, port, &aiHints, &aiList)) != 0)
{
printf("getaddrinfo() failed.\n");
}
else
{
int SockErr;

addrinfo *pAddrInfo = aiList;
while (pAddrInfo)
{
addr = *pAddrInfo->ai_addr;

SOCKET s = socket(pAddrInfo->ai_family, pAddrInfo->ai_socktype, pAddrInfo->ai_protocol);

if (s != INVALID_SOCKET)
{
if (SOCKET_ERROR == connect(s, &addr, sizeof(sockaddr)))
{
SockErr = WSAGetLastError();
}
closesocket(s);
}
else
{
SockErr = WSAGetLastError();
}

pAddrInfo = pAddrInfo->ai_next;
}
}

Mark Salsbery11-Jan-07 16:02

11-Jan-07 16:02

At the socket call I assume. WSAGetLastError() gives you what after that call?

11-Jan-07 16:07

11-Jan-07 16:07

Message Removed

11-Jan-07 16:24