Question

我需要使用TCP套接字接收两个数据，并在服务器中进行验证。当我使用“strcmp”来比较两个数据时。如果它是正确的，它应该发送服务器的networkconfig或发送“Wrong”到客户端。

在我的代码中。当我收到合适的用户和密码并正确发送数据时它会工作。但是当我收到错误的用户和＆amp; passwd，它什么都不做！只需挂在那里等待发送的正确数据。

PS。 0,0是我的defalut用户＆passwd.they由“＃”组合发送

以下是我的代码

#include <sys/socket.h>
#include <sys/types.h>
#include <netinet/in.h>
#include <arpa/inet.h>
#include <netdb.h>
#include <unistd.h>
#include <string.h>
#include <stdio.h>
#include <stdlib.h>
#include <fcntl.h>   
#include "xmlmgr.h"
void SetConfig(char *textS);
void ReadConfig(char *textR);
int main()
{
  int serverSocket;
  struct sockaddr_in serverAddr;
  struct sockaddr_in clientAddr;
  int port = 8000;
  int val = 1;
  int ret;
  fd_set set1;
  char textS[1024];
  char textR[1024];
  char account[32];
  char login[1024];
  int len;
  int sock;
  int clientAddrSize;
  struct timeval timeout;

  ReadConfig(textR);
  //  ** //this is my network config,it has been sended and received correctly in client server.**

  serverSocket = socket(AF_INET, SOCK_STREAM, 0);
  setsockopt(serverSocket, SOL_SOCKET, SO_REUSEADDR, &val, sizeof(val));
  serverAddr.sin_family = AF_INET;
  serverAddr.sin_port = htons(port);
  serverAddr.sin_addr.s_addr = htonl(INADDR_ANY );
  memset(&(serverAddr.sin_zero), 0, 8);
  int sock_opt = 1;

  if (setsockopt(serverSocket, SOL_SOCKET, SO_REUSEADDR, (void*) &sock_opt,
      sizeof(sock_opt)) == -1)
  {
    printf(" false");
  }

  if (bind(serverSocket, (struct sockaddr*) &serverAddr,
      sizeof(struct sockaddr)) < 0)
  {
    close(serverSocket);
    printf("Bad bind\n");
    exit(1);
  }

  if (listen(serverSocket, 5) < 0)
  {
    printf("Bad listen\n");
    exit(1);
  }
  clientAddrSize = sizeof(struct sockaddr_in);
  sock = accept(serverSocket, (struct sockaddr*) &clientAddr,
      (socklen_t*) &clientAddrSize);
  FD_ZERO(&set1);
  FD_SET(sock, &set1);
  len = recv(sock, account, 1024, 0);
  // ** //receive user&passwd**

  while (1)
  {
    if (strcmp("0#0", account) == 0)
    {
      // ** //this is done correctly**
      printf("login!!\n");
      send(sock, &textR, strlen(textR) + 1, 0);
      printf("sended!!\n");

      sock = accept(serverSocket, (struct sockaddr*) &clientAddr,
          (socklen_t*) &clientAddrSize);
      recv(sock, &textS, 1024, 0);

      close(serverSocket);
      SetConfig(textS);
      system("/etc/init.d/network restart");

      exit(0);
    }
    else
    {
      // ** //Never do this part,this is my problem**
      strcpy(login, "Wrong!!!\n")
      send(sock, &login, strlen(login) + 1, 0);
      printf("wrong");
      sock = accept(serverSocket, (struct sockaddr*) &clientAddr,
          (socklen_t*) &clientAddrSize);
      close(serverSocket);

      exit(0);
    }
  }
}

Answer 1

您客户可能有问题。即使passwd错了你等待另一个连接（在else中接受）。您也应该发布您的客户端代码。

Answer 2

发出＆＃34;错误＆＃34;代码卡在这一行：

  sock = accept(serverSocket, (struct sockaddr*) &clientAddr,
      (socklen_t*) &clientAddrSize);

只需删除它，就会无用地等待另一个传入连接。

Answer 3

这方面的一个大问题是，在测试凭证字符串后，两个子句都关闭了serverSocket，这意味着在用户名#password两次错误尝试之后，服务器将不再接受任何连接！

下一个问题是recv调用不在while循环中，因此服务器只会读取一次登录字符串，之后在重新启动之前不再读取任何尝试。

另一个问题是if / else结构的两个子句都退出程序，因此while（1）是多余的，第二个子句中的accept（）调用也是如此。

在“Correct”子句中，您接受另一个连接，并从中读取一个字符串...而不是通过用户名#password自动验证的连接。

这些不是你要问的错误，但它们仍然使程序不可靠。所以在你做任何事之前：

声明一个名为“finished”的int，它最初设置为零;
使用do {}替换while（1）{} while（！finished）;
将accept（）和recv（）调用移入do-while循环，就在if / strcmp测试之前
删除exit（）调用，将“correct”子句替换为finished = 1;
从“正确”和“不正确”条款中删除accept（）调用
在do-while循环后关闭serverSocket。正如其他响应者所说，你还需要捕获并检查send（）和recv（）的返回值，检查accept的返回值也没有坏处。

By The Way，如果你必须使用带字符串的recv（），你可能想要使用strncmp（），你将recv（）返回值作为“len”捕获，为什么你没有测试过它看到recv做了什么？

从TCP套接字接收用户和密码并检查

3 个答案: