Socket Programming in C, server code with an error

Question

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
#include <arpa/inet.h>
#include <sys/types.h>
#include <sys/socket.h>
#include <netinet/in.h>

int main(int argc, char *argv[]){
    // established the socket
    char inputBuffer[256] = {};
    char message[] = {"Hi this is the server.\n"};
    int sockfd = 0;
    int forClientSocketfd = 0;
    sockfd = socket(AF_INET, SOCK_STREAM, 0);
    if(sockfd == -1) printf("Fail to create the socket.");

    // socket connection
    struct sockaddr_in serverInfo, clientInfo;
    int addrlen = sizeof(clientInfo);
    bzero(&serverInfo, sizeof(serverInfo));

    serverInfo.sin_family = PF_INET;
    serverInfo.sin_addr.s_addr = INADDR_ANY;
    serverInfo.sin_port = htron(10024);
    bind(sockfd, (struct sockaddr *) &serverInfo, sizeof(serverInfo));
    listen(sockfd, 5);

    while(1){
        forClientSocketfd = accept(sockfd, (struct sockaddr*) &clientInfo, &addrlen);
        send(forClientSocketfd, message, sizeof(message), 0);
        recv(forClientSocketfd, inputBuffer, sizeof(inputBuffer), 0);
        printf("Received from client: %s\n", inputBuffer);
    }

    return 0;
}

This is the code for socket programming that I seen through from the net. when I compiled it, it throw the error message as below. Having no idea what's going on, even though searching through the internet. p.s. Client operate as normal.

enter image description here

I did a search for htron and couldn't find it. Did you mean htons? — 001
– 001, Commented Sep 29, 2019 at 13:08
Please show textual information as text, instead of picure of text. meta.stackoverflow.com/a/285557/7733418 — Yunnosch
– Yunnosch, Commented Sep 29, 2019 at 13:12
YuCheng Liao. The htons / htron typo is exteremely plausible (@JohnnyMopp), see linux.die.net/man/7/ip and search that page for the "sin_port " identifier you are assigning the result to. The question seems therefor typo-caused. Please give feedback on whether in your opinion there must be a different explanation and why. — Yunnosch
– Yunnosch, Commented Sep 29, 2019 at 13:22

Mahesh Jamdade · Accepted Answer · 2019-11-21 02:39:54Z

3

you have a typo on line number 24 it should be htons and not htron

htons()

The htons function takes a 16-bit number in host byte order and returns a 16-bit number in network byte order used in TCP/IP networks(the AF_INET or AF_INET6 address family). The htons function can be used to convert an IP port number in host byte order to the IP port number in network byte order

also add the stdio header file to your code to remove the other warnings heres the final corrected code with no warnings or errors.

#include <string.h>
#include <unistd.h>
#include <arpa/inet.h>
#include <sys/types.h>
#include <sys/socket.h>
#include <netinet/in.h>
#include <stdio.h>

int main(int argc, char *argv[])
{
    // established the socket
    char inputBuffer[256] = {};
    char message[] = {"Hi this is the server.\n"};
    int sockfd = 0;
    int forClientSocketfd = 0;
    sockfd = socket(AF_INET, SOCK_STREAM, 0);
    if (sockfd == -1)
        printf("Fail to create the socket.");

    // socket connection
    struct sockaddr_in serverInfo, clientInfo;
    int addrlen = sizeof(clientInfo);
    bzero(&serverInfo, sizeof(serverInfo));

    serverInfo.sin_family = PF_INET;
    serverInfo.sin_addr.s_addr = INADDR_ANY;
    serverInfo.sin_port = htons(10024);
    bind(sockfd, (struct sockaddr *)&serverInfo, sizeof(serverInfo));
    listen(sockfd, 5);

    while (1)
    {
        forClientSocketfd = accept(sockfd, (struct sockaddr *)&clientInfo, &addrlen);
        send(forClientSocketfd, message, sizeof(message), 0);
        recv(forClientSocketfd, inputBuffer, sizeof(inputBuffer), 0);
        printf("Received from client: %s\n", inputBuffer);
    }

    return 0;
}

edited Nov 21, 2019 at 2:39

answered Sep 29, 2019 at 13:17

Mahesh Jamdade

20.9k11 gold badges133 silver badges150 bronze badges

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

7 Comments

Yunnosch Over a year ago

For one, the answer seems to pick up the idea from another users comment (not mine). Also, there is still some assumption, second guessing OPs intentions (though very plausibly), it would therefor be better to first clarify with OP, you do have the commenting privilege. Finally if you are correct, then the whole question is a typoe and hence off-topic.

Mahesh Jamdade Over a year ago

glad it helped! please use a nice editor/ide to get a detailed info of warnings and errors I recommend vs code

Mahesh Jamdade Over a year ago

@Yunnosch first thing commenting is for those if you are not sure about the answer/unclear po need some clarification about the topic,And I have provided the answer after trying it out myslef to make sure I add something new and it works as I claim

Yunnosch Over a year ago

I am aware of everything in your comment and stand with everything I wrote in mine.

alk Over a year ago

Linking to and quoting from windows documentation for a question not related to windows at all is not the best idea.

|

Collectives™ on Stack Overflow

Socket Programming in C, server code with an error

1 Answer 1

7 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

7 Comments

Your Answer

Sign up or log in

Post as a guest

Related