zurzeit versuche ich mit Sockets ein Binary File (ein Bild) herunterzuladen. Das funktioniert, jedoch wird es nicht richtig abgespeichert..

Zuerst der Code:

#include <sys/types.h>
#include <sys/socket.h>
#include <netdb.h>
#include <arpa/inet.h>
#include <iostream>
#include <string.h>
#include <fstream>
using namespace std;

int main(void)
{
int s = socket(AF_INET, SOCK_STREAM, 0);
struct hostent *host = gethostbyname("www.google.de");
struct sockaddr_in addr;

addr.sin_addr = *(struct in_addr *)host->h_addr;
addr.sin_port = htons(80);
addr.sin_family = AF_INET;

connect(s, (sockaddr *)&addr, sizeof(addr));
string rrr = "GET /intl/de_de/images/logo.gif HTTP/1.1\r\nHost: www.google.de\r\nConnection: close\r\n\r\n";
send(s, rrr.c_str(), rrr.size(), 0);

char recieved[1024];
memset(recieved, 0, sizeof(recieved));fstream f1("logo.gif", ios::out | ios::binary);
while(recv(s, recieved, sizeof(recieved )-1, 0))
{
f1 << recieved;

memset(recieved, 0, sizeof(recieved));
}

f1.close();
}

Ja, die HTTP-Response headers werden mit in die Datei geschrieben, aber das ist ein Problem, welches nicht besprochen werden muss da die Lösung mir geläufig ist
Das Problem ist scheinbar die Nullbyte-Terminierung. Öffne ich das "richtig heruntergeladene" Logo der Suchmaschine mit einem Hexeditor, so erhalte ich:

47 49 46 38 39 61 2d 01 6e 00 e7

Die Version meines Programms:

47 49 46 38 39 61 2d 01 6e 82 5a 69 0e...

Hier ist das Nullbyte laut meiner Vermutung das Problem.Ich glaube, ich sollte Memory-Funktionen verwenden,
statt String-Funktionen um diese entsprechen zu behandeln. Weiss jemand genau, wie man das Bild richtig abspeichert?

Dann ist eine einfache Lösung, die Ausgabe byteweise zu machen.

#include <sys/types.h>
#include <sys/socket.h>

Und ich glaube, Windows Compiler würden unter anderem diese Headerdateien nicht finden ;).

Dann ist eine einfache Lösung, die Ausgabe byteweise zu machen.

und wie?

Ich habe das Gefühl, du weisst nicht, von was du sprichst. recv ist keineswegs Windows-Kram.

War der Top Treffer bei ggogle für recv. Aber ist für das Problem ja auch völlig egal.

und wie?

for (int i=0; i<1024;++i) f1.put(recieved[i]);

edit: Oder auch mit write(), ist wahrscheinlich sogar besser.

edit2: Ja, ist besser:

f1.write(recieved,1024);

Nur mal als Denkanstoß: Du gehst wohl nicht davon aus, dass jedes Bild bei Google exakt 1024 Bytes lang ist. Dir ist zudem bereits bewusst, dass Nullbytes C-Strings terminieren. Eine Länge speicherst Du auch nirgends.
Ja, wie soll denn der operator<< nun überhaupt feststellen können, wie lang die Daten sind?