[C] scambio dati fra client e server

[Eddie1985]

Salve a tutti, devo sviluppare un'applicazione client-server in C (che giri sotto un sistema UNIX-like); client e server devono comunicare tramite socket TCP.
Fra le altre cose questa applicazione deve svolgere la seguente operazione:
- il client spedisce un file al server, che lo riceve e lo memorizza in una directory

Il mio problema è proprio nello scambio di dati, quindi posto esclusivamente la parte di codice relativa lasciando perdere il resto. In particolare posterò l'operazione svolta da un mio amico (che funziona) e la mia soluzione (che non funziona: Segmentation fault), nella speranza che mi diciate perchè.

// soluzione non funzionante parte server
char *buff = NULL;
while (lunghezza_file > 0) {
     recv (sock, (char *)buff, 1, 0);
     fwrite (buff, 1, 1, fp);
}
//soluzione non funzionante parte client
char *buff = NULL;
while (!feof (fp)) {
	fread (buff, 1, 1, fp);
	send (sk, buff, 1, 0);
}

// soluzione funzionante parte server
char buff [1024];
int cont = 1;
while (lunghezza_file >= cont*1024) {
	memset (buff, 0, 1024);
	recv (sock, (char *)buff, 1024, 0);
	fwrite (buff, 1, 1024, fp);
	cont++;
}
						
if (lunghezza_file % 1024 > 0) {
	recv (sock, buff, (lunghezza_file % 1024), 0);
	fwrite (buff, 1, (lunghezza_file % 1024), fp);
}
// soluzione funzionante parte client
char buff [1024];
cont = 0;
while (!feof (fp)) {
	fread (buff, 1, 1024, fp);
	if ((info.st_size - cont*1024) >= 1024) 
		send (sk, buff, 1024, 0);
	else 
	        send (sk, buff, (info.st_size % 1024), 0);
	cont++;
}

Vi ringrazio

[Eddie1985]

nessuno sa aiutarmi?

[starfred]

La follia di scrivere un'area di memoria non allocata .
Scherzi a parte ci sono diversi errorucci.

Per prima cosa tu fai un loop infinito nel:

while (lunghezza_file > 0)

Seconda cosa: "buff" è un puntatore a NULL. Tu scrivi su una zona di memoria non allocata.

Nella soluzione corretta infatti tu noterai un bel char buff [1024]; e poi un ciclo che: pulisce buff, riceve i dati, li mette in buff che poi lo salva su file ed incrementa il contatore per sapere quando finisce il file.

[Eddie1985]

Originariamente inviato da starfred

La follia di scrivere un'area di memoria non allocata .
Scherzi a parte ci sono diversi errorucci.

Per prima cosa tu fai un loop infinito nel:

while (lunghezza_file > 0)

Seconda cosa: "buff" è un puntatore a NULL. Tu scrivi su una zona di memoria non allocata.

Nella soluzione corretta infatti tu noterai un bel char buff [1024]; e poi un ciclo che: pulisce buff, riceve i dati, li mette in buff che poi lo salva su file ed incrementa il contatore per sapere quando finisce il file.

si dopo la fwrite mi sono dimenticato di riportare lunghezza_file--
in questo modo il problema del ciclo infinito sarebbe risolto?

e per quanto riguarda la zona di memoria non allocata...non sono sicuro di aver ben compreso il problema....se io scrivessi char buff [1] invece che char *buff cambierebbe qualcosa?

[Eddie1985]

cioè mi spiego meglio...io vorrei inviare un byte alla volta....sulla falsariga di quanto scritto sopra farei così:

//parte client
char buffer [1];
memset (buffer, 0, 1);
while (!feof(fp)) {
	fread (buffer, 1, 1, fp);
	send (sd, buffer,1,0);
}

//parte server
char buffer [1];
while (lunghezza_file > 0) {
        memset (buffer, 0, 1);
	recv (sock_des, (char*)buffer, 1, 0);
	fwrite (buffer, 1, 1024, fp);
	lunghezza_file--;
}

Così però mi dà segmentation fault.
Perchè? e cmq avete altre soluzioni?
grazie

[starfred]

1024 ?
Guardati PER BENE cosa fa la fwrite.

[Eddie1985]

ok errore di distrazione...
mettendo 1 al posto di 1024 funziona.....
grazie!