Что делать если listen возвращает 10022?

Question

[armadillo-cld]

Пишу TCP сервер на MFC.
Инициализирую так:

CCYBERRMSDlg* dlg = (CCYBERRMSDlg*)Param;
	
	CString ip, port;
	dlg->GetDlgItem(IDC_EDIT2)->GetWindowTextA(ip);
	dlg->GetDlgItem(IDC_EDIT3)->GetWindowTextA(port);

	string IP, PORT;
	IP = string(CT2CA(ip));
	PORT = string(CT2CA(port));

	WSAData wsaData;
	WORD DllVersion = MAKEWORD(2, 1);
	if (WSAStartup(DllVersion, &wsaData) != 0) {
		return false;
	}

	SOCKADDR_IN addr;
	int sizeofaddr = sizeof(addr);
	addr.sin_family = AF_INET;
	addr.sin_addr.s_addr = inet_addr(IP.c_str());
	addr.sin_port = atoi(PORT.c_str());
	
	SOCKET sListen = socket(AF_INET, SOCK_STREAM, NULL);
	bind(sListen, (SOCKADDR*)&addr, sizeof(addr));
	if (WSAGetLastError() != 0) {
		dlg->AddLog("bind error");
		return -1;
	}
	if (listen(sListen, SOMAXCONN) != 0) {
		dlg->AddLog("listen error");
		return -1;
	}

Это всё запускается и работает в отдельном потоке.
Как итог - в лог бокс выводится "listen error".
Изначально всё работало нормально, но потом что-то изменил, не помню что, и всё работать перестало. Магия программирования...

Comments

[Евгений Шатунов]

armadillo-cld , у listen[?] есть код ошибки. Только ты его не запоминаешь и не проверяешь - это ошибка уже твоя.
Какой именно код ошибки тебе отдается?

[armadillo-cld]

Евгений Шатунов, Здравствуйте.
Получаю код ошибки:

if (listen(sListen, SOMAXCONN) != 0) {
		string code = to_string(WSAGetLastError());
		dlg->AddLog(string("listen error. Code error: " + code).c_str());
		return -1;
	}

В лог боксе выводит: listen error. Code error: 10022

[maaGames]

> не помню что, и всё работать перестало
используй любую систему контроля версий. Хоть git, хоть svn, хоть какую-нибудь...

[armadillo-cld]

maaGames, Не знаю, не знаю. Никогда не видел смысла использования гита в своих проектах...

[armadillo-cld]

WSAGetLastError - выдаёт 10022
Это: Недопустимый аргумент.
Был предоставлен какой-то недопустимый аргумент (например, указание недопустимого уровня для функции setsockopt). В некоторых случаях это также относится к текущему состоянию сокета—например, вызов accept на сокете, который не слушает.

Но что именно не так? Вроде всё верно ввёл, сокет слушает...

[maaGames]

armadillo-cld, Ну тогда пытайся вспомнить, что и когда изменил.

[Евгений Шатунов]

armadillo-cld, давай я просто цитатами пройду этот момент.

Изначально всё работало нормально, но потом что-то изменил, не помню что, и всё работать перестало.

Никогда не видел смысла использования гита в своих проектах

Этот смысл у тебя сейчас прямо перед глазами, но ты его все равно не видишь. Я правильно понимаю?
Ты просто физически не поднимешь даже небольшой проект с небольшим кодом в течении длительного времени без истории, веток, таймлапса и без возможности всегда вспомнить подробности написанного.

По поводу ошибки. Да, ты верно все написал про WSAEINVAL. Это буквально означает что ты передаешь именно в listen неправильные аргументы. Второй аргумент у тебя не может быть неправильным. Остается вопрос к первому - к сокету.

У bind[?] ты ошибку тоже не проверяешь. Стоит это сделать.

[armadillo-cld]

Евгений Шатунов, На bind ошибку тоже проверял, всё чисто.

SOCKET sListen = socket(AF_INET, SOCK_STREAM, NULL);
	bind(sListen, (SOCKADDR*)&addr, sizeof(addr));
        int err = WSAGetLastError();
	if (err != 0) {
		string code = to_string(err());
		dlg->AddLog(string("Error bind: " + code).c_str());
		return -1;
	}

Всегда выдаёт на listen. Менял порт, всё равно именно в listen выдаёт ошибку.

if (listen(sListen, SOMAXCONN) != 0) {
		string code = to_string(WSAGetLastError());
		dlg->AddLog(string("Error listen: " + code).c_str());
		return -1;
	}

Error listen: 10022

[Евгений Шатунов]

armadillo-cld, ты не проверяешь результат вызова bind.
Давай посмотрим вместе, что пишется в документации listen относительно кода WSAEINVAL.

WSAEINVAL The socket has not been bound with bind.

[armadillo-cld]

Евгений Шатунов, Но как тогда проверять результат выполнения bind?
Напрямую не получается:
if(bind(...) != 0)
Т.е bind имеет другой тип возвращаемого значения...
Остаётся только через WSAGetLastError, и WSAGetLastError выдаёт 0 после выполнения bind...

[Евгений Шатунов]

armadillo-cld, я несколько раз привел ссылку на документацию к bind. Ты ведь изучил документацию?
WSAGetLastError, согласно документации, вызывать нужно только когда bind возвращает SOCKET_ERROR. Только так ты можешь понять, что WSAGetLastError вернул код ошибки именно от bind.
И у listen точно так же. Тебе надо просто лучше обрабатывать ошибки. У тебя реальная ошибка может быть где угодно выше по тексту и ты ее просто проглядел.

Я рекомендую тебе к каждой функции прочитать документацию и писать свой код только в соответствии с документацией.

Answer 1

[armadillo-cld]

Так и не понял в чём заключалась проблема, но мне кажется что нужно было использовать ::bind а не bind
Нашел рабочий код тут: https://www.binarytides.com/code-tcp-socket-server...

WSACleanup();
	WSADATA wsaData;
	SOCKET sListen, newConnection;
	struct sockaddr_in server, client;
	int c;
	
	if (WSAStartup(MAKEWORD(2, 2), &wsaData) != 0) {
		dlg->AddLog("Error WSAStartup");
		return -1;
	}

	if ((sListen = socket(AF_INET, SOCK_STREAM, 0)) == INVALID_SOCKET) {
		dlg->AddLog("Error create socket sListen");
		return -1;
	}

	server.sin_family = AF_INET;
	server.sin_addr.s_addr = inet_addr(IP.c_str());
	server.sin_port = htons(atoi(PORT.c_str()));

	int result = ::bind(sListen, (struct sockaddr*)&server, sizeof(server));
	if (result != 0) {
		string code = to_string(WSAGetLastError());
		dlg->AddLog(string("Bind error: " + code).c_str());
	}

	listen(sListen, 256);

	dlg->AddLog("Waiting for connections...");

	c = sizeof(struct sockaddr_in);

	newConnection = accept(sListen, (struct sockaddr*)&client, &c);
	if (newConnection == INVALID_SOCKET) {
		dlg->AddLog("Error accept");
	}

	dlg->AddLog("Connected new user!");

	closesocket(sListen);