Почему буфер приема WinSock заполняется мусором?

Question

[sddvxd]

Здравствуйте.
Исходники взял с MSDN:

do {
			iResult = recv(ClientSocket, recvbuf, recvbuflen, 0);
			if (iResult > 0) {
				iSendResult = send(ClientSocket, recvbuf, iResult, 0);
				if(iSendResult==SOCKET_ERROR){
					printf("send failed with error: %d\n", WSAGetLastError());
					closesocket(ClientSocket);
					WSACleanup();
				}
				for (int i = 0; i < iResult; i++) {
					std::cout << recvbuf[i];
				}
			}
			else if (iResult == 0) {
				//std::cout << "Connection closing...\n";
				std::cout << std::endl;
			}
			else {
				printf("recv failed with error: %d\n", WSAGetLastError());
				closesocket(ClientSocket);
				WSACleanup();
				std::cout << "Error in receive\n";
			}
		} while (iResult > 0);

Проблема состоит в том, что recvbuf почему то содержит какой-то мусор, хотя iResult, допустим если я отправляю 4 байта на сервер - тоже содержит число 4
А если я смотрю strlen(recvbuf) то получаю 16. Откуда берется такой хвост? При выводе recvbuf выводятся первые 4 байта и дальше мусор

Comments

[Roman]

recvbuf размер какой
Вы бы целиком показали что делаете
Протокол TCP UDP

[sddvxd]

Роман, спасибо, уже решил проблему

char * useful = new char[iResult + 1];
memcpy(useful, recvbuf, iResult);
std::cout << useful;

Answer 1

[res2001]

А если я смотрю strlen(recvbuf) то получаю 16. Откуда берется такой хвост?

В recvbuf вы получаете не с-строку, а массив байт, применять строковые функции к массиву байт бессмысленно.
Отличие с-строки от массива в том, что строка оканчивается завершающим нулем, а в массиве байт вы должны знать длину массива. В с-строке символ 0 не может быть в теле самой строки, только в качестве терминального символа, в массиве байт 0 - такой же равноправный элемент данных, как и все остальные, может быть в любом месте массива или не быть вовсе.
Вам еще повезло, что выдает 16 байт, вместо 4, вполне могло бы сложиться так, что нулевого байта не было бы на достаточно продолжительном участке памяти, тогда был бы просто вылет из программы или какие-либо трудно диагностируемые глюки.

Comments

[sddvxd]

То-есть мне нужно создать массив длиной iResult+1 для нуль-терминатора и воспользоваться функцией memcpy с указанием в качестве последнего аргумента iResult?

[sddvxd]

Да, действительно. Это помогло

[res2001]

sddvxd, Если речь о выводе на экран, то можно воспользоваться Си функцией printf, там в шаблонах можно указать длину строки для вывода (тогда будет выводится заданное количество символов, не взирая на наличие нулевого символа).
Не знаю, есть ли подобная возможность в плюсовом cout.
Так же, если в дальнейшем планируется работать с полученной информацией как со строкой, то необходимо вручную добавить нулевой байт в конец. Тогда вы получите стандартную с-строку.

[sddvxd]

res2001, можно еще короче сделать, чем у меня было:)

recvbuf[iResult] = '\0';
std::cout << recvbuf;

[res2001]

sddvxd, вот теперь правильно.
Только еще один момент: в этом случае вы должны в recv передавать размер буфера на 1 меньше, чем есть на самом деле, для того что бы всегда была возможность вставить терминальный ноль, даже когда recv запишет весь доступный объем буфера.