Форум программистов [Powered by Invision Power Board]

Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > C/C++: Общие вопросы > Access violation при использовании умного указат.

Автор: magesi 31.7.2012, 13:16

В чем проблема на линии obj->ip = "127.0.0.1"? Получаю violation, из-за чего именно в коде, будьте добры подсказать пожалуйста?

Код


typedef struct Header
{
    SOCKET sock;
    SOCKADDR_IN addr;
    WORD WSAVersion;
    WSAData data;
    char *ip;
    u_short port;
    int result;
} Header;

class A
{
private:
    std::unique_ptr<Header> obj;

public:
    A(void)
    {
        obj(new Header);
        obj->ip = "127.0.0.1"; // тут проблема
    }

    ~A(void)
    {
        obj.release();
    }
};

int main(void)
{
    std::unique_ptr<A> handler(new A);

Автор: boostcoder 31.7.2012, 13:37

Цитата(magesi @ 31.7.2012, 13:16

)

obj(new Header);

что означает это выражение? оО
вроде должно быть так:

Код


obj.reset(new Header);

Автор: baldina 31.7.2012, 13:38

ну, он не настолько умный, что бы выделить память для

Код


char *ip;

Код


new Header

распределяет память для Header, но не выделяет место под строку, которая представлена указателем.

Код


obj->ip = "127.0.0.1";

присваивает указателю адрес временной строки. это вам не std::string.

Добавлено через 2 минуты и 41 секунду

Цитата(boostcoder @ 31.7.2012, 13:37

)

что означает это выражение? оО

ну конечно, конструктор то не так вызываться должен...

Автор: boostcoder 31.7.2012, 13:56

Цитата(baldina @ 31.7.2012, 13:38

)

код C++
1:
new Header
распределяет память для Header, но не выделяет место под строку, которая представлена указателем.
код C++
1:
obj->ip = "127.0.0.1";

присваивает указателю адрес временной строки. это вам не std::string.

и что тут невалидного?

Добавлено через 1 минуту и 23 секунды
указателю присваивается другой адрес. под сам же указатель память выделяется, если правильно инициализировать obj

Автор: baldina 31.7.2012, 14:04

да, проблема не в указателе, а в том что память вообще не выделяется.
со строкой проблем нет, она не временная (да и доступа к ней в программе нет), я ступил.
она в немодифицируемую области, и во всех экземплярах одина и тот же адрес.

Автор: magesi 31.7.2012, 14:20

boostcoder, baldina,

спасибо, сейчас исправил все, работает, код:

Код


#include <iostream>
#include <memory>
#include <WinSock2.h>

#pragma comment(lib, "Ws2_32.lib")

typedef struct Header
{
    SOCKET sock;
    SOCKADDR_IN addr;
    WORD WSAVersion;
    WSAData data;
    char *ip;
    u_short port;
    int result;
} Header;

class Connector
{
private:
    std::unique_ptr<Header> obj;

public:
    Connector(void)
    {
        obj.reset(new Header);

        obj->ip = "193.201.231.44";
        obj->port = 2775;
        obj->WSAVersion = 0x202;

        obj->addr.sin_family = AF_INET;
        obj->addr.sin_addr.s_addr = inet_addr(obj->ip);
        obj->addr.sin_port = htons(obj->port);

        this->MakeConnection(&obj);
    }

    ~Connector(void)
    {
        obj.release();
    }

    int MakeConnection(std::unique_ptr<Header> *pointer)
    {
        (*pointer)->result = WSAStartup((*pointer)->WSAVersion, &(*pointer)->data);

        if((*pointer)->result != NO_ERROR)
        {
            std::cout << "WSA failed to start with the error: " << std::endl << (*pointer)->result << std::endl;
            return 0;
        }
        else
        {
            std::cout << "WSA started successfully" << std::endl;

            (*pointer)->sock = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);

            if((*pointer)->sock == SOCKET_ERROR)
            {
                std::cout << "Failed to init new socket, with the error: " << WSAGetLastError() << std::endl;
                return 0;
            }
            else
            {
                (*pointer)->result = connect(obj->sock, (sockaddr*)&(*pointer)->addr, sizeof((*pointer)->addr));

                if((*pointer)->result == SOCKET_ERROR)
                {
                    std::cout << "Failed to connect to the remote host " << (*pointer)->ip << ":" << (*pointer)->port
                        << ", with the error: " << WSAGetLastError() << std::endl;

                    return 0;
                }
                else
                {
                    std::cout << "Connected to the remote host " << (*pointer)->ip << ":" << (*pointer)->port << std::endl;

                    return 1;
                }
            }
        }
    }
};

void init(void)
{
    std::unique_ptr<Connector> obj;

    obj.reset(new Connector);
    obj.release();
}

int main(void)
{
    init();

    return 0;
}

Автор: boostcoder 31.7.2012, 14:25