c言語のaccept関数が待機を行いません

質問

c言語のaccept関数が待機を行いません
現在C言語によるプログラミングを勉強中で、
「Windows　ゲームプログラミング」という書籍を教材に
ネットワーク通信のプログラミングを行っております。
ネットワーク通信のプログラミングは初めてで
現在一つの壁にぶつかっております。
どうか護教授をお願い致します。

下記のサーバー側のコードですが、
(1)ソケットを作成して、bind関数でアドレスと関連付ける
(2)listen関数で接続を準備する
(3)accept関数でクライアントが接続するまで待機する
(4)while文の中でクライアントからのメッセージを受け取る

という目的で作成いたしましたが、クライアントからの接続はしていないはずなんですが、
どうもaccept関数で待機を行わず、while文に進んでしまい、無限ループに突入してしまいます。
accept関数の戻り値を確認すると「INVALID_SOCKET」でした。
現在はこのサーバーに接続するシステムは作成していないので接続はないと思います。
ポート番号などを変更してもダメでした。
お恥ずかしい質問かもしれませんが、どうかよろしくお願い致します。

以下コード

WSADATA wsaData;
sturuct sockaddr_in hostAddr,clAddr;
SOCKET s,client;
char buffer[1024];
int length;

if(WSAStartup(MAKEWORD(2,2),&wsaData)!=0)
return;
(1)
s=socket(AF_INET,SOCK_STREAM,IPPROTO_TCP);

hostAddr.sin_family=AF_INET;
hostAddr.sin_port=ポート番号;
hostAddr.sin_addr.S_un.S_addr=inet_addr("127.0.0.1");

bind(s,&hostAddr,sizeof(struct sockaddr_in));
(2)
listen(s,1);

printf("クライアントからの接続を待機...\n");
(3)
client=accept(s,&clAddr,&length);

printf("クライアントからの接続を確認\n受信を待機...\n");
(4)
while(1){
length=recv(client,buffer,1024,0);
buffer[length]=0;
printf("文字列を受信：%s\n",buffer);
if(strcmp(buffer,"exit")==0)
break;
}
shutdown(client,SD_BOTH);
closesocket(client);
closesocket(s);
WSACleanup();
return 0;

回答No.3

papagal · ベストアンサー

> 仰っていただいた通りにエラーを確認してみると、「WSAEFAULT」でした。
accept()の第3引数を初期化していなかったということですね。
第3引数は入出力両用の引き数なので、lengthをsizeof(sturuct sockaddr_in)で
初期化してから呼び出さなければならなかったということですね。

> ちなみに「NULL」を指定しなくても、while文をコメントアウトすると、
> acceptは待機しました。これはrecv()が呼び出されていないからなのでしょうか？
accept()より後ろにあるwhile文の有無がaccept()の実行結果に影響するとは
考えにくいのではないでしょうか？
まぁもっと大きなループで囲まれているならば別ですが

while文をコメントアウトして実行した時に、スタックの状態に因り
偶々lengthがいい値になったからではないかと考えます。

おそらくlengthは自動変数ではないかと思いますが、
自動変数は初期化しないと、その値は不定です。

回答No.2

papagal

すみません勘違いしていました。
Windows上でソケットを使ったことがないので...
> accept関数の戻り値を確認すると「INVALID_SOCKET」でした。
これはaccept関数が~0を返したという意味ですね。

で、socket()は正常終了していると...
当然bind()、listen()も正常に終了しているのですよね？

では、accept関数が異常終了した時のエラーコードは見てみましたか？
これを見ればどんなエラーが発生しているかわかると思うのですが

お礼

ありがとうございました。
bind(),listen()も正常に終了していました。
仰っていただいた通りにエラーを確認してみると、「WSAEFAULT」でした。
調べてみると、lpSockAddrLen(&length)が小さすぎるとのことで「NULL」を指定してみると、
無事待機後、クライアントからのデータを受信しました。
int型のポインターを指定したのですが....
ちなみに「NULL」を指定しなくても、while文をコメントアウトすると、
acceptは待機しました。これはrecv()が呼び出されていないからなのでしょうか？
recv()を呼び出さなければ通信できないと思うのですが....
この辺りを詳しくご存知でしょうか？

回答No.1

papagal

INVALID_SOCKETといわれているのですから、
まずsocket関数が正常終了しているか確認してはいかがでしょうか？

お礼

早速のご回答ありがとうございました。
if((s=socket(AF_INET,SOCK_STREAM,IPPROTO_TCP))==INVALID_SOCKET){
printf("ソケットの作成に失敗しました");
return;
}
でソケットの作成が正常に行われているか確認しましたが、特に問題はなさそうでした。
確認方法が間違っているのでしょうか？