8

UNIX ネットワーク プログラミングは初めてで、Google のサーバーに接続するプログラムを作成しようとしました。ただし、connect() 関数の使用中にエラーが発生しました。(OS: OS X)

接続エラー: 非ソケットでのソケット操作

4時間作業しましたが、問題を見つけることができませんでした。これが私のコードです:

#define SERVPORT 80

int main (int argc, char **argv)
{
  int i, sockfd;
  struct hostent *host;
  struct sockaddr_in serv_addr;

  if ( (host = gethostbyname(argv[1])) == NULL) {
    printf("gethostbyname error\n");
    exit(1);
  }

  for (i = 0; host->h_addr_list[i]; i++) {
    if ( (sockfd = socket(AF_INET, SOCK_STREAM, 0) == -1)) {
    printf("socket error\n");
    exit(1);
    }

    bzero(&serv_addr, sizeof(serv_addr));
    serv_addr.sin_family = AF_INET;
    serv_addr.sin_port = htons(SERVPORT);
    serv_addr.sin_addr = *( (struct in_addr *)host->h_addr_list[i]);
    const char *ip = inet_ntoa(serv_addr.sin_addr);
    printf("connect to %s\n", ip);

    if (connect(sockfd, (struct sockaddr *) &serv_addr,
            sizeof(struct sockaddr)) == -1) {
      printf("connect error:%s\n", strerror(errno));
      exit(1);
    }

 }
  return 0;
}
4

1 に答える 1

27

問題がわかりました。それはこの行です:

if ( (sockfd = socket(AF_INET, SOCK_STREAM, 0) == -1))

== 演算子は = 演算子よりも優先されます。私が何を意味するかを理解するために、その式で括弧が構成されている方法をもう少し注意深く見てください。sockfd は、ブール式 (socket(...) == -1) が割り当てられた結果、「0」に初期化されます。

ソケットの初期化を次のように変更します。

  for (i = 0; host->h_addr_list[i]; i++) 
  {

    sockfd = socket(AF_INET, SOCK_STREAM, 0);
    if (sockfd == -1)
    {
        printf("socket error\n");
        exit(1);
    }

または、同じ行のアプローチで「割り当てと比較」を好む場合は、おそらく次のように言うことができます。

if ( (sockfd = socket(AF_INET, SOCK_STREAM, 0)) == -1)

微妙な違いに注目してください。

于 2013-02-06T04:00:13.493 に答える