Question

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

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

#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;
}

Answer 1

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

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

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

4b d9 e7 b3 00 96 89 fb

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

4b d9 e7 b3 2f 96 89 fb

no 空字节，然后strcpy()或strlen()，或者任何字符串函数将继续超过缓冲区的末尾，结果不可预测。

Answer 2

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

Answer 3

仅出于调试目的，sleep()和write()之间的read()可能会有所帮助，以便让服务器有时间回复？配置适当的超时将是长期处理它的正确方法。然后，我可能完全走错了路。

read（）函数中的随机空字符

3 个答案: