C言語のWinSockでのファイル転送

Question

WinSockでファイルを転送したいです。
少し調べてみたところ
http://oshiete.goo.ne.jp/qa/652766.html
上記のアンサーをそのままやるとダメで
色々と試行錯誤したのですが分かりませんでした。
上記の質問の解答は何処が間違っているのでしょうか？
詳しく教えていただけると嬉しく思います。

Comments

[KoichiSugiyama]

参考のリンクにあるアンサーをそのままやる、と書かれていますが、実際に書かれたコードを見てみると、勘違い等があるかもしれません。差し支えない限りでコードを開示されてはどうでしょうか。

Answer 1

リンク先を見てみましたが、fread, fwriteの使い方が間違っていますね。あのままだとちゃんと動きません。

それぞれのプロトタイプを確認すると、

size_t fread(void * buffer, size_t size, size_t count, FILE * stream);
size_t fwrite(const void * buffer, size_t size, size_t count, FILE * stream);

となっています。freadはsizeバイトのデータをcount個読み取り、実際に読み取った「個数」を返します。fwriteはsizeバイトのデータをcount個書き出し、実際に書いた「個数」を返します。

さて、リンク先の回答ではどうやって使っているかというと、

送る方では、

while( num_read = fread(buf,sizeof(buf),1, file) )
if( write(socket,buf,num_read) < num_read )
return; //error

sizeof(buf)は4096なので、fread関数で4096バイトのデータを1個読み、正しく読めたらnum_readに1が入り、write関数で読み取ったデータをソケットに1バイト送っています。

受け取る方では、

while( num_read = read(socket,buf,sizeof(buf)) )
if( fwrite(buf,num_read,1,file) < num_read )
return; //error

read関数で4096バイトのデータを受け取り、fwrite関数で4096バイトのデータを1個書き出し、正しくかけたら1が返ってくるけど、4096よりも小さいのでエラーになってしまいます。

ということで、fread/fwriteの引数には、sizeを1にしてcountにバッファサイズ／データサイズを渡すようにしてください。そうすれば意図通りに動くと思います。

Answer 2

これの前にソケットをセットアップしなくてはなりません。

Comments

[lyc0ris_sec]

すみません、前提としてソケットは既に作成したものでお願い致します。

[Zuishin]

ちゃんと接続されているのにうまくいかないのですか？
どうなるのでしょう？