Question

我正在尝试编写一个使用类，启动和侦听网络连接的c ++程序，然后为每个新客户端旋转一个新线程。

幸运的是，我已经弄清楚如何从类中生成一个线程，但是当我尝试在类中执行一个accept（）时，我会遇到一个分段错误。我会发布代码，以便更容易地显示我遇到问题的地方。

#include <iostream>
#include <string.h> //for memset
#include <pthread>
#include <sys/types.h> //network
#include <sys/socket.h> //network
#include <netinet/in.h> //network

using namespace std;
class network
{
  public:
    void my_listen();
    static void *handleClient(void * in_stream);
};

void* network::handleClient(void * in_stream)
{

  int *stream = reinterpret_cast<int *>(in_stream);
  write(*stream,"Hello Client\n", 12);
}

void network::my_listen()
{
  /*
   * Name: my_listen()
   * Purpose: Listens and accepts new connections. Once accpeted, a new thread
   *          is spun off. 
   * Input: none
   * Output: none
  */

  int *new_socket_desc;
  int port_num = 9876;
  socklen_t client_addr_len;

  int socket_desc = socket(AF_INET,SOCK_STREAM,0);

  sockaddr_in serv_addr, cli_addr;

  if(socket_desc == -1)
  {
    cerr << "Unable to create new sockets\n";
  }

  memset(&serv_addr, 0, sizeof(serv_addr));

  serv_addr.sin_family = AF_INET;
  serv_addr.sin_addr.s_addr = INADDR_ANY;
  serv_addr.sin_port = htons(port_num);

  if(bind(socket_desc, (struct sockaddr *) &serv_addr, sizeof(serv_addr)) < 0)
  {
    cerr << "Error on binding " << port_num << endl;
  }

  listen(socket_desc,5);

  client_addr_len = sizeof(cli_addr);

  pthread_t thread[10];
  int count = 0;

  *new_socket_desc = accept(socket_desc, (struct sockaddr *) &cli_addr, &client_addr_len); //Right here I segment fault

  cout << "Connected client " << " @ " << new_socket_desc << " (" << *new_socket_desc << ")" <<  endl;

  pthread_create(thread[0], handleClient, (void *)new_socket_desc);

  pthread_join(thread[0],NULL);
}

主要不是那么令人兴奋：

#include "network.h"
using namespace std;

int main()
{
  network my_network;

  my_network.my_listen();
}

有趣的是，我可以在不使用课程的情况下完成所有这些工作。我确定它与范围有关，但我不知道为什么。

顺便说一下，我正在使用gcc 4.6.2，目标平台为x86_64

Answer 1

您将new_socket_desc声明为指针，但从不设置它指向的内容。当你试图在指针的位置放一个值（带有接受调用的结果）时，它会写入内存中的某个随机位置，这可能会也可能不会立即导致崩溃（在此版本的代码中，它是导致崩溃但总是非常糟糕。

将new_socket_desc设为您班级的常规int成员，使用＆amp;创建线程时运算符，否则不要使用*运算符，否则你应该有更好的运气。

Answer 2

int *new_socket_desc;
// ....
*new_socket_desc = accept(socket_desc ....

您将new_socket_desc声明为单位指针，然后将accept的结果分配给new_socket_desc指向的随机存储位置。显然，这会导致SEGFAULT。你应该这样做：

int new_socket_desc;
// ....
new_socket_desc = accept(socket_desc ....

C ++类网络编程的分段错误

2 个答案: