Ошибка Winsock 10022 при прослушивании

0

Я делаю небольшой IRC-сервер, но я столкнулся с проблемой; при попытке прослушивания сокета я получаю сообщение об ошибке 10022 (Invalid Argument).

Ошибка также появляется на accept(), но это происходит потому, что сокет не прослушивается (проблема, о которой я отправляю).

Я не включил функцию accept, потому что я чувствую, что это не обязательно и будет добавлять бессмысленный код.

#include <iostream>
#include <ws2tcpip.h>
#include <winsock2.h>
#include <thread>
#include <string>
#pragma comment(lib, "Ws2_32.lib")
#define maxConnections 10
class Server
{
        struct sockaddr_storage their_addr;
        struct addrinfo hints, *res;
        struct addrinfo *servinfo;
    int status;
    SOCKET sock;
public:
    void Start(const char *port);
};

void Server::Start(const char *port)
{
    WSADATA WSAData;
    if (WSAStartup(MAKEWORD(2, 0), &WSAData) != 0)
    {
        std::cout << "[ERROR]: " << GetLastError() << ".\n";
    }
    memset(&hints, 0, sizeof(hints));
    hints.ai_family = AF_INET;
    hints.ai_socktype = SOCK_STREAM;
    hints.ai_protocol = IPPROTO_TCP;
    status = getaddrinfo(NULL, port, &hints, &res);
    sock = socket(res->ai_family, res->ai_socktype, res->ai_protocol);
    if (sock == SOCKET_ERROR)
    {
        std::cout << "[ERROR]: " << WSAGetLastError() << "Bad Socket.\n";
    }
    bind(sock, res->ai_addr, res->ai_addrlen);

Ошибка:

    if (listen(sock, maxConnections) == SOCKET_ERROR)
    {
        std::cout << "[ERROR]: " << WSAGetLastError() << " Listening Failed.\n";
    }

В приведенном выше коде подробно описано создание и привязка сокета, все из которых успешны (хотя и не обязательно правильные). Возможно, создание сокета, включая "NULL", может быть проблемой.

Благодаря :)

  • 0
    Аргументы, которые вы передаете для bind выглядят неверно - вы должны связываться с адресом и портом. Попробуйте проверить ошибку, возвращенную из этого вызова.
  • 0
    Нет ошибок - WSAGetLastError () возвращает 0.
Теги:
sockets
winsock2

2 ответа

0
Лучший ответ

WSAStartup() и getaddrinfo() не используют (WSA)GetLastError(), они вместо этого возвращают фактический код ошибки. Вы не учитываете это в сообщениях об ошибках.

socket() возвращает INVALID_SOCKET при SOCKET_ERROR, а не SOCKET_ERROR.

При использовании getaddrinfo() для создания сокета, вы должны указать AI_PASSIVE в addrinfo.ai_flags поле hints параметра. Это будет заполнять выходной addrinfo данными, которые подходят для передачи bind().

Попробуйте что-нибудь еще:

class Server
{
private:
    bool winsockStarted;
    SOCKET sock;
    ...
public:
    Server();
    ~Server();
    bool Start(const char *port);
    void Stop();
    ...
};


Server::Server()
    : sock(INVALID_SOCKET), winsockStarted(false)
{
    WSADATA WSAData = {0};
    int status = WSAStartup(MAKEWORD(2, 0), &WSAData);
    if (status != 0)
        std::cout << "[ERROR]: " << status << " Unable to start Winsock." << std::endl;
    else
        winsockStarted = true;
}

Server::~Server()
{
    Stop();

    if (winsockStarted)
        WSACleanup();
}

bool Server::Start(const char *port)
{
    Stop();

    struct addrinfo hints = {0};
    struct addrinfo *res = NULL;

    hints.ai_flags = AI_PASSIVE;
    hints.ai_family = AF_INET;
    hints.ai_socktype = SOCK_STREAM;
    hints.ai_protocol = IPPROTO_TCP;

    int status = getaddrinfo(NULL, port, &hints, &res);
    if (status != 0)
    {
        std::cout << "[ERROR]: " << status << " Unable to get address info for Port " << port << "." << std::endl;
        return false;
    }

    SOCKET newsock = socket(res->ai_family, res->ai_socktype, res->ai_protocol);
    if (newsock == INVALID_SOCKET)
    {
        std::cout << "[ERROR]: " << WSAGetLastError() << " Unable to create Socket." << std::endl;
        freeaddrinfo(res);
        return false;
    }

    if (bind(newsock, res->ai_addr, res->ai_addrlen) == SOCKET_ERROR)
    {
        std::cout << "[ERROR]: " << WSAGetLastError() << " Unable to bind Socket." << std::endl;
        freeaddrinfo(res);
        closesocket(newsock);
        return false;
    }

    freeaddrinfo(res);

    if (listen(newsock, maxConnections) == SOCKET_ERROR)
    {
        std::cout << "[ERROR]: " << WSAGetLastError() << " Unable to Listen on Port " << port << "." << std::endl;
        closesocket(newsock);
        return false;
    }

    sock = newsock;
    return true;
}

void Server::Stop()
{
    if (sock != INVALID_SOCKET)
    {
        closesocket(sock);
        sock = INVALID_SOCKET;
    }
}
Поделиться
  • 0
    Что я могу сказать, кроме полного, всеобъемлющего ответа, который полностью отвечает на мой вопрос эффективно. Большое спасибо :)
0

Я перечитал свой код и понял, что мне нужно добавить чек здесь

status = getaddrinfo(NULL, port, &hints, &res);

Я изменил его на

if (status = getaddrinfo(NULL, port, &hints, &res) != 0)
{
    std::cout << "[ERROR]: " << WSAGetLastError() << "Get Address Info failed.\n";
}

и запуск был успешным.

Если кто-то может объяснить, почему, я улучшу свой ответ.

Поделиться
  • 0
    Вам нужно использовать дополнительный паратез при выполнении присваивания и сравнения в одном и том же выражении: if ((status = getaddrinfo(NULL, port, &hints, &res)) != 0) , в противном случае разделите их на разные операторы: status = getaddrinfo(NULL, port, &hints, &res); if (status != 0) .
  • 0
    Спасибо за указатель :) <вставьте шутку c ++ здесь />

Ещё вопросы

Сообщество Overcoder
Наверх
Меню