0

The exercise ask to resend the messages back to the client.

This exercise with some pieces of code were provided by our teacher.

I don't know why the last message that the program send does not appear. I can't understand where is the error.

I have changed the read adding the & before x. Now the x value is displayed correctly but the last value is still missing

When I insert only one value the message is missing. The server is always running and I don't know how to fix this.

The server code is:

#include <stdio.h>      
#include <sys/types.h>
#include <sys/socket.h>   
#include <netdb.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
#include <time.h> 

/*const*/ char MESSAGE[100] = "";
char buff[100];

int main(int argc, char *argv[]) {

    int simpleSocket = 0;
    int simplePort = 0;
    int returnStatus = 0;
    struct sockaddr_in simpleServer;
    
    if (argc != 2) {

        fprintf(stderr, "Usage: %s <port>\n", argv[0]);
        exit(1);

    }

    simpleSocket = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);

    if (simpleSocket == -1) {

        fprintf(stderr, "Could not create a socket!\n");
        exit(1);

    }
    else {
        fprintf(stderr, "Socket created!\n");
    }

    /* retrieve the port number for listening */
    simplePort = atoi(argv[1]);

    /* setup the address structure */
    /* use INADDR_ANY to bind to all local addresses  */
    memset(&simpleServer, '\0', sizeof(simpleServer)); 
    simpleServer.sin_family = AF_INET;
    simpleServer.sin_addr.s_addr = htonl(INADDR_ANY);
    simpleServer.sin_port = htons(simplePort);

    /*  bind to the address and port with our socket  */
    returnStatus = bind(simpleSocket,(struct sockaddr *)&simpleServer,sizeof(simpleServer));

    if (returnStatus == 0) {
        fprintf(stderr, "Bind completed!\n");
    }
    else {
        fprintf(stderr, "Could not bind to address!\n");
    close(simpleSocket);
    exit(1);
    }

    /* lets listen on the socket for connections      */
    returnStatus = listen(simpleSocket, 5);

    if (returnStatus == -1) {
        fprintf(stderr, "Cannot listen on socket!\n");
    close(simpleSocket);
        exit(1);
    }

    int x;
    int i=0;

    while (1)

    {

        struct sockaddr_in clientName = { 0 };
        int simpleChildSocket = 0;
        int clientNameLength = sizeof(clientName);

        /* wait here */

            simpleChildSocket = accept(simpleSocket,(struct sockaddr *)&clientName, &clientNameLength);

        if (simpleChildSocket == -1) {

            fprintf(stderr, "Cannot accept connections!\n");
            close(simpleSocket);
            exit(1);

        }

        /* handle the new connection request  */
        /* write out our message to the client */
        

        //read the number of messages that have to be send
        read(simpleChildSocket, &x, sizeof(x));
        printf("x value is: %d\n", x);
        
        do{
            // read the message from client and copy it in buffer 
            read(simpleChildSocket, buff, sizeof(buff));

            //copy buff in MESSAGE
            strcpy(MESSAGE, buff);

            //sending the message
            write(simpleChildSocket, MESSAGE, strlen(MESSAGE));
            
            //cleaning the buffer
            memset(&simpleServer, '\0', sizeof(simpleServer));
            i++;
        }while(i<x);
        
        close(simpleChildSocket);
    }

    close(simpleSocket);
    return 0;

}

The client code is:

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

int main(int argc, char *argv[]) {

    int simpleSocket = 0;
    int simplePort = 0;
    int returnStatus = 0;
    char buffer[256] = "";
    struct sockaddr_in simpleServer;

    if (argc != 3) {

        fprintf(stderr, "Usage: %s <server> <port>\n", argv[0]);
        exit(1);

    }

    /* create a streaming socket      */
    simpleSocket = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);

    if (simpleSocket == -1) {

        fprintf(stderr, "Could not create a socket!\n");
        exit(1);

    }
    else {
        fprintf(stderr, "Socket created!\n");
    }

    /* retrieve the port number for connecting */
    simplePort = atoi(argv[2]);

    /* setup the address structure */
    /* use the IP address sent as an argument for the server address  */
    //bzero(&simpleServer, sizeof(simpleServer)); 
    memset(&simpleServer, '\0', sizeof(simpleServer));
    simpleServer.sin_family = AF_INET;
    //inet_addr(argv[2], &simpleServer.sin_addr.s_addr);
    simpleServer.sin_addr.s_addr=inet_addr(argv[1]);
    simpleServer.sin_port = htons(simplePort);

    /*  connect to the address and port with our socket  */
    returnStatus = connect(simpleSocket, (struct sockaddr *)&simpleServer, sizeof(simpleServer));

    if (returnStatus == 0) {
        fprintf(stderr, "Connect successful!\n");
    }
    else {
        fprintf(stderr, "Could not connect to address!\n");
    close(simpleSocket);
    exit(1);
    }

    /*create the message*/
    char buff[100];
    int i=0, x;
    //int n;
    //while((buff[n++] = getchar()) != '\n');
    printf("How many messages do you want to send?\n");
    scanf("%d", &x);
    write(simpleSocket, x, sizeof(x));

    printf("Insert the message:\n");

    do{
        fgets(buff, 100, stdin);
        write(simpleSocket, buff, sizeof(buff));
        i++;
    }while(i<=x);

    /* get the message from the server   */
    returnStatus = read(simpleSocket, buffer, sizeof(buffer));

    if ( returnStatus > 0 ) {
        printf("%d: %s", returnStatus, buffer);
    } else {
        fprintf(stderr, "Return Status = %d \n", returnStatus);
    }

    close(simpleSocket);
    return 0;
}
Ema
  • 23
  • 4

1 Answers1

0

You have an issue with read. This should be :

read(simpleChildSocket, &x, sizeof(x));

Read is expecting a pointer.

Ôrel
  • 7,044
  • 3
  • 27
  • 46
  • You can take a look to [the warning](https://stackoverflow.com/questions/57842756/why-should-i-always-enable-compiler-warnings) of your compiler – Ôrel Jul 19 '20 at 22:47
  • It does not work changing the read and giving a pointer. – Ema Jul 20 '20 at 10:34