1

我正在用 TCP 构建一个基本的客户端和服务器。它正在工作，除非密钥变得有点大。它只是来回发送密钥，直到时间结束。当它变大（又名 4096）时，read() 语句将随机获得一个 null，然后将密钥分成两个 msg，然后打破循环。

我真的不确定它为什么这样做，服务器代码正在工作，因为它已经与其他客户端进行了测试，所以我的客户端有问题。任何想法（很短）：

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

#define BUFFER_LENGTH 5120

// Error message taken from reference
void error(const char *msg)
{
    printf("%s\n", msg);
    exit(0);
}
int main(int argc, char *argv[])
{
    int sockfd, portNumber, n;
    struct sockaddr_in serv_addr;
    struct hostent *server;

    // Invalid arguments
    if (argc < 4)
        exit(0);
    else if (atoi(argv[3]) < 1 || atoi(argv[3]) > 4096)
        exit(0);

    char buffer[BUFFER_LENGTH];
    bzero(buffer, BUFFER_LENGTH);
    char buffer2[BUFFER_LENGTH];
    bzero(buffer2, BUFFER_LENGTH);
    strcpy(buffer2, "Connect.  Key length:  ");
    strcpy(buffer, strcat(buffer2, argv[3]));
    portNumber = atoi(argv[2]); // Get port number in int format
    sockfd = socket(AF_INET, SOCK_STREAM, 0); // Create socket connection to server using internet constants

    // Did we open the socket succesfully?
    if (sockfd < 0)
        error("Error opening socket.");

    // Is the IP Address valid?
    server = gethostbyname(argv[1]);
    if (server == NULL)
        error("Could not connect to server. Terminating.");

    // Initialize to zero and then set
    // Taken from reference
    bzero((char *) &serv_addr, sizeof(serv_addr));
    serv_addr.sin_family = AF_INET;
    bcopy((char *)server->h_addr, (char *)&serv_addr.sin_addr.s_addr, server->h_length);
    serv_addr.sin_port = htons(portNumber);
    if (connect(sockfd,(struct sockaddr *) &serv_addr,sizeof(serv_addr)) < 0) 
        error("Could not connect to server. Terminating.");

    short connected = 1;
    do 
    {
        // Send initial request on first pass
        // Afterwards send the resposne we were given
        printf("Sending: %s\n", buffer);
        printf("Length of msg: %d\n", (strlen(buffer) + 1));
        n = write(sockfd, buffer, strlen(buffer) + 1);
        if (n < 1) 
             error("Failed to send Message. Terminating.");

        // Get session key response
        bzero(buffer, BUFFER_LENGTH);
        n = read(sockfd, buffer, BUFFER_LENGTH);
        if (n < 1)
            error("Could not fetch result.  Terminating.");

        // Stop
        if (strcmp(buffer, "Invalid session key.  Terminating.") == 0)
            break;

        printf("%s\n", buffer);
        sleep(1);
    } while (connected == 1);

    // Done (this should never be reached in this client)
    error("Could not fetch result.  Terminating.");
    close(sockfd);
    return 0;
}
4

3 回答 3

5

不要在二进制数据上使用strcpy()或任何函数。str*()你不是在处理字符串。使用memcpy()并指定要复制的字节数。

这些str*()函数对字符串进行操作；C 标准将字符串定义为“由第一个空字符终止并包括第一个空字符的连续字符序列”。但在二进制数据中，空字符（零字节）只是另一块不一定有任何意义的数据。

如果您的二进制数据如下所示（十六进制）：

4b d9 e7 b3 00 96 89 fb

然后strcpy()将忽略00字节之后的所有内容。更糟糕的是，如果您的二进制数据如下所示：

4b d9 e7 b3 2f 96 89 fb

没有空字节， then ， or ，或任何字符串函数将继续超过缓冲区的末尾，并产生不可预测的结果。strcpy()strlen()

于 2012-10-14T22:32:53.207 回答
1

您假设 one 发送的数据write()恰好由 one 读取read()。TCP 或 Sockets API 规范中没有任何内容支持这种假设。您必须阅读直到收到完整的消息，并且“消息”是由您定义的，而不是由 TCP 或 API 定义的。

于 2012-10-15T01:06:11.523 回答
-1

仅出于调试目的，在andsleep()之间可能会有帮助，以便让服务器有时间响应？从长远来看，配置适当的超时将是处理它的正确方法。再说一次，我可能完全走错了路。write()read()

于 2012-10-14T22:30:12.013 回答