1. дома
  2. Вопрос и ответ
  3. C++ unordered_map SIGSEGV on find() или insert()

C++ unordered_map SIGSEGV on find() или insert()

2015-04-29 2 views
1

В настоящее время я пытаюсь реализовать какой-то базовый уровень доступа к файлу для системы баз данных, которую нам нужно построить для университета. Для этого мне предлагается кэшировать страницы в памяти. Чтобы быстро решить, какие страницы загружены, я сохраняю их все в unordered_map.C++ unordered_map SIGSEGV on find() или insert()

Проблема заключается в том, что после некоторых вызовов этой функции SIGSEGV появляется внутри _M_find_before_node неупорядоченного_мапа. Я попытался ее отладки с помощью GDB, но я понятия не имею, где искать его, как происходит ошибка при вызове к unordered_map:

#0 0x0000000000402ab0 in std::_Hashtable<unsigned long, std::pair<unsigned long const, BufferFrame>, std::allocator<std::pair<unsigned long const, BufferFrame> >, std::__detail::_Select1st, std::equal_to<unsigned long>, std::hash<unsigned long>, std::__detail::_Mod_range_hashing, std::__detail::_Default_ranged_hash, std::__detail::_Prime_rehash_policy, std::__detail::_Hashtable_traits<false, false, true> >::_M_find_before_node (this=0x608038, __n=2, [email protected]: 2, __code=2) at /usr/include/c++/4.9/bits/hashtable.h:1442 
#1 0x0000000000402286 in std::_Hashtable<unsigned long, std::pair<unsigned long const, BufferFrame>, std::allocator<std::pair<unsigned long const, BufferFrame> >, std::__detail::_Select1st, std::equal_to<unsigned long>, std::hash<unsigned long>, std::__detail::_Mod_range_hashing, std::__detail::_Default_ranged_hash, std::__detail::_Prime_rehash_policy, std::__detail::_Hashtable_traits<false, false, true> >::_M_find_node (this=0x608038, __bkt=2, [email protected]: 2, __c=2) at /usr/include/c++/4.9/bits/hashtable.h:625 
#2 0x0000000000401cd6 in std::_Hashtable<unsigned long, std::pair<unsigned long const, BufferFrame>, std::allocator<std::pair<unsigned long const, BufferFrame> >, std::__detail::_Select1st, std::equal_to<unsigned long>, std::hash<unsigned long>, std::__detail::_Mod_range_hashing, std::__detail::_Default_ranged_hash, std::__detail::_Prime_rehash_policy, std::__detail::_Hashtable_traits<false, false, true> >::find (this=0x608038, [email protected]: 2) at /usr/include/c++/4.9/bits/hashtable.h:1304 
#3 0x000000000040198f in std::unordered_map<unsigned long, BufferFrame, std::hash<unsigned long>, std::equal_to<unsigned long>, std::allocator<std::pair<unsigned long const, BufferFrame> > >::find (
this=0x608038, [email protected]: 2) at /usr/include/c++/4.9/bits/unordered_map.h:574 
#4 0x00000000004013dd in BufferManager::fixPage (this=0x608030, pageId=2, exclusive=true) at BufferManager.cpp:36 
#5 0x0000000000404268 in main (argc=4, argv=0x7fffffffe2c8) at buffertest.cpp:87

Я также попытался его отладки с помощью Valgrind, как я думал о том I может получить незаметную повреждение памяти в некоторый момент времени до появления ошибки. Это дало мне следующий результат:

