2012-02-06 6 views
1

cでソケットプログラミングを使用して、あるコンピュータから別のコンピュータにオーディオファイルを送信しようとしています。ヘッダーやテイラーなどのフレーミング情報なしで単純な文字列を送信すると、完全に送信されます。しかし、socket_dataやpacket_noのようなヘッダ情報を使って同じ情報を送信しようとすると、それは正しく送信されません。両方のマシンで端末出力さえSAMEですが、作成されるファイルはまったく異なり、再生できません。私はパケットを送信するためにシリアライズの概念を使用しています。コードを添付しています。何が間違っているのかご意見ください。cソケットを使用したオーディオファイルの送信

Server.c

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 
#include <sys/types.h> 
#include <sys/socket.h> 
#include <netinet/in.h> 
#include <fcntl.h> 

void set_socket(struct sockaddr_in *socket, int type, int host_short, int addr_type) 
{ 
    socket -> sin_family = type; 
    socket -> sin_port = htons(host_short); 
    socket -> sin_addr.s_addr = htonl(addr_type); 
} 

void serialize(char *buffer, int count, char *data) 
{ 
    int i=0, j=0; 
    char temp1[20]; 

    sprintf(temp1, "%d", count); 
    while(temp1[i] != '\0') 
    { 
     buffer[j++] = temp1[i++]; 
    } 
    buffer[j++]=' '; 

    for(i=0; data[i] != '\0'; i++) 
    { 
     buffer[j++] = data[i]; 
    } 
    buffer[j] = '\0'; 
    printf("BUFFER =%ld\n", sizeof(buffer)); 
} 

int main() 
{ 
    int sid = 0, bid = 0, fp; 
    char *send_data = (char *)malloc(1024*sizeof(char)); 
    char temp[1024]; 
    char *receive_data = (char *)malloc(1024*sizeof(char)); 
    int fd, count, cnt=0; 
    struct sockaddr_in server_socket, client_socket; 
    int size = sizeof(client_socket); 

    if((sid = socket(AF_INET, SOCK_DGRAM, IPPROTO_UDP)) == -1) 
    { 
     printf("Connection error..\n"); 
     exit(1); 
    } 

    set_socket(&server_socket, AF_INET, 6000, INADDR_ANY); 

    if((bid = bind(sid, (struct sockaddr *)&server_socket, sizeof(struct sockaddr))) == -1) 
    { 
     printf("Binding error..\n"); 
     exit(1); 
    } 

    printf("I am waiting for client..\n"); 

    recvfrom(sid, receive_data, 1024, 0,(struct sockaddr *)&client_socket, &size); 
    printf("received data is : %s\n", receive_data); 
    fd = open(receive_data, O_RDONLY); 
    printf("size = %ld\n", sizeof(send_data)); 

    while((count=read(fd, temp, 500)) != 0) 
    { 
     printf("I am inside the loop : %d\n", cnt++); 
     serialize(send_data, count, temp); 
     printf("Serialized : %s\n", send_data); 
     sendto(sid, send_data, 1024, 0, (struct sockaddr *)&client_socket, size); 
    } 
    printf("I am outside the loop : %d\n", count); 
    strcpy(temp, "ENDOFFILE"); 
    serialize(send_data, sizeof(temp), temp); 
    sendto(sid, send_data, 1024, 0, (struct sockaddr *)&client_socket, size); 

    fcloseall(); 
    close(sid); 
    close(fd); 
    return 0; 
} 

いるclient.c

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 
#include <sys/types.h> 
#include <sys/socket.h> 
#include <netinet/in.h> 
#include <netdb.h> 
#include <fcntl.h> 

void set_socket(struct sockaddr_in *socket, int type, int host_short) 
{ 
    socket -> sin_family = type; 
    socket -> sin_port = htons(host_short); 
} 

void deserialize(char *buffer, int *size, char *data) 
{ 
    int i=0, j=0; 
    char temp1[20]; 

    while(buffer[i] != ' ') 
    { 
     temp1[j++] = buffer[i++]; 
    } 
    temp1[j] = '\0'; 
    printf("\nINT : %s\n", temp1); 
    *size = atoi(temp1); 

    i++; 
    j=0; 

    while(buffer[i] != '\0') 
    { 
     data[j++] = buffer[i++]; 
    } 
    data[j++] = '\0'; 
} 

