2012-07-08 43 views
0

這是一個簡單執行RETR cmd其中服務器第一次收到文件名,然後它發送文件。c socket發送的文件中包含其他消息

/************************* RECEIVE FILE NAME AND SEND FILE *************************/ 
if(recv(newsockd, buffer, sizeof(buffer), 0) < 0){ 
    perror("error receiving file name"); 
    onexit(newsockd, sockd, 0, 2); 
} 
other = strtok(buffer, " "); 
filename = strtok(NULL, "\n"); 
if(strcmp(other, "RETR") == 0){ 
    printf("received RETR request\n"); 
} else onexit(newsockd, sockd, 0, 2); 

fd = open(filename, O_RDONLY); 
    if (fd < 0) { 
    fprintf(stderr, "cannot open '%s': %s\n", filename, strerror(errno)); 
    onexit(newsockd, sockd, 0, 2); 
} 

if(fstat(fd, &fileStat) < 0){ 
    perror("Error fstat"); 
    onexit(newsockd, sockd, fd, 3); 
} 
fsize = fileStat.st_size; 
if(send(newsockd, &fsize, sizeof(fsize), 0) < 0){ 
    perror("Error on sending file size\n"); 
    onexit(newsockd, sockd, fd, 3); 
} 

rc = sendfile(newsockd, fd, &offset, fileStat.st_size); 
if(rc == -1) { 
     fprintf(stderr, "error sending file: '%s'\n", strerror(errno)); 
     onexit(newsockd, sockd, fd, 3); 
} 
if((uint32_t)rc != fsize) { 
    fprintf(stderr, "transfer incomplete: %d di %d bytes sent\n", rc, (int)fileStat.st_size); 
    onexit(newsockd, sockd, fd, 3); 
} 
memset(buffer, 0, sizeof(buffer)); 
strcpy(buffer, "226 File trasferito con successo\n\0"); 
if(send(newsockd, buffer, strlen(buffer)+1, 0) < 0){ 
    perror("Errore durante l'invio 226"); 
    onexit(newsockd, sockd, 0, 2); 
} 
memset(buffer, 0, sizeof(buffer)); 
strcpy(buffer, "221 Goodbye\n\0"); 
if(send(newsockd, buffer, strlen(buffer)+1, 0) < 0){ 
    perror("Errore durante l'invio 221"); 
    onexit(newsockd, sockd, 0, 2); 
} 
/************************* END PART *************************/ 

,這是客戶端程序的代碼片段:

/************************* SEND FILE NAME AND RECEIVE FILE *************************/ 
printf("Inserire il nome del file da scaricare: "); 
if(fgets(dirpath, BUFFGETS, stdin) == NULL){ 
    perror("fgets name file"); 
    close(sockd); 
} 
filename = strtok(dirpath, "\n"); 
sprintf(buffer, "RETR %s", dirpath); 
if(send(sockd, buffer, strlen(buffer), 0) < 0){ 
    perror("error sending file name"); 
    close(sockd); 
    exit(1); 
} 
if(read(sockd, &fsize, sizeof(fsize)) < 0){ 
    perror("error on receiving file size\n"); 
    close(sockd); 
    exit(1); 
} 
fd = open(filename, O_CREAT | O_WRONLY, 0644); 
if (fd < 0) { 
    perror("open"); 
    exit(1); 
} 

while(((uint32_t)total_bytes_read != fsize) && ((nread = read(sockd, filebuffer, fsize)) > 0)){ 
    if(write(fd, filebuffer, nread) < 0){ 
     perror("write"); 
     close(sockd); 
     exit(1); 
    } 
    total_bytes_read += nread; 
} 
memset(buffer, 0, sizeof(buffer)); 
if(recv(sockd, buffer, 34, 0) < 0){ 
    perror("Error receiving 226"); 
    close(sockd); 
    exit(1); 
} 
printf("%s", buffer); 
memset(buffer, 0, sizeof(buffer)); 
if(recv(sockd, buffer, 13, 0) < 0){ 
    perror("Error receiving 221"); 
    close(sockd); 
    exit(1); 
} 
printf("%s", buffer); 
memset(buffer, 0, sizeof(buffer)); 
close(fd); 
/************************* END PART *************************/ 

