C-Server行为不合理

时间:2017-02-16 14:22:17

标签: c

C-SERVER设计为在收到“ 111 ”时打印“确定不是任何人。

(我想办法删除 \ n 所以“111 \ n”是准确的。)

echo '111' | nc <ip> <port>
echo 'TEST-OOO' | nc <ip> <port>
echo '111' | nc <ip> <port>

C-SERVER :(以上3个与以下答案相关)

111
ok

TEST-OOO

111
-OOO

STRANGE :为什么打印 -OOO

它设计为在“ 111 ”时打印“确定”。不 -OOO

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

char outt[15];
char inn[100];

int main()
{
  int one = 1, client_fd;
  struct sockaddr_in svr_addr, cli_addr;
  socklen_t sin_len = sizeof(cli_addr);

  int sock = socket(AF_INET, SOCK_STREAM, 0);
  if (sock < 0)
    err(1, "can't open socket");

  setsockopt(sock, SOL_SOCKET, SO_REUSEADDR, &one, sizeof(int));

  int port = 85;
  svr_addr.sin_family = AF_INET;
  svr_addr.sin_addr.s_addr = INADDR_ANY;
  svr_addr.sin_port = htons(port);

  if (bind(sock, (struct sockaddr *) &svr_addr, sizeof(svr_addr)) == -1) {
    close(sock);
    err(1, "Can't bind");
  }

  listen(sock, 5);
  while (1) {
    client_fd = accept(sock, (struct sockaddr *) &cli_addr, &sin_len);

    read(client_fd,inn,100);
    printf(inn);

    if (strcmp( inn, "111\n" ) == 0){
    printf("ok\n");
    }

    snprintf(outt, sizeof outt, "%s\n", inet_ntoa(cli_addr.sin_addr));


    if (client_fd == -1) {
      perror("Can't accept");
      continue;
    }

    write(client_fd, outt, sizeof(outt) - 1); /*-1:'\0'*/
    close(client_fd);
  }
}

2 个答案:

答案 0 :(得分:3)

此代码:

read(client_fd,inn,100);
printf(inn);

是未定义的行为。 read()不会使用NUL字符终止接收的数据。在这种情况下,循环的上一次迭代中inn的内容会出现在输出中,因为它们仍然存在

这会更好:

ssize_t bytesRead = read(client_fd,inn,sizeof(inn) - 1);
if ( bytesRead > 0 )
{
    inn[ bytesRead ] = '\0';
    printf(inn);
}

将打印作为NUL终止字符串发送的数据,假设客户端实际发送了没有NUL终止的C样式字符串。

答案 1 :(得分:0)

您未查看来自read()的电话的结果,以了解已阅读的内容。

然后,您不会终止inn,因此它不会在您认为应该终止的位置终止,而是包含上一个已发送行的一部分。