int main() 
{ 
    int sid = 0, bid = 0, con = 0; 
    char *send_data = (char *)malloc(1024*sizeof(char)); 
    char *receive_data = (char *)malloc(1024*sizeof(char)); 
    char *temp = (char *)malloc(1024*sizeof(char)); 
    struct hostent *host; 
    struct sockaddr_in server_socket; 
    int size = sizeof(server_socket); 

    if((sid = socket(AF_INET, SOCK_DGRAM, IPPROTO_UDP)) == -1) 
    { 
     printf("Connection error at client side..\n"); 
     exit(1); 
    } 

    set_socket(&server_socket, AF_INET, 6000); 

    if (inet_aton("127.0.0.1", &server_socket.sin_addr)==0) 
    { 
     fprintf(stderr, "inet_aton() failed\n"); 
     exit(1); 
    } 

    printf("Enter the name of the file you want to see : "); 
    scanf("%s", send_data); 
    int fd = open("sanket.mp3", O_CREAT | O_RDWR, S_IRUSR | S_IWUSR | S_IXUSR); 
    sendto(sid, send_data, 1024, 0, (struct sockaddr *)&server_socket, size); 
    printf("================= Contents of the File =====================\n"); 
    while(1) 
    { 
     int size; 
     recvfrom(sid, temp, 1024, 0, (struct sockaddr *)&server_socket, &size); 
     printf("Deserialize it : %s\n",temp); 
     deserialize(temp, &size, receive_data); 
     if(!strcmp(receive_data, "ENDOFFILE")) 
     { 
      printf("============================================================\n"); 
      break; 
     } 
     else 
      write(fd, receive_data, size); 
    } 
    fcloseall(); 
    close(sid); 

    return 0; 
} 

私が送信され、受信したファイルのサイズをチェックすると、このように、大きさは同じですが、内容が異なっています受信したオーディオファイルを再生できません。

+0

エンディアンと関連する可能性があります。エンコーディングがマシン間で異なる可能性はありますか? –

+0

TCPの代わりにUDP経由でファイルを送信しているようです。それは意図的なのでしょうか? – grieve

+0

UDPの代わりにTCPを使用することを選択した場合は、別の投稿からのこの回答が役立ちます。http://stackoverflow.com/a/666788/34329 – grieve

答えて

4

バイナリデータファイルから読み取っているようですが、ヌル終端文字列データの場合には、データを扱っているserialize関数内です。例えば、serialize内部このループ:

for(i=0; data[i] != '\0'; i++) 
{ 
    buffer[j++] = data[i]; 
} 

は、それが遭遇する最初のゼロ値で終了します。あなたのオーディオファイルの真のバイナリデータであれば、バッファの終わりを示すのではなく、実際のオーディオデータである0の値が得られるはずです。 NULL値で終了するのではなく、whileループでreadへの呼び出しで読み取られたserializeに渡すバッファのサイズで終了する必要があります。そうすれば、readコールから読み取られたすべてのデータが送信バッファにパックされていることを確認できます。 がポインタである場合には、sizeof(buffer)を印刷すると、ポインタが指す実際のバッファのサイズではなく、ポインタ型のサイズだけが出力されます。ここでも、その値を明示的にserialize関数に渡す必要があります。

第3に、ヌル値でバッファを終了しています...もう一度、これは生のバイナリデータで、ヌルで終了する文字列ではないという最初の点に基づいては良い考えではありません。データの一部ではない値のセットとなるバッファ内の送信終了を示す文字列を指定するか、あなたがパケットデータに埋め込んだ "数"に

最後に、データをシリアライズしていません...シリアライズの概念は、通常、プラットフォームに依存しない方法でデータを転送することを意味します。受信側が同じエンディアンを持っていると仮定すると、読み込まれたバイトを単純にパックしてネットワークに送信しています。かなりシンプルなシリアライゼーション手法は、すべてのデータ値からASCII文字列を作成するようなことを行います。かなりのデータを膨らませることになります。 JSON、SOAPなどのシリアライズされたデータには、他のクロスプラットフォーム標準があります。

+0

ありがとう..それは魅力のように働いた.. – curiousMind

関連する問題