socket TCP su C

Cimmo · 05-09-2002, 12:08

Ho questo problema:
lavoro su socket TCP su C, spedisco un sacco di messaggi da un host all'altro, pero' il secondo host non riceve niente (sta bloccato sulla read) finche' l'host che spedisce non chiude la connessione o gli faccio io un kill -9...perche'?
Se sono stato troppo vago fatemelo sapere...

ciao e grazie
Cimmo

cionci · 05-09-2002, 15:48

Diminuisci il buffer della read...

ilsensine · 05-09-2002, 18:05

Dovrebbe funzionare ugualmente, in quanto la read dovrebbe ritornare anche se è disponibile una quantità di dati inferiore a quella richiesta (blocca solo se non ci sono dati).
Puoi mandare il codice incriminato? (sia del client che del server)

Cimmo · 05-09-2002, 19:10

Non posso mandarvi tutto il codice perche' e' una pappardella che non ci cavereste i piedi, pero' vi mando quello che riceve e dopo un po' di debug ho scoperto qualcosa.
Io mi sono fatto una funzione per ricevere i messaggi (quella che posto dopo nel messaggio) e lui cosa fa: praticamente va' avanti a leggere pero' non becca mai il delimitatore '\0' e quindi non esce mai dal while e cosi' neanche dalla funzione. Diminuendo il buffer non fa altro che fare piu' giri di while per leggere tutto, pero' quando ha finito si inchioda lo stesso in attesa del e non esce.
Se pero' il processo che spedisce termina, allora il processo che contiene questa funzione si sblocca...mi sapreste dire il perche'?

grazie e 1000

ciao
Cimmo

int receiveData(int socketfd, char *msg)
{
int nread, n;

/* lettura */
nread=0;
printf ("\nread()\n");
fflush(stdout);

while (1)
{
if ((n=read(socketfd, &(msg[nread]), MAXSIZE)) <=0)
{
if (errno!=EINTR)
{
char msgerror[1024];
sprintf(msgerror,"read() failed [err %d] ",errno);
perror(msgerror);
fflush(stderr);
return(1);
}
}
else
{
nread+=n;

if ((msg[nread-1]=='\0') || (nread>=MAXSIZE))
break;
}
}

fflush(stdout);

return(0);
};

cionci · 05-09-2002, 19:45

Ma tu sei sicuro di spedirlo lo '\0' ?
Se la stringa che spedisci è lunga 10 caratteri...dovrai spedirne 11 con la send...

ilsensine · 05-09-2002, 21:41

