当尝试在子线程中访问套接字描述符(使用bind()或listen())时,我收到错误:
错误的文件描述符
这是代码(忽略标题):
class task {
private:
std::int32_t socFd;
std::string path;
char buffer[MAX_SOC_BUFFER];
struct sockaddr_un socAddr;
public:
void init() {
if ((socFd = socket((std::int32_t)AF_UNIX, (std::int32_t)SOCK_STREAM, 0)) == -1) {
perror("socket");
exit(1);
}
socAddr.sun_family = AF_UNIX;
strcpy(socAddr.sun_path, getSocPath().c_str());
}
task(const std::string& path) : path{path}
{
init();
}
auto getSocketFd() ->decltype(socFd) {
return socFd;
}
const std::string& getSocPath() {
return path;
}
auto getSocAddr() -> decltype(socAddr)&
{
return socAddr;
}
char* getBuff() {
return buffer;
}
virtual ~ task() {
close(getSocketFd());
}
};
class server_task : public task{
public:
void init () {
::unlink(getSocPath().c_str());
size_t len = sizeof(getSocAddr());
if (bind(getSocketFd(), (struct sockaddr *)&(getSocAddr()), len) == -1) {
perror("bind");
exit(1);
}
}
server_task(const std::string& path) : task(path)
{
init();
}
void operator()() {
struct sockaddr_un remote;
if (listen(getSocketFd(), 5) == -1) {
perror("listen");
exit(1);
}
for(;;) {
int done, n, socFd2;
printf("Waiting for a connection...\n");
socklen_t t = sizeof(remote);
if ((socFd2 = accept(getSocketFd(), (struct sockaddr *)&remote, &t)) == -1) {
perror("accept");
exit(1);
}
printf("Connected.\n");
done = 0;
do {
n = recv(socFd2, getBuff(), 100, 0);
if (n <= 0) {
if (n < 0) perror("recv");
done = 1;
}
if (!done) {
std::string buffStr(getBuff());
std::transform(buffStr.begin(), buffStr.end(),buffStr.begin(), ::toupper);
if (send(socFd2, buffStr.c_str(), n, 0) < 0) {
perror("send");
done = 1;
}
}
} while (!done);
close(socFd2);
}
}
};
int main(){
std::vector<std::thread> threads;
server_task server("/tmp/temp_socket");
threads.push_back(std::thread(server));
for(auto& thread : threads)
thread.join();
return 0;
}
如果我将main()
功能更改为:
int main(){
server_task server("/tmp/temp_socket");
server();
return 0;
}
没有发生此类错误。据我所知,文件描述符表在父进程和它的子线程之间共享,因此套接字句柄应该是有效的。
我做错了什么?
答案 0 :(得分:5)
threads.push_back(std::thread(server));
这创建了server
对象的至少一个副本,可能是多个副本,并且因为您没有定义复制构造函数,每个副本共享同一个套接字,而第一个超出范围的副本关闭套接字。到新线程启动时,临时副本已被销毁并关闭套接字,因此在新线程中运行的server_task
副本引用一个已关闭的套接字。
您应该为task
和server_task
类型定义移动构造函数,以便传输套接字的所有权而不复制,并将对象移动到线程中:
thread.push_back(std::thread(std::move(server)));
或者您应该通过引用将服务器传递给新线程,因此没有副本:
thread.push_back(std::thread(std::ref(server)));