Winsock recv printing garbage ascii characters

[demoylan]

Hello!
I have made a simple HTTP client. I can connect to a server and request a specific path to a page on that server. The issue is not that it doesn't work, but it just prints a bunch of garbage after it prints the web page.

So basically my program works by connecting to the requested server, then it tells you to enter a path to a file you want to request. After you do that, it will print out the contents of that particular file. After that, it will print out a small block of the 204 ascii character, then it will print a seemingly random character and another character that does not appear to be on the ascii table (perhaps the copyright symbol or the trademark).

I don't know exactly why this is happening, and I don't have a clue how to prevent it. Please help.

My Code for the HTTP client:

#include "stdafx.h"
#include <WinSock2.h>
#include <ws2tcpip.h>
#include <iostream>
#include <thread>
#include <string>

#pragma comment (lib, "Ws2_32.lib")
#pragma comment (lib, "Mswsock.lib")
#pragma comment (lib, "AdvApi32.lib")

#define DEFAULT_PORT "80"
#define DEFAULT_BUFFER_LENGTH 2048

void error(char msg[] = "An error occured", SOCKET socket = NULL) {
	int e = WSAGetLastError();
	closesocket(socket);
	std::cout << msg;
	std::cout << "\nerror code was: " << e;
	WSACleanup();
}

int send_l(SOCKET s, char *host, std::string path) {
	int er;
	char msg[DEFAULT_BUFFER_LENGTH];
	std::string st("GET ");
	std::string hostst(host);
	st += path;
	st += " HTTP/1.1\r\n";
	st += "Host: ";
	st += hostst;
	st += "\r\n\r\n";
	strcpy_s(msg, st.c_str());
	er = send(s, msg, DEFAULT_BUFFER_LENGTH, NULL);
	if (er == SOCKET_ERROR) {
		std::cout << "\nerror with sending: your last message was not sent\ntry again\n";
	}
	memset(msg, '\0', sizeof(msg));
	st.clear();
	return 0;
}

int recv_l(SOCKET s) {
	int er;
	char recvstr[DEFAULT_BUFFER_LENGTH];
	er = recv(s, recvstr, DEFAULT_BUFFER_LENGTH, NULL);
	if (er > 0) {
		std::cout << "\n\n___________________________________\n\n" << recvstr;
	} else if (er == 0) {
		std::cout << "connection terminated\n";
	} else {
		error("error with recv: the last message was not received\n", s);
		return (-1);
	}
	return 0;
}

int main(int argc, char *argv[]) {
	char *host = "";
	char *port = DEFAULT_PORT;
	int er;
	std::string path;

	//Large amounts of branching logic for inputs.
	if (argc <= 2) {
		error("Relaunch the program by typing '-h' and then a VALID host name (default port is 8888, do '-p' to change this).");
		return (-1);
	} else if (argc == 3) {
		if ((strcmp(argv[1], "-h") == 0)) {
			host = argv[2];
		} else {
			error("Relaunch the program by typing '-h' and then a VALID host name (default port is 8888, do '-p' to change this).\n");
			return (-1);
		}
	} else {
		error("Please enter valid input.");
		return (-1);
	}
	
	//declaring the winsock instance
	WSADATA WINSOCKYEAA;
	er = WSAStartup(MAKEWORD(2, 2), &WINSOCKYEAA);
	if (er != 0) {
		error("startup failed");
		return (-1);
	}

	//creating the socket
	SOCKET bill = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
	if (bill == INVALID_SOCKET) {
		error("socket creation failed");
		return (-1);
	}

	addrinfo hints, *result = NULL, *ptr = NULL;
	memset(&hints, '\0', sizeof(hints));
	hints.ai_family = AF_INET;
	hints.ai_socktype = SOCK_STREAM;
	hints.ai_protocol = IPPROTO_TCP;

	er = getaddrinfo(host, port, &hints, &result);
	if (er != 0) {
		error("getaddrinfo failed (maybe an invalid hostname or port)");
		return (-1);
	}

	std::cout << "connecting to server...\n";

	er = connect(bill, result->ai_addr, (int)result->ai_addrlen);
	if (er == SOCKET_ERROR) {
		error("connection failed with error code ");
		return (-1);
	}

	freeaddrinfo(result);
	
	std::cout << "connected to server\n";
	std::cout << "type a path to send to the server\n";
	std::cin >> path;
	send_l(bill, host, path);

	recv_l(bill);

	error("\nshutting down", bill);
    return 0;
}

EDIT: I found out that the characters displayed matches my maximum buffer length (2048). So the real question is how do I stop my program from printing all the characters from the buffer length?

[demoylan]

Also here is an example input-output chain from the program:

(windows prompt)>HTTPCLIENT -h httpbin.org
connecting to server...
connected to server
type a path to send to the server
/ip


___________________________________

HTTP/1.1 200 OK
Server: nginx
Date: Tue, 05 Jul 2016 16:01:16 GMT
Content-Type: application/json
Content-Length: 32
Connection: keep-alive
Access-Control-Allow-Origin: *
Access-Control-Allow-Credentials: true

{
  "origin": "96.244.245.45"
}
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠√
shutting down
error code was: 0
>

[demoylan]

Nevermind: I solved my own problem

The solution was to fill the receive string with nulls, so that when the program stopped receiving from the server, it would not continue receiving and get garbage.

Code:

char recvstr[DEFAULT_BUFFER_LENGTH];
memset(recvstr, '\0', DEFAULT_BUFFER_LENGTH);
er = recv(s, recvstr, DEFAULT_BUFFER_LENGTH, NULL);