Codice:

   if (errno!=EINTR)
   {
    char msgerror[1024];
    sprintf(msgerror,&quot;read() failed [err %d] &quot;,errno);
    perror(msgerror);
    fflush(stderr);
    return(1);
   }
  }
  else
  {
   nread+=n;

Dove sta scritto che la read setta errno=EINTR in caso di successo?

Codice:

   
   if ((msg[nread-1]=='\0') || (nread&gt;=MAXSIZE))
    break;

Sbagliato.
Nessuno può assicurarti che più pacchetti non siano stati riassemblati prima della read, dopo essere stati ad es. divisi a metà durante la trasmissione. Questo vuol dire che con una singola read ne puoi ricevere uno e mezzo, poi altre due metà ecc.
Devi controllare tutti i byte ricevuti in una read, alla ricerca di eventuali '\0' intermedi. Il tcp è stream, ma la dinamica del flusso non la puoi controllare.

Cimmo · 06-09-2002, 11:48

Quote:

Originariamente inviato da ilsensine
[b]Dove sta scritto che la read setta errno=EINTR in caso di successo?

No qui non hai capito tu, nel senso che se c'e' un errore (cioe' la read ha restituito un numero <=0) allora non tiene conto dell'errore EINTR che altro non e' che una interruzione dovuta ad una sistem call. Se invece la read restituisce un numero >0 va' nel ramo else e quell'if non lo esegue neanche.

Quote:

[b]
Sbagliato.
Nessuno può assicurarti che più pacchetti non siano stati riassemblati prima della read, dopo essere stati ad es. divisi a metà durante la trasmissione. Questo vuol dire che con una singola read ne puoi ricevere uno e mezzo, poi altre due metà ecc.
Devi controllare tutti i byte ricevuti in una read, alla ricerca di eventuali '

Quote:

Originariamente inviato da ilsensine
Dove sta scritto che la read setta errno=EINTR in caso di successo?

No qui non hai capito tu, nel senso che se c'e' un errore (cioe' la read ha restituito un numero <=0) allora non tiene conto dell'errore EINTR che altro non e' che una interruzione dovuta ad una sistem call. Se invece la read restituisce un numero >0 va' nel ramo else e quell'if non lo esegue neanche.

Quote:

Sbagliato.
Nessuno può assicurarti che più pacchetti non siano stati riassemblati prima della read, dopo essere stati ad es. divisi a metà durante la trasmissione. Questo vuol dire che con una singola read ne puoi ricevere uno e mezzo, poi altre due metà ecc.
Devi controllare tutti i byte ricevuti in una read, alla ricerca di eventuali '\0' intermedi. Il tcp è stream, ma la dinamica del flusso non la puoi controllare.

' intermedi. Il tcp è stream, ma la dinamica del flusso non la puoi controllare.

Si lo so che la read puo' leggere pezzi a caso, ma io allora come faccio? E poi comunque ho notato che anche quando ha letto tutto quello che c'era da leggere alla fine non trova lo stesso il '\0' e si blocca.
Addirittura succede cosi':
faccio partire il server che si mette in attesa, faccio partire il client che si connette e comincia a spedire al server il quale riceve, ma si inchioda come ho gia' spiegato, poi il client a suo volta lo faccio bloccare e tutto rimane bloccato. Se a manoni uccido (con kill -9) il client, il server si sblocca dalla read e mi ritorna magicamente tutto il messaggione con tutto quello che ho spedito.
Cosa devo fare?

ilsensine · 06-09-2002, 13:46

Quote:

No qui non hai capito tu, nel senso che se c'e' un errore (cioe' la read ha restituito un numero <=0) allora non tiene conto dell'errore EINTR che altro non e' che una interruzione dovuta ad una sistem call. Se invece la read restituisce un numero >0 va' nel ramo else e quell'if non lo esegue neanche.

Ok avevo perso qualche parentesi

Quote:

Si lo so che la read puo' leggere pezzi a caso, ma io allora come faccio? E poi comunque ho notato che anche quando ha letto tutto quello che c'era da leggere alla fine non trova lo stesso il '

Quote:

No qui non hai capito tu, nel senso che se c'e' un errore (cioe' la read ha restituito un numero <=0) allora non tiene conto dell'errore EINTR che altro non e' che una interruzione dovuta ad una sistem call. Se invece la read restituisce un numero >0 va' nel ramo else e quell'if non lo esegue neanche.

Ok avevo perso qualche parentesi

Quote:

Si lo so che la read puo' leggere pezzi a caso, ma io allora come faccio? E poi comunque ho notato che anche quando ha letto tutto quello che c'era da leggere alla fine non trova lo stesso il '\0' e si blocca.
Addirittura succede cosi':
faccio partire il server che si mette in attesa, faccio partire il client che si connette e comincia a spedire al server il quale riceve, ma si inchioda come ho gia' spiegato, poi il client a suo volta lo faccio bloccare e tutto rimane bloccato. Se a manoni uccido (con kill -9) il client, il server si sblocca dalla read e mi ritorna magicamente tutto il messaggione con tutto quello che ho spedito.
Cosa devo fare?

' e si blocca.
Addirittura succede cosi':
faccio partire il server che si mette in attesa, faccio partire il client che si connette e comincia a spedire al server il quale riceve, ma si inchioda come ho gia' spiegato, poi il client a suo volta lo faccio bloccare e tutto rimane bloccato. Se a manoni uccido (con kill -9) il client, il server si sblocca dalla read e mi ritorna magicamente tutto il messaggione con tutto quello che ho spedito.
Cosa devo fare?

Innanzitutto verifica quello che ha detto cionci, ovvero che se devi spedire una stringa s devi trasmettere strlen(s)+1 byte.
Poi potresti mettere dei messaggi di debug dopo la read per vedere quello che succede.
Nota che il risultato della read è <0 se c'è stato un errore e ==0 se il client si è disconnesso (e in questo caso errno rimane indefinito)

Cimmo · 06-09-2002, 14:42

Quote:

Originariamente inviato da ilsensine
[b]
Ok avevo perso qualche parentesi

Innanzitutto verifica quello che ha detto cionci, ovvero che se devi spedire una stringa s devi trasmettere strlen(s)+1 byte.
Poi potresti mettere dei messaggi di debug dopo la read per vedere quello che succede.
Nota che il risultato della read è <0 se c'è stato un errore e ==0 se il client si è disconnesso (e in questo caso errno rimane indefinito)

Ma sto '\0' glielo devo mettere io o mi basta che durante la spedizione do' come lunghezza della stringa strlen(s)+1? Il mio professore dell'universita' mi ha detto che non dovevo fare cosi'...AIUTO!
Comunque il debug l'ho fatto (con DDD) e mi si inchioda sulla read non boccheggia il '\0'...

cionci · 06-09-2002, 15:15

Se è una stringa lo '\0' c'è sempre in fondo...e per definizione una stringa in C è una sequenza di caratteri a aprtire da un certo indirizzo con un '\0' che indica la fine della stringa...

Quindi se quello che mandi è una stringa allora c'è lo '\0' in fondo e per mandarlo devi dare la dimensione strlen(str) + 1 alla send...

Cimmo · 06-09-2002, 15:27

Ok nella send ho messo strlen(s)+1, io faccio 5 write:
1) scrivo 1/
2) scrivo 6/
3) scrivo 247/
4) scrivo 1/
5) scrivo 3/

e il risultato e' che invece di arrivarmi 1/6/247/1/3/ mi arriva cosi': 1/\0006/\000247/\0001/\0003/\000

perche'?

grazie della pazienza!

cionci · 06-09-2002, 15:53

Ci puoi far vedere il modo in cui costruisci i messaggi che invii ?

ilsensine · 06-09-2002, 15:54

Congruente con quello che ti ho detto. Leggendo MAXSIZE byte, dici al kernel di "svuotare" la coda tcp fino ad un massimo di MAXSIZE byte. Ovviamente i pacchetti che sono arrivati fin'ora vengono "impacchettati" e letti tutti insieme dalla read().
Soluzioni possibili:
- Leggi 1 byte per volta (bleah!)
- Dividi la lettura in 2 strati: uno (quello attuale) che legge i dati e li mette in un buffer fregandosene di trovare gli '\0'; un altro che legge dal buffer, cerca la prima occorrenza di '\0' ed estrae la stringa dal buffer.
- Prima di spedire una stringa, spedisci un int indicante la sua lunghezza. Così non hai neanche necessità di inviare lo '\0' finale.

cionci · 06-09-2002, 15:58

Quote:

Originariamente inviato da Cimmo
[b]Ok nella send ho messo strlen(s)+1, io faccio 5 write:
1) scrivo 1/
2) scrivo 6/
3) scrivo 247/
4) scrivo 1/
5) scrivo 3/

e il risultato e' che invece di arrivarmi 1/6/247/1/3/ mi arriva cosi': 1/

Quote: