Problem mit recv: Es werden falsche Daten ausgegeben

DirkB

@Bassmaster Sind es immer verschiedene zufällige Zeichen?
Wird auch die richtige Datei ausgewählt?

Nur ein Zeichen mit recv zu empfangen ist etwas ineffektiv.

Wozu benutzt du einen vector als Puffer, wenn du genau ein Zeichen liest?

std::cout<<&buf[0]; Da der Puffer kein terminierendes Nullbyte enthält, ist das falsch.

Bassmaster

Hallo, ja einen vector kann ich mir sparen keine Ahnung wieso ich den gestern verwendet hatte.
Jedenfalls lag das Problem beim get request. Mit dem request welchen ich zuerst verwendet hatte gab der Server mir zwar ein OK allerdings kam es ja zu dem oben erwähnten Problem. Jetzt habe ich in einem anderen Code gesehen das da der get request anders aussah. Ich habe meinen request string etwas angepasst und danach funktionierte es.

get request von gestern:

string str =
"GET /test.txt HTTP/1.1\x0d\x0a"
"Host: www.example.com\x0d\x0a"
"User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:69.0) Gecko/20100101 Firefox/69.0\x0d\x0a"
"Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8\x0d\x0a"
"Accept-Language: de,en-US;q=0.7,en;q=0.3\x0d\x0a"
"Accept-Encoding: gzip, deflate\x0d\x0a"
"Connection: keep-alive\x0d\x0a"
"Upgrade-Insecure-Requests: 1\x0d\x0a"
"\x0d\x0a\x0d\x0a";

Dieser get request entstand dadurch das ich mit Wireshark und dem Firefox Plugin HTTP Header Live mir angeschaut hatte wie so requests aussehen. Ohne weiter drüber nachzudenken hatte ich diesen so übernommen ....

Und jetzt der überarbeitete request:

string str =
"GET /test.txt HTTP/1.1\x0d\x0a"
"Host: www.example.com\x0d\x0a"
"\x0d\x0a\x0d\x0a";

ok \x0d und \x0a kann ich noch jeweils durch \n \r ersetzten.

	long TotalReceivedBytes=0;
	long rc=0;

	char buf[512]={'\x00'};

	do
	{
		rc = recv( net.DataTransferSocket, &buf[0], 512 ,0);
        buf[rc]='\x00';
		if ( rc > 0)
		{
            TotalReceivedBytes = TotalReceivedBytes + rc;
			std::cout<<&buf[0];
		}

		else if ( rc == 0 )
		{
            std::cout<<"\n";
			std::cout<<"connection closed.\n";
			return -1;
		}

		else
		{
			std::cout<<"recv error: "<<WSAGetLastError()<<"\n";
			return -2;
		}
	}
	while( rc > 0  );

	return TotalReceivedBytes;

@Bassmaster
std::cout<<&buf[0]; das ist immer noch falsch.

Bassmaster

@manni66 sagte in Problem mit recv: Es werden falsche Daten ausgegeben:

@Bassmaster
std::cout<<&buf[0]; das ist immer noch falsch.

rc = recv( net.DataTransferSocket, &buf[0], 512 ,0);
        buf[rc]='\x00';

Es wird hier doch Nullterminiert abgesehen davon ist der ganze buffer mit einem Nullterminator initialisiert.

WIe macht man es den richtig?

@Bassmaster sagte in Problem mit recv: Es werden falsche Daten ausgegeben:

@manni66 sagte in Problem mit recv: Es werden falsche Daten ausgegeben:

@Bassmaster
std::cout<<&buf[0]; das ist immer noch falsch.
rc = recv( net.DataTransferSocket, &buf[0], 512 ,0);
        buf[rc]='\x00';
Es wird hier doch Nullterminiert abgesehen davon ist der ganze buffer mit einem Nullterminator initialisiert.

WIe macht man es den richtig?

Die Zuweisung habe ich übersehen, weil sie da auch falsch ist. buf[-1] =…?
Ebenso buf[512] =….

Belli

@Bassmaster sagte in Problem mit recv: Es werden falsche Daten ausgegeben:

WIe macht man es den richtig?

        long TotalReceivedBytes=0;
	long rc=0;

	char buf[512]={'\x00'};

	do
	{
		rc = recv( net.DataTransferSocket, buf, 511 ,0);  //Platz für 0-Byte lassen!
        
		if ( rc > 0)
		{
                     buf[rc] = '\x00';  //nur, wenn rc nicht negativ!
                     TotalReceivedBytes = TotalReceivedBytes + rc;
	             std::cout << buf;
		}

		else if ( rc == 0 )
                ...

DirkB

man kann auch buf vergrößern.

0 ist auch nicht negativ.

Belli

@DirkB
buf[rc] = 0 für rc == 0 ist auch kein Problem.

Edit:
Ah ja ... Blödsinn, mein Kommentar ist natürlich falsch

Bassmaster

Ja, buf[512]; Element 0 bis 511 sind reserviert. Ich habe länger nicht programmiert wird Zeit nochmal ein paar Grundlagen wieder aufzufrischen ....

DirkB

@Bassmaster Bei recv gibst du die Anzahl (Größe) an.