Ancora read e write su socket in C

[Cimmo]

Sto di nuovo sotto il mio programmozzo...devo spedire da A a B un pacchetto, ho deciso di spedire prima una sorta di header che dice quanto e' lungo il pacchetto che poi spedisco dopo l'header. Ho fatto 2 funzioni, ma non so perche' l'header arriva, ma poi quando deve ricevere il pacchetto chi spedisce si ferma in un: write() failed [err 104]: Connection reset by peer mentre chi riceve piglia sto errore: read() failed [err 14]: Bad address

i 2 codici sono questi prima chi spedisce:

int sendData(int socketfd, char msg[MAXSIZE])
{
        long nwrite, len, n;

        /* scrittura */
        len=strlen(msg);
        nwrite=0;

        do
        {
                if ((n=write(socketfd, &len, (HEADERSIZE-nwrite)))<0)
                {
                        if (errno!=EINTR)
                        {
                                char msgerror[1024];
                                sprintf(msgerror,"write() failed [err %d]",errno);
                                perror(msgerror);
                                fflush(stderr);

                                return(1);
                        }
                }
                else
                        nwrite+=n;
        } while (nwrite<HEADERSIZE);

        nwrite=0;

        while ((len-nwrite)>0)
        {
                if ((n=write(socketfd, &(msg[nwrite]), (len-nwrite)))<0)
                {
                        if (errno!=EINTR)
                        {
                                char msgerror[1024];
                                sprintf(msgerror,"write() failed [err %d]",errno);
                                perror(msgerror);
                                fflush(stderr);

                                return(1);
                        }
                }
                else
                        nwrite+=n;
        }

        return(0);
};

chi riceve invece ha:

int receiveHeaderAndData(int socketfd, char *msg)
{
        long n,nread,header;

        nread=0;

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

        nread=0;

        printf("\n\nSocketfd: %d:  header: %ld\n\n",socketfd,header);

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

        return(0);
}

[cionci]

Io direi prima di tutto di cambiare i cicli di lettura in questo modo :

        do
        { 
                //Nota il 3o parametro della write
                if ((n=write(socketfd, &len, HEADERSIZE-nwrite))<0) 
                { 
                        if (errno!=EINTR) 
                        { 
                                char msgerror[1024]; 
                                sprintf(msgerror,"write() failed [err %d]",errno); 
                                perror(msgerror); 
                                fflush(stderr); 

                                return(1); 
                        } 
                } 
                else nwrite+=n; 
        } while(nwrite == HEADERSIZE)

Ora prova anche se non credo che sia quello l'errore...
Ah anche durante la lettura del messaggio devi emttere header-nread...

[Cimmo]

Originally posted by "cionci"

Io direi prima di tutto di cambiare i cicli di lettura in questo modo :

        do
        { 
              ...

        } while(nwrite == HEADERSIZE)

Ora prova anche se non credo che sia quello l'errore...
Ah anche durante la lettura del messaggio devi emttere header-nread...

Beh se metto il do while devo mettere != e non ==
Inoltre anche if ((n=read(socketfd, &header, HEADERSIZE))<0) devo mettere (HEADERSIZE-nread) ?
In ogni caso non funzia...

[Cimmo]

Ho modificato il post iniziale con le nuove funzioni...HELP ME!

[cionci]

Hai ragione su ==. Metti comunque < non != non si testa mai il diverso su un incremento altrimenti in qualche modo (magari per sbaglio) potresti rischiare di andare in ciclo

Riguardo all'errore...boh... Magari me lo guardo meglio...

[Cimmo]

Originally posted by "cionci"

Hai ragione su ==. Metti comunque < non != non si testa mai il diverso su un incremento altrimenti in qualche modo (magari per sbaglio) potresti rischiare di andare in ciclo

Riguardo all'errore...boh... Magari me lo guardo meglio...

Ok fatta la modifica del while!
Odio quando non riesco a capire il perche' delle cose -> ->

[Cimmo]

Eh eh eh...ilsensine ne sa una in piu' del diavolo...o forse sono io che non ne so ancora abbastanza?
Beh l'importante e' aver messo a posto il tutto

[cionci]

Ehm...dov'è il post di ilsensine ? Dov'era il problema...

[Cimmo]

Originally posted by "cionci"

Ehm...dov'è il post di ilsensine ? Dov'era il problema...

Diciamo che l'ho impezzato da un'altra parte
Prima di tutto mi ha fatto mettere sizeof(len) e sizeof(header) nei primi 2 while delle funzioni e poi il problema era che il chiamante della funzione read non allocava la stringa cioe' la dichiaravo cosi': char *msg; e invece lui la voleva dichiarata cosi': char msg[MAXSIZE]; Non capisco perche' la prima dichiarazione funziona in certi altri casi, ma questa volta no...ok non e' allocato lo spazio, ma il C non dovrebbe allocarlo quando ci scrivi?

[cionci]

Ecco perchè non andava !!!!