問題是什麼?
問題是,已發送的文件還包含由服務器發送的2個消息(226和221),我不知道爲什麼我得到這個行爲烏
例子:
RETR tryfile.txt
File tryfile.txt received
cat tryfile.txt
「這是tryfile.txt,你好客戶
226文件trasferito CON SUCCESSO
221再見」

+3

1)假設消息邊界由TCP保存。他們不是。 2)你假設NUL終止的字符串,但你不發送NUL。 3)IIRC,基於行的協議應該用'\ r \ n'來終止行。 – wildplasser 2012-07-08 13:27:52

+0

1)呃?我不明白,sorrhy :(2)更正'發送'與'strlen(緩衝區)+ 1' 3)我假設使用正常線:)這是一個非常簡單的項目 – polslinux 2012-07-08 13:37:38

+0

@wildplasser:雖然反直覺,絕對使用'\ r \ n'。雖然這是Windows的行結束,但我的經驗是,儘管Windows在接收UNIX行尾時抱怨並斷裂。但是,在標準的UNIX方式下,UNIX系統很樂意從DOS格式轉換成我,並沒有抱怨等。 – Linuxios 2012-07-08 13:38:36

回答

0

我發現並(終於)解決了這個問題。
這裏是解決方案:

uint32_t fsize_tmp = fsize; 
while(((uint32_t)total_bytes_read != fsize) && ((nread = read(sockd, filebuffer, fsize_tmp)) > 0)){ 
    if(write(fd, filebuffer, nread) < 0){ 
    perror("write"); 
    close(sockd); 
    exit(1); 
     } 
     total_bytes_read += nread; 
     fsize_tmp -= nread; 
} 

問題是由於我沒有勾選「動態」的文件大小(如果文件在75%我while循環FSIZE再次預計發送,這是不可能的),所以與此代碼我會)它是需要:)

+2

您仍然沒有檢查寫入的返回值。它可能小於未讀。另外:讀取可能返回零,導致片段永遠循環。 – wildplasser 2012-07-08 15:49:42

+0

哦謝謝!我會糾正它! – polslinux 2012-07-08 16:49:43

+0

,但我已經把條件'nread> 0',所以如果0返回沒有問題:) – polslinux 2012-07-08 16:51:18

1

strlen的(每次不減小文件大小算\ 0,這樣的strlen()將返回12:

strcpy(buffer, "221 Goodbye\n\0"); 
if(send(newsockd, buffer, strlen(buffer), 0) < 0){ ...} 

客戶端使用硬編碼值13.(對於其他狀態消息,其中發送33並且期望34)的相同。)BTW:真的需要一些緩衝機制,至少在客戶端。

UPDATE:

#include <stdio.h> 
#include <string.h> 

int main(void) 
{ 
fprintf(stderr, "strlen is %u\n", (unsigned) strlen("221 Goodbye\n\0")); 

return 0; 
} 

說明:: 要以 「嵌入的空」 顯示一個字符串的函數strlen的strlen()只計數的字符,直到遇到 '\ 0' 字符

+0

在上面的評論我寫了,我糾正了我的代碼:)但我沒有更新的問題抱歉!但是它有什麼問題呢?這些是預定義的字符串,所以我的代碼有什麼問題?謝謝! – polslinux 2012-07-08 18:07:43

+0

你可以在源文件中加入額外的調試'fprintf(stderr,「strlen is%u \ n」,(unsigned)strlen(「221 Goodbye \ n \ 0」));'。 – wildplasser 2012-07-08 18:12:46

+0

是的,strlen是13 :)我不明白你想對我說什麼:( – polslinux 2012-07-08 18:16:21