GetLocalAddress

[Laurent]

Doesn't work at all? Can you show me your code?

[TheMagicNumber]

It gave the same result.

Code: [Select]

sf::IpAddress GetLocalAddress()
{
// The method here is to connect a UDP socket to anyone (here to localhost),
// and get the local socket address with the getsockname function.
// UDP connection will not send anything to the network, so this function won't cause any overhead.

sf::IpAddress localAddress;

// Create the socket
SOCKET sock = socket(PF_INET, SOCK_DGRAM, 0);
if (sock == INVALID_SOCKET)
return localAddress;

// Connect the socket to localhost on any port
sockaddr_in address = CreateAddress(INADDR_LOOPBACK, 0);
if (connect(sock, reinterpret_cast<sockaddr*>(&address), sizeof(address)) == -1)
{
closesocket(sock);
return localAddress;
}

char buffer[] = {0};
send(sock, buffer, 1, 0);

// Get the local address of the socket connection
int size = sizeof(address);
if (getsockname(sock, reinterpret_cast<sockaddr*>(&address), &size) == -1)
{
closesocket(sock);
return localAddress;
}

// Close the socket
closesocket(sock);

// Finally build the IP address
localAddress = sf::IpAddress(ntohl(address.sin_addr.s_addr));

return localAddress;
}

[Laurent]

Ok... last try: check the return value of send, just to be sure (compare it against SOCKET_ERROR).

[TheMagicNumber]

Done, it didn't return SOCKET_ERROR.

[Laurent]

Can you add WSA initialization to your own code?

Code: [Select]

WSADATA init;
WSAStartup(MAKEWORD(2, 2), &init);

[TheMagicNumber]

Just put that at the beginning of main() and ran, same result. D:

I'll be back later.

[Laurent]

I've run out of ideas

[TheMagicNumber]

:?

[Beernutts]

I am having this same issue at home (using 2.0), and I've dealt with PC's with multiple IP address' in the past.  I didn't have time to debug it, but I'll look at it tonight and get back to you here.

I do think the API should be modified since often PC's are on multiple networks.  In fact, the PC I use at work has 3 IP address' to 3 different networks.  For an simple test app I wrote, I included this bit of code (Windows only) to select which IP address to use (see below).

I would love for the GetLocalIpAddress to maybe pass an index, so we can get all the possible local IP addresses, or some other method of getting all the IP addresses the PC has.

Thanks.

Code: [Select]

/* retreive our IP address */
    {
        PMIB_IPADDRTABLE pIPAddrTable;
        DWORD dwSize = 0;

        pIPAddrTable = (MIB_IPADDRTABLE*) malloc( sizeof( MIB_IPADDRTABLE) );
       
        /* Make an initial call to GetIpAddrTable to get the
         necessary size into the dwSize variable */
        if (GetIpAddrTable(pIPAddrTable, &dwSize, 0) == ERROR_INSUFFICIENT_BUFFER)
        {
            free( pIPAddrTable );
            pIPAddrTable = (MIB_IPADDRTABLE *) malloc ( dwSize );
        }

        /* Make a second call to GetIpAddrTable to get the
         actual data we want */
        GetIpAddrTable( pIPAddrTable, &dwSize, 0 );
       
        /* fill in our addrs */
        for (i = 0; i < pIPAddrTable->dwNumEntries; i++)
        {
            if ((pIPAddrTable->table[i].dwAddr != 0) &&
                (pIPAddrTable->table[i].dwAddr != LOCALHOST))
            {
                addr[NumAddrs].s_addr = pIPAddrTable->table[i].dwAddr;
                NumAddrs++;
            }
        }          
    }  
   
    /* ask users which IP address to use if more than one */
    while ((NumAddrs > 1) && bContinue)
    {
        C8 c8Char;
       
        printf("Choose Which IP address to listen on:\n");
       
        for (i = 0; i < NumAddrs; i++)
        {
            printf("%d: IP %d.%d.%d.%d\n", i+1, PRINT_IP(addr[i].s_addr));
        }
       
        c8Char = getch() - '0';
       
        if ((c8Char > 0) && (c8Char < NumAddrs+1))
        {
            bContinue = FALSE;
            gu32IpIndex = c8Char-1;
            printf("------------------------------\n");
        }
    }

[Beernutts]

I am having this same issue at home (using 2.0), and I've dealt with PC's with multiple IP address' in the past.  I didn't have time to debug it, but I'll look at it tonight and get back to you here.

I do think the API should be modified since often PC's are on multiple networks.  In fact, the PC I use at work has 3 IP address' to 3 different networks.  For an simple test app I wrote, I included this bit of code (Windows only) to select which IP address to use (see below).

I would love for the GetLocalIpAddress to maybe pass an index, so we can get all the possible local IP addresses, or some other method of getting all the IP addresses the PC has.

Thanks.

So, I tried to create a solution to this in my app by checking if the GetLocalAddress() was LOCALHOST, and, instead creating an sf::IpAddress with my real IP address (using code close to that above).  However, using that code doesn't seem to allow me to send any packets out the ethernet interface I'm trying to use.  Checking Wireshark, there's no data being sent on my ethernet interface (using UdpSocket here).

So, until sf:Network is modified to properly handle this situation (perhaps you need to modify Bind() to include the IP address to Bind the socket to as well as port assuming there will be multiple possible network interfaces), I'm going to implement my solution in good old sockets.