Ошибка сегментации в std :: map :: insert (...) - PullRequest
6 голосов
/ 02 сентября 2010

Я использовал поиск, но не нашел ответа, удовлетворяющего меня ... так что ... это кусок кода:

 //VoteContainer.h    
    typedef uint32_t order_id_t;
    typedef int driver_id_t;

    class Vote {

        public:
            enum DriverVoteResponse {YES, NO, TIMEOUT};

            struct DriverResponse {
                driver_id_t driver_id;
                time_t time;
                DriverVoteResponse response;
            };

            Vote() : m_order_id(0), m_time_until(0) {};
            Vote(order_id_t inOrderId, std::vector<driver_id_t> inPermittedDrivers, int inSeconds);
            Vote(const Vote & other) : m_order_id(other.m_order_id), m_time_until(other.m_order_id) {
                m_drivers_responses = other.m_drivers_responses;
                m_permitted_drivers = other.m_permitted_drivers;
            };

            virtual ~Vote() {};

            virtual void addDriverVote(driver_id_t inDriverId, DriverVoteResponse inDriverResponse);
            virtual void getAppropriateDriverId(driver_id_t * inDriverId); //with min response time

        private:

            order_id_t m_order_id;
            time_t m_time_until;
            std::vector<DriverResponse> m_drivers_responses;
            std::vector<driver_id_t> m_permitted_drivers;
        };

class VoteContainer {
public:

    VoteContainer() {};
    virtual ~VoteContainer() {};

    void registerVote(order_id_t inOrderId, std::vector<driver_id_t> inPermittedDrivers, int inSeconds);

private:
    std::map<order_id_t, Vote> m_votes;
};

и как я его использую:

//VoteContainer.cpp
void VoteContainer::registerVote(order_id_t inOrderId, std::vector<driver_id_t> inPermittedDrivers, int inSeconds) {
        m_votes.insert(std::make_pair(inOrderId,  Vote(inOrderId, inPermittedDrivers, inSeconds)));
    return;
};

У меня есть segfault независимо от того, что я делаю:

m_votes.insert(std::make_pair(inOrderId,  Vote(inOrderId, inPermittedDrivers, inSeconds)));

Сначала я попытался использовать std :: map :: find (...), но у меня тот же результат.обратная трассировка:

#0 0x41096a std::less<unsigned int>::operator() (this=0x407a59, __x=@0x7fffffff0b50, __y=@0x758948f87d894905) (/usr/include/c++/4.4/bits/stl_function.h:230)
#1 0x4105fb std::_Rb_tree<unsigned int, std::pair<unsigned int const, Vote>, std::_Select1st<std::pair<unsigned int const, Vote> >, std::less<unsigned int>, std::allocator<std::pair<unsigned int const, Vote> > >::_M_insert_unique(this=0x407a59, __v=...) (/usr/include/c++/4.4/bits/stl_tree.h:1170)
#2 0x40fb25 std::map<unsigned int, Vote, std::less<unsigned int>, std::allocator<std::pair<unsigned int const, Vote> > >::insert(this=0x407a59, __x=...) (/usr/include/c++/4.4/bits/stl_map.h:500)
#3 0x40f06f VoteContainer::registerVote(this=0x407a51, inOrderId=1, inPermittedDrivers=..., inSeconds=32) (/home/user/workspace/src/merit_your_name/VoteContainer.cpp:81)

Полагаю, причиной segfault является аргумент __y=@0x758948f87d894905.Я понятия не имею, почему это так!в этот момент карта m_votes пуста.пожалуйста, предложите мне ...

Как говорит Матье М., наиболее вероятной причиной является неинициализированное значение __y=@0x758948f87d894905, но __y имеет тип order_id_t, но не Vote

Я пытался переписать код:

std::map<int, int> m_votes;

, и это не решило мою проблему, поэтому проблема не в моих типах ...

вот код вызоваregisterVote() метод.

void OrderProcessor::processOrder(Order inOrder) {
    //test!!!
    driver_id_t driver_ids[] = {1,2};
    std::vector<driver_id_t> drivers(driver_ids, driver_ids + sizeof(driver_ids) / sizeof(driver_id_t) );

    m_vote_container->registerVote(inOrder.getId(), drivers, 32);

    for(size_t i = 0; i < drivers.size(); i++) {
        std::cout << "sending vote to " << drivers[i] << " driver. " << std::endl;
        std::cout << "send returns " << Arch::send_to_socket_nonblock((*m_drivers_connections)[drivers[i]], "<vote>1</vote>") << std::endl;
    }

    sleep(32);

    Vote vote = m_vote_container->getVote(inOrder.getId());
    vote.getAppropriateDriverId(driver_id);
    m_vote_container->deleteVote(inOrder.getId());
};

Вчера я обнаружил, что проблема не в моем коде!я заменил std :: map на другие структуры stl, но результат был таким же!я удалил stl из этого кода, и segfault был в конструкторе голосования, я удалил этот класс, и segfault был в других структурах stl моего кода!что это такое?Помогите мне, пожалуйста.

Я выяснил причину моей проблемы, это не этот код.проблема была в моем предыдущем коде.Спасибо всем за участие в этой дискуссии.

Ответы [ 3 ]

5 голосов
/ 02 сентября 2010

Из того, что я вижу, я бы рискнул пропустить действительно важный код.

Как уже отмечалось: this=0x407a59, __x=@0x7fffffff0b50, __y=@0x758948f87d894905 довольно странно, адреса расположены слишком далеко друг от друга, поэтому мы можем предположить, что один изони (по крайней мере) просто неинициализированы.И для моего же здравомыслия я предполагаю, что ваша реализация std::map не глючит.

Мне было бы интересно искать неинициализированную карту и, следовательно, и неинициализированный объект VoteContainer.Будете ли вы иметь VoteContainer*, которые вы забыли выделить, прежде чем вызывать registerVote на нем?

1 голос
/ 02 сентября 2010

Если вы работаете под Linux, я могу порекомендовать инструмент valgrind, он должен помочь вам найти, где проблема

0 голосов
/ 02 сентября 2010

Две догадки:

  1. Можете ли вы показать код, вызывающий метод registerVote()? (Я подозреваю, что это может дать некоторую подсказку.)

  2. Можешь поставить

    m_votes.clear ();

в конструкторе VoteContainer?

...