Клиентский сервер C UDP не работает - PullRequest
0 голосов
/ 29 июня 2018

Итак, у меня есть один UDP-сервер, который возвращает клиенту час или дату со следующим кодом:

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

void main(int argc, char *argv[]) {
    struct addrinfo hints;
    struct addrinfo *res;
    struct sockaddr_storage cli;
    time_t rawtime;
    struct tm* timeinfo;
    char tbuffer[9];
    char buf[81], host[NI_MAXHOST], serv[NI_MAXSERV];

    hints.ai_family = AF_UNSPEC;    /* Allow IPv4 or IPv6 */
    hints.ai_socktype = SOCK_DGRAM; /* Datagram socket */
    hints.ai_flags = AI_PASSIVE;    /* For wildcard IP address */
    hints.ai_protocol = 0;          /* Any protocol */

    getaddrinfo(argv[1], argv[2], &hints, &res);
    int sd = socket(res->ai_family, res->ai_socktype, 0);
    bind(sd, (struct sockaddr *)res->ai_addr, res->ai_addrlen);
    freeaddrinfo(res);

    while(1) {
        socklen_t clen = sizeof(cli);
        int c = recvfrom(sd, buf, 80, 0, (struct sockaddr*) &cli, &clen);
        buf[c] = '\0';
        getnameinfo((struct sockaddr*) &cli, clen, host, NI_MAXHOST, serv, NI_MAXSERV, NI_NUMERICHOST);
        time(&rawtime);
        timeinfo = localtime(&rawtime);
        if(buf[0] == 't') {
                printf("%ld bytes de %s:%s\n", c, host, serv);
                ssize_t chars = strftime(tbuffer, sizeof(tbuffer), "%T", timeinfo);
                sendto(sd, tbuffer, chars, 0, (struct sockaddr *)&cli, clen);
        } else if(buf[0] == 'd') {
                printf("%ld bytes de %s:%s\n", c, host, serv);
                ssize_t chars = strftime(tbuffer, sizeof(tbuffer), "%D", timeinfo);
                sendto(sd, tbuffer, chars, 0, (struct sockaddr *)&cli, clen);
        } else if(buf[0] == 'q') {
                printf("Saliendo...\n");
                exit(EXIT_SUCCESS);
        } else {
                printf("Comando no soportado %s", buf);
        }
    }
}

Для проверки кода я использовал ./server <IPServerAddres> <PortAddress> на стороне сервера и nc -u <IpServerAddress> <PortServer> на стороне клиента (другой компьютер).

Теперь я хочу создать UDP-клиент UDP и сделать то же самое без использования nc, просто записав ./client <IPServerAddres> <PortAddress> <command>, где command может быть t для времени d для даты и q для выхода, как на сервере.

Вот код клиентской программы:

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

int main(int argc, char *argv[]) {
    struct addrinfo hints;
    struct addrinfo *res, *resp;
    int sd, j, s;
    size_t len;
    ssize_t nread, nwrite;
    char buf[500];

    if (argc < 3) {
        fprintf(stderr, "Usage: %s host port command...\n", argv[0]);
        exit(EXIT_FAILURE);
    }

    memset(&hints, 0, sizeof(struct addrinfo));
    hints.ai_family = AF_UNSPEC;    /* Allow IPv4 or IPv6 */
    hints.ai_socktype = SOCK_DGRAM; /* Datagram socket */
    hints.ai_flags = 0;
    hints.ai_protocol = 0;          /* Any protocol */

    getaddrinfo(argv[1], argv[2], &hints, &res);

    for(resp = res; resp != NULL; resp = resp->ai_next) {
        sd = socket(resp->ai_family, resp->ai_socktype, resp->ai_protocol);
        if(sd == -1) {
            perror("socket()");
            continue;
        }

        if(connect(sd, resp->ai_addr, resp->ai_addrlen) == -1) {
            perror("socket()");
        } else {
            break;
        }

        close(sd);
    }

    freeaddrinfo(res);

    for(j = 3; j < argc; j++) {
        len = strlen(argv[j]) + 1;
        nwrite = write(sd, argv[j], len);
        if(nwrite == -1) perror("write()");
        nread = read(sd, buf, 500);
        buf[nread] = 0;
        if(nread == -1) perror("read()");
        printf("Recibidos %ld bytes: %s\n", (long) nread, buf);
    }

    exit(EXIT_SUCCESS);
}

Проблема в том, что, когда я пытаюсь запустить свою клиентскую программу, я не получаю ответ, поэтому я предполагаю, что это бесконечный цикл или что-то в этом роде.

Любая помощь будет оценена, я изучаю C, поэтому извините за ошибки, спасибо.

EDITED: исправлено break в цикле for, если connect() возвращает что-то отличное от -1.

1 Ответ

0 голосов
/ 29 июня 2018

Проблема в расчете длины сообщения для отправки:

len = strlen(argv[j] + 1);

Получает длину строки , начиная со второго символа . Вы хотите:

len = strlen(argv[j]) + 1;

Это дает вам длину строки плюс 1 для нулевого терминатора.

Кроме того, после прочтения ответа вам нужно будет завершить нулем то, что вы получили:

buf[nread] = 0;
...