==27031== Memcheck, a memory error detector 
==27031== Copyright (C) 2002-2013, and GNU GPL'd, by Julian Seward et al. 
==27031== Using Valgrind-3.10.0 and LibVEX; rerun with -h for copyright info 
==27031== Command: ./buffer-test 11 3 1 
==27031== 
==27031== Invalid write of size 8 
==27031== at 0x401642: std::__detail::_Hash_node_base::_Hash_node_base() (hashtable_policy.h:234) 
==27031== by 0x4020A0: std::_Hashtable<std::string, std::pair<std::string const, int>, std::allocator<std::pair<std::string const, int> >, std::__detail::_Select1st, std::equal_to<std::string>, std::hash<std::string>, std::__detail::_Mod_range_hashing, std::__detail::_Default_ranged_hash, std::__detail::_Prime_rehash_policy, std::__detail::_Hashtable_traits<true, false, true> >::_Hashtable(unsigned long, std::hash<std::string> const&, std::__detail::_Mod_range_hashing const&, std::__detail::_Default_ranged_hash const&, std::equal_to<std::string> const&, std::__detail::_Select1st const&, std::allocator<std::pair<std::string const, int> > const&) (hashtable.h:799) 
==27031== by 0x401BB5: std::_Hashtable<std::string, std::pair<std::string const, int>, std::allocator<std::pair<std::string const, int> >, std::__detail::_Select1st, std::equal_to<std::string>, std::hash<std::string>, std::__detail::_Mod_range_hashing, std::__detail::_Default_ranged_hash, std::__detail::_Prime_rehash_policy, std::__detail::_Hashtable_traits<true, false, true> >::_Hashtable(unsigned long, std::hash<std::string> const&, std::equal_to<std::string> const&, std::allocator<std::pair<std::string const, int> > const&) (hashtable.h:420) 
==27031== by 0x4018DC: std::unordered_map<std::string, int, std::hash<std::string>, std::equal_to<std::string>, std::allocator<std::pair<std::string const, int> > >::unordered_map(unsigned long, std::hash<std::string> const&, std::equal_to<std::string> const&, std::allocator<std::pair<std::string const, int> > const&) (unordered_map.h:143) 
==27031== by 0x401257: BufferManager::BufferManager(unsigned int) (BufferManager.cpp:21) 
==27031== by 0x4041B6: main (buffertest.cpp:79) 
==27031== Address 0x5c1d0e0 is 16 bytes after a block of size 64 alloc'd 
==27031== at 0x4C29180: operator new(unsigned long) (vg_replace_malloc.c:324) 
==27031== by 0x4041A3: main (buffertest.cpp:79) 
==27031== 
==27031== Invalid write of size 8 
==27031== at 0x4020A5: std::_Hashtable<std::string, std::pair<std::string const, int>, std::allocator<std::pair<std::string const, int> >, std::__detail::_Select1st, std::equal_to<std::string>, std::hash<std::string>, std::__detail::_Mod_range_hashing, std::__detail::_Default_ranged_hash, std::__detail::_Prime_rehash_policy, std::__detail::_Hashtable_traits<true, false, true> >::_Hashtable(unsigned long, std::hash<std::string> const&, std::__detail::_Mod_range_hashing const&, std::__detail::_Default_ranged_hash const&, std::equal_to<std::string> const&, std::__detail::_Select1st const&, std::allocator<std::pair<std::string const, int> > const&) (hashtable.h:799) 
==27031== by 0x401BB5: std::_Hashtable<std::string, std::pair<std::string const, int>, std::allocator<std::pair<std::string const, int> >, std::__detail::_Select1st, std::equal_to<std::string>, std::hash<std::string>, std::__detail::_Mod_range_hashing, std::__detail::_Default_ranged_hash, std::__detail::_Prime_rehash_policy, std::__detail::_Hashtable_traits<true, false, true> >::_Hashtable(unsigned long, std::hash<std::string> const&, std::equal_to<std::string> const&, std::allocator<std::pair<std::string const, int> > const&) (hashtable.h:420) 
==27031== by 0x4018DC: std::unordered_map<std::string, int, std::hash<std::string>, std::equal_to<std::string>, std::allocator<std::pair<std::string const, int> > >::unordered_map(unsigned long, std::hash<std::string> const&, std::equal_to<std::string> const&, std::allocator<std::pair<std::string const, int> > const&) (unordered_map.h:143) 
==27031== by 0x401257: BufferManager::BufferManager(unsigned int) (BufferManager.cpp:21) 
==27031== by 0x4041B6: main (buffertest.cpp:79) 
==27031== Address 0x5c1d0e8 is 24 bytes after a block of size 64 in arena "client" 
==27031== 

valgrind: m_mallocfree.c:304 (get_bszB_as_is): Assertion 'bszB_lo == bszB_hi' failed. 
valgrind: Heap block lo/hi size mismatch: lo = 128, hi = 0. 
This is probably caused by your program erroneously writing past the 
end of a heap block and corrupting heap metadata. If you fix any 
invalid writes reported by Memcheck, this assertion failure will 
probably go away. Please try that before reporting this as a bug. 


