SOCKADDR_IN remoteAddr;
vector<SOCKADDR_IN*> clients;
bytes=recvfrom(s,buf,BUF_SIZE,0,(SOCKADDR*)&remoteAddr,&addrlen);
clients.push_back(&remoteAddr);

Warum funktioniert folgender Aufruf nicht? (keine Fehlermeldung)

bytes=sendto (s,buf,strlen(buf),0,(SOCKADDR*)clients[0],sizeof(SOCKADDR_IN));

Gruß,
Marcel

Ein bisschen wenig Kontext, aber es sieht so aus, als würdest Du SOCKADDR_IN Pointer in clients speichern. Vermutlich ist das Objekt danach aber nicht mehr gültig (z.B. weil Du recvfrom(..) und sendto(..) in verschiedenen Scopes hast).
Simon

scopes? wie kann ich denn das problem umgehen das es dennoch gueltig ist?

Oder:
Du erzeugst deine SOCKADDR_IN Objekte auf dem Heap. (Löschen nicht vergessen...).

Simon

Edit: Ich sehe gerade, dass kopieren des Vectors und deren Elemente kein Problem darstellt. std::vector geht auch.

Entweder:
Speichere die SOCKADDR_IN Objekte in der Liste als Wert, nicht als Pointer.
Beim Zugriff holst Du dir dann (via Referenz) den Pointer.
Dazu ist aber eine std::list nötig (std::vector geht nicht, da allenfalls umkopiert wird).

Oder:
Du erzeugst deine SOCKADDR_IN Objekte auf dem Heap. (Löschen nicht vergessen...).

Simon

Edit: Ich sehe gerade, dass kopieren des Vectors und deren Elemente kein Problem darstellt. std::vector geht auch.

Mhh okay, war folgendes gemeint?

SOCKADDR_IN remoteAddr;
vector<SOCKADDR_IN*> clients;
bytes=recvfrom(s,buf,BUF_SIZE,0,(SOCKADDR*)&remoteAddr,&addrlen);
clients.push_back(remoteAddr); 
.....
bytes=sendto (s,buf,strlen(buf),0,(SOCKADDR*)&clients[0],sizeof(SOCKADDR_IN));

das funktioniert bei mir nämlich nicht. Habe ich einen Denkfehler?

Zeigmal ein wenig mehr Code.

Ist das Teil einer Klasse? Was sind Member (der clients vector?)?

Simon

Ich meinte nicht so. Du speicherst immer noch Pointer.

Zeigmal ein wenig mehr Code.

Ist das Teil einer Klasse? Was sind Member (der clients vector?)?

Simon

Nein ich speicher jetzt nicht mehr als pointer, wie dem "&" zu entnehmen ist. Ich habe nur das * vergessen zu löschen: vector<SOCKADDR_IN*> clients; was ich selbstverständlich im code getan habe.

Ich bin dankbar fuer weitere Vorschläge Ich wüsste jetzt nicht genau was dir noch an code weiter helfen wuerde?!

theta schrieb:

Ich meinte nicht so. Du speicherst immer noch Pointer.

Zeigmal ein wenig mehr Code.

Ist das Teil einer Klasse? Was sind Member (der clients vector?)?

Simon

Nein ich speicher jetzt nicht mehr als pointer, wie dem "&" zu entnehmen ist. Ich habe nur das * vergessen zu löschen: vector<SOCKADDR_IN*> clients; was ich selbstverständlich im code getan habe.

Ich bin dankbar fuer weitere Vorschläge Ich wüsste jetzt nicht genau was dir noch an code weiter helfen wuerde?!

*g* ja, habe nur bis vector<..> gelesen..

Zeig doch mehr code, da kann Dir besser geholfen werden.
Simon

// Erster aufruf + Callback
void DLL_EXPORT ext_refresh() {
        bytes=recvfrom(s,buf,BUF_SIZE,0,(SOCKADDR*)&remoteAddr,&addrlen);
        buf[bytes]='\0';
        clients.push_back(&remoteAddr);
        (*sp_on)(buf);
}
// Funktionsaufruf, um auf "clients" zuzugreifen
char DLL_EXPORT *ext_getpath(int client) {
    string strcheck;
    char *get1 = "Hallo12";

    sprintf(buf,"give");
    sendto (s,buf,strlen(buf),0,(SOCKADDR*)clients[0],addrlen);
    Sleep(10);
    bytes=recvfrom(s,buf,BUF_SIZE,0,(SOCKADDR*)&remoteAddr, &addrlen);
    buf[bytes] = '\0';
    strcpy(get1, buf);

    return get1;
}

vector<SOCKADDR_IN*> clients; // ist in der dll global deklariert.

rufe ich clients[0] direkt nach dem anlegen mit sendto auf, funktioniert es einwandfrei, nur nach dem callbkac und dem aufruf der zweiten funktion nicht mehr. Danke fuer eure Hilfe

Edit:
Das löscht ja Volkard wahrscheinlich wieder, aber zu meiner Verteidigung, es ist nicht böse gemeint.

Sorry, ich habe keine Nerven mehr.
Da ist sovieles Durcheinander....
Simon

Edit:
Das löscht ja Volkard wahrscheinlich wieder, aber zu meiner Verteidigung, es ist nicht böse gemeint.

genau das gleiche hatte ich mir auch gedacht xD
aber dann mal doch wenigsten noch das, was mir als allererstes aufgefallen ist:

char* x = "asd";
x[1] = 'S';

das hier ist verboten - damit ist insbesondere auch dein strcpy verkehrt.
(literale dürfen in einem speicher mit nur lese-rechten gespeichert werden -> schreiboperationen auf adressen in diesem bereich führen zu undefiniertem verhalten.)

die zeile char* x = "asd"; gibt imho schon eine warning, wegen des vergessenen cv-qualifiers... (google mal nach const correctness)

du brauchst also die länge...
bzw. eine feste länge für einen puffer und verwendest dann std::string bzw std::vector<char> - je nach dem, was logischer erscheint...

Sleep(10);
wieso tust du das?
was ist, wenn eine übermittlung mal länger als 10ms dauert? (was ja gar nicht mal so unrealistisch ist...)

etc...

bb