Assignment makes integer from pointer without a cast warning

Question

Can anybody explain why the following code gives the following warning and how to remedy it?

warning: assignment makes integer from pointer without a cast [enabled by default]

Here is the code:

int tcp_socket(void)
{
    int s;

    if ((s = socket(PF_INET, SOCK_STREAM, 0)) == -1) {
        _logf(LOG_DEBUG, "LOG %s:%d (%s) - %s", __FILE__, __LINE__, __func__, "Error creating socket");
        return -1;
    }
    return s;       
}

int main(int argc, char **argv)
{
    int socket;
    socket = tcp_socket;
    if(socket == -1) {
        _logf(LOG_INFO, "LOG %s:%d (%s) - %s", __FILE__, __LINE__, __func__, "Error creating socket, exiting...");
        exit(-1);
    } 
}

0xF1 · Accepted Answer · 2013-09-10 12:07:15Z

8

Correct this:

socket = tcp_socket;

to

socket = tcp_socket();

tcp_socket() is a function call, while tcp_socket is a pointer to the function location in memory.

answered Sep 10, 2013 at 12:07

0xF1

6,1563 gold badges31 silver badges51 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

0xF1 Over a year ago

@Tomcelic : Based on your reputation, I think either you were in hurry, or frustrated (of long hours of coding) to do this kind of mistake. :-)

TomSelleck Over a year ago

I'm actually newish to C, she is a cold and cruel mistress !

0decimal0 Over a year ago

@Tomcelic Be patient with her and one day she will take you to the wonderland :)

Collectives™ on Stack Overflow

Assignment makes integer from pointer without a cast warning

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related