host stacktrace: 
==27031== at 0x380A48EF: show_sched_status_wrk (m_libcassert.c:319) 
==27031== by 0x380A49E4: report_and_quit (m_libcassert.c:390) 
==27031== by 0x380A4B66: vgPlain_assert_fail (m_libcassert.c:455) 
==27031== by 0x380B170D: get_bszB_as_is (m_mallocfree.c:302) 
==27031== by 0x380B170D: get_bszB (m_mallocfree.c:312) 
==27031== by 0x380B170D: get_pszB (m_mallocfree.c:386) 
==27031== by 0x380B170D: vgPlain_describe_arena_addr (m_mallocfree.c:1532) 
==27031== by 0x3809DC93: vgPlain_describe_addr (m_addrinfo.c:188) 
==27031== by 0x3809C73B: vgMemCheck_update_Error_extra (mc_errors.c:1133) 
==27031== by 0x380A05BA: vgPlain_maybe_record_error (m_errormgr.c:818) 
==27031== by 0x3809BCB2: vgMemCheck_record_address_error (mc_errors.c:753) 
==27031== by 0x803EDF459: ??? 
==27031== by 0x802D95EEF: ??? 
==27031== by 0x3807295F: vgMemCheck_malloc (mc_malloc_wrappers.c:388) 
==27031== by 0x4020A0: std::_Hashtable<std::string, std::pair<std::string const, int>, std::allocator<std::pair<std::string const, int> >, std::__detail::_Select1st, std::equal_to<std::string>, std::hash<std::string>, std::__detail::_Mod_range_hashing, std::__detail::_Default_ranged_hash, std::__detail::_Prime_rehash_policy, std::__detail::_Hashtable_traits<true, false, true> >::_Hashtable(unsigned long, std::hash<std::string> const&, std::__detail::_Mod_range_hashing const&, std::__detail::_Default_ranged_hash const&, std::equal_to<std::string> const&, std::__detail::_Select1st const&, std::allocator<std::pair<std::string const, int> > const&) (hashtable.h:799) 

sched status: 
    running_tid=1 

Thread 1: status = VgTs_Runnable 
==27031== at 0x401682: std::__detail::_Prime_rehash_policy::_Prime_rehash_policy(float) (hashtable_policy.h:464) 
==27031== by 0x4020C4: std::_Hashtable<std::string, std::pair<std::string const, int>, std::allocator<std::pair<std::string const, int> >, std::__detail::_Select1st, std::equal_to<std::string>, std::hash<std::string>, std::__detail::_Mod_range_hashing, std::__detail::_Default_ranged_hash, std::__detail::_Prime_rehash_policy, std::__detail::_Hashtable_traits<true, false, true> >::_Hashtable(unsigned long, std::hash<std::string> const&, std::__detail::_Mod_range_hashing const&, std::__detail::_Default_ranged_hash const&, std::equal_to<std::string> const&, std::__detail::_Select1st const&, std::allocator<std::pair<std::string const, int> > const&) (hashtable.h:799) 
==27031== by 0x401BB5: std::_Hashtable<std::string, std::pair<std::string const, int>, std::allocator<std::pair<std::string const, int> >, std::__detail::_Select1st, std::equal_to<std::string>, std::hash<std::string>, std::__detail::_Mod_range_hashing, std::__detail::_Default_ranged_hash, std::__detail::_Prime_rehash_policy, std::__detail::_Hashtable_traits<true, false, true> >::_Hashtable(unsigned long, std::hash<std::string> const&, std::equal_to<std::string> const&, std::allocator<std::pair<std::string const, int> > const&) (hashtable.h:420) 
==27031== by 0x4018DC: std::unordered_map<std::string, int, std::hash<std::string>, std::equal_to<std::string>, std::allocator<std::pair<std::string const, int> > >::unordered_map(unsigned long, std::hash<std::string> const&, std::equal_to<std::string> const&, std::allocator<std::pair<std::string const, int> > const&) (unordered_map.h:143) 
==27031== by 0x401257: BufferManager::BufferManager(unsigned int) (BufferManager.cpp:21) 
==27031== by 0x4041B6: main (buffertest.cpp:79) 


Note: see also the FAQ in the source distribution. 
It contains workarounds to several common problems. 
In particular, if Valgrind aborted or crashed after 
identifying problems in your program, there's a good chance 
that fixing those problems will prevent Valgrind aborting or 
crashing, especially if it happened in m_mallocfree.c. 

If that doesn't help, please report this bug to: www.valgrind.org 

In the bug report, send all the above text, the valgrind 
version, and what OS and version you are using. Thanks.

К сожалению, я не мог понять, что эти ошибки хотят сказать мне. Код, в котором они происходят, даже не делает ничего с этим unordered_map, это просто конструктор.

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

BufferManager.hpp (обновлено)

#ifndef BUFFER_MANAGER_HPP 
#include "BufferFrame.hpp" 
#include <stdint.h> 
#include <unordered_map> 

class BufferManager { 

public: 
    BufferManager(const unsigned int pageCount); 
    BufferFrame& fixPage(const uint64_t pageId, const bool exclusive); 
    void unfixPage(BufferFrame& frame, const bool isDirty); 
    ~BufferManager(); 

private: 
    uint64_t pageCount; 
    std::unordered_map<uint64_t, BufferFrame *> pages; 
    std::unordered_map<std::string, int> files; 
}; 


#define BUFFER_MANAGER_HPP 
#endif

BufferManager.cpp (части) (обновлено):

BufferManager::BufferManager(const unsigned int pageCount) { 
    this->pageCount = pageCount; 
    std::cout << "BufferFrame constructor" << std::endl; 
} 

BufferFrame& BufferManager::fixPage(const uint64_t pageId, const bool exclusive) { 
    std::cout << "fixPage called with pageId: " << pageId << " requesting exclusive: " << exclusive<< std::endl; 

    std::cout << "Pages.size(): " << pages.size() << std::endl; 

    //Test if we got the frame in the buffer 
    BufferFrame * frame; 
    std::unordered_map<uint64_t, BufferFrame *>::const_iterator got = pages.find(pageId); 
    if (got == pages.end()) { 
     frame = new BufferFrame(); 
     frame->data = malloc(pageSize); 
     frame->pageId = pageId; 
     frame->isDirty = false; 

     std::pair<uint64_t, BufferFrame *> element(pageId, frame); 
     pages.insert(element); 
     std::cout << "fixPage of page: " << pageId << " (empty page used. Load not implemented!)" << std::endl; 
    } 
    else { 
     frame = got->second; 
     std::cout << "fixPage of page: " << pageId << " (Page loaded from map)" << std::endl; 
    } 
    return *frame; 
}

BufferFrame.hpp:

#ifndef BUFFER_FRAME_HPP 
#include <stdint.h> 

class BufferFrame { 
    friend class BufferManager; 

public: 

    void* getData(); 

private: 

    void * data; 
    uint64_t pageId; 
    bool isDirty; 
}; 



#define BUFFER_FRAME_HPP 
#endif

То, что я пытался до сих пор заменяет BufferFrame в unordered_map на BufferFrame *, который ничего не менял. Даже замена его на int не показала никакого эффекта.

EDIT: обновлены BufferManager.hpp и BufferManager.cpp, чтобы использовать unordered_map.

EDIT2: Прежде всего: Спасибо всем, что пытался помочь мне

Я нашел проблему: Это была ошибка в Makefile, который вызвал один из моих .o файлы не быть регенерируется. Это приводит к разным файлам, использующим другую версию BufferFrame.hpp. Это, очевидно, должно привести к ошибкам памяти bizzarre (например, тем, которые у меня есть).

источник

2015-04-29 Sebastian

ответ

1

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

BufferFrame& BufferManager::fixPage(
    .... 
    BufferFrame frame; 
    .... 
    return frame;

Вы можете изменить дизайн

std::unordered_map<uint64_t, BufferFrame *> pages;

или даже

std::unordered_map<uint64_t, std::shared_ptr<BufferFrame> > pages;

Я хотел бы также предложить создать BufferFrame в качестве собственного класса C++ с помощью конструктора и деструктора, так что он управляет собственной памятью.

источник

2015-04-29 23:08:30 Arun

+0

Это была моя первая идея, как это сделать. Я только удалил указатель, потому что во время отладки у меня кончились идеи. В любом случае вы правы, он не может работать так. Но изменение его не изменило Ошибка. Я изменил BufferFrame на соответствующий класс C++ позже, но это не должно быть проблемой. В любом случае, я дам вам знать, если это что-то изменит. – Sebastian

0

Пожалуйста, посмотрите на эти два аспекта тесно:

  • Где находится множество и «PAGESIZE» на какое значение? Я не вижу ничего.
  • Вам действительно нужно смешивать «новый» и malloc, выделяя «кадр».
  • Вы не указали, как очистить объекты BufferFrame после использования.
  • Если вы делаете новый + бесплатный или malloc + delete, у вас проблемы.

Я бы пошел с ctor и dtor для BufferFrame, как предложил Арун.

источник

2015-04-30 07:16:18 KalyanS

Смежные вопросы

 Смежные вопросы