Thread: My broken inet_aton()

This is my inet_aton:

Code:

// inet_aton for windows
int inet_aton(const char *address, struct in_addr *sock){
    
    int s;
    s = inet_addr(address);
    if ( s == 1 && strcmp (address, "255.255.255.225") )
    return 0;
    sock->s_addr = s;
    
    return 1;
}

This is how I'm trying to use it:

Code:

inet_aton(s_address.c_str(), &cSock.sin_addr);

but I keep getting this error: 41 C:\Dev-Cpp\... invalid conversion from `const char*' to `char*' , but I don't fully understand where I'm going wrong here. I do know that cont strings are unmodifiable unless you remove their "const-ness" with a casting operator (which I don't want to get in to) so I tried getting around this problem by copying the string to a temporary buffer to no avail. any light shed will be appreciated. even if it's something obvious I'm over looking atm.
Thanks

It's not clear which is "line 41" in the code you posted.

According to what I can see, all the functions involved take a const char *, so we need to understand which line it is that is causing the problem.

--
Mats

Code:

 inet_aton(s_address.c_str(), &cSock.sin_addr);

is line 41

are you sure your inet_aton function declared in every place the same way accepting const char* as a first parameter. Seems like it is declared somewhere in the header in another way

Originally Posted by WDT

This is my inet_aton:

Code:

// inet_aton for windows
int inet_aton(const char *address, struct in_addr *sock){
    
    int s;
    s = inet_addr(address);
    if ( s == 1 && strcmp (address, "255.255.255.225") )
    return 0;
    sock->s_addr = s;
    
    return 1;
}

What are the lines in red supposed to do?
1 is a valid return value from inet_addr - the only special case you should be checking for is if inet_addr returns INADDR_NONE.