bind: работа с сокетом на не-сокете - PullRequest
Новая
       4

bind: работа с сокетом на не-сокете

5 голосов
/ 09 апреля 2011

Я пишу сервер и клиент и продолжаю получать 'bind: Socket operation on non-socket'.

Я исследовал все это, есть другой код, который выполняется в другом приложении, и у меня естьисчерпал 8 часов, пытаясь найти эту ошибку.

Код: 

void TCPSocket::buildTCPSocket(int port)
{
    initializeSocket1();
    getSocket();
    bindSocket();
    listenToSocket();
    acceptSocket();
         // now you can send() and recv() with the
        // connected client via socket connectedTCPSocket
}

void TCPSocket::getSocket()
{
        // Get an internet domain socket AF_INET
    if(socket1 = socket(AF_INET, SOCK_STREAM,0) == -1)
    {
        perror("socket");
        exit(1);
    }    
}


void TCPSocket::bindSocket()
{
  // Bind to a port on the host
    int myAddressSize = sizeof(myAddress);
    int bindReturnValue = bind(socket1, (struct sockaddr *) &myAddress, AddressSize);
    if (bindReturnValue == -1)
    {
        perror("bind");  // <== Error message generated here
        exit(1);
    }
    printf("Socket for TCP bound to port %d\n", port);    
}

Кроме того, до этого я запоминал блок памяти с этой функцией. 

void TCPSocket::initializeSocket1()
{
    // Fill tcpSocket struct with 0's

    memset(&myAddress, '\0', sizeof(myAddress));
    myAddress.sin_family = AF_INET;
    myAddress.sin_addr.s_addr = INADDR_ANY;
   // Conver PORT to big-endian if necessary
    myAddress.sin_port = htons(this->port);
}

Переменные объявлены в заголовочном файле класса. 

public:
    struct sockaddr_in myAddress, clientAddress;

    void buildTCPSocket(int newPort);

private:
    int port;
    int socket1, socket2;

    socklen_t clientAddressLength;

- Редактировать код теперь должно быть немного понятнее.socket1 инициализируется в getSocket ().

Я видел, где куча парней пропустили парены в if, но я думаю, что устранил эту ошибку, объявив myAddressSize и bindReturnValue.

Anyвход приветствуется.
Спасибо, Тед С

Хорошо, проблема решена.Конечно, проблема не в том, куда ты смотришь, если бы нашел ее.Вот исправленный код.Проблема была в пропущенном наборе символов в вызове socket (). 

void TCPSocket::getSocket()
{
        // Get an internet domain socket AF_INET
    if((socket1 = socket(AF_INET, SOCK_STREAM,0)) == -1)
    {
        perror("socket");
        exit(1);
    }    
}

Еще раз спасибо!

1 Ответ

9 голосов
/ 09 апреля 2011

Я почти гарантирую вам, что вы получаете эту ошибку, потому что вы никогда не инициализировали socket1.

Обычно вы должны сделать что-то вроде этого: 

 int socket1 = socket(AF_INET, SOCK_STREAM, 0);
 bind(socket1, ...);

Я не вижу нигде кода для настройки socket1. Это то, что сообщение об ошибке говорит вам, в конце концов. socket1 не является сокетом, поэтому он выходит из строя.

Изменить: В качестве продолжения, это одна из причин, почему я стараюсь избегать использования синтаксиса 

if ((foo = bar()) == ERROR)
{
   // handle me
}

И вместо этого придерживайтесь: 

void TCPSocket::getSocket()
{
        // Get an internet domain socket AF_INET
    socket1 = socket(AF_INET, SOCK_STREAM, 0);
    if (socket == -1)
    {
        perror("socket");
        exit(1);
    }    
}
...