c++ 链表的构造函数、析构函数和复制构造函数的内存问题

omqzjyyz  于 2023-01-18  发布在  其他
关注(0)|答案(1)|浏览(201)

我的老师告诉我我的内存有问题，我想是析构函数和复制构造函数的问题，但是我找不到问题所在。这是我的部分代码和来自valgrind的报告。

struct Board 
{
    string s;
    Board* next;
    Board* prev;
};
typedef Board* Pboard;

struct Player::Impl
{
    Pboard head;
    Pboard tail;
    int player;
    void destroy(Pboard head);
};

void Player::Impl::destroy(Pboard head)
{
    if (head) {
        destroy(head->next);
        delete head;
    }
}

Player::Player(int player_nr)
{
    if (player_nr != 1 && player_nr != 2) throw player_exception{ (player_exception::err_type::index_out_of_bounds), "Giocatore non definito"};
    pimpl = new Impl;
    pimpl->player = player_nr;
    pimpl->head = nullptr;
    pimpl->tail = nullptr;
}

Player::~Player()
{
    pimpl->destroy(pimpl->head);
    delete pimpl;
}

Player::Player(const Player& source)
{
    pimpl = new Impl;
    pimpl = source.pimpl;
}

Player& Player::operator=(const Player& s)
{
    if (s.pimpl->head == nullptr) {
        pimpl->head = nullptr;
        pimpl->tail = nullptr;
    }
    else {
        pimpl->destroy(pimpl->head);
        pimpl->head = nullptr;
        pimpl->tail = nullptr;
        Pboard pb = s.pimpl->head;
        pimpl->player = s.pimpl->player;
        while (pb) {
            Pboard pboard = new Board;
            pboard->next = nullptr;
            pboard->s = pb->s;
            pboard->prev = pimpl->tail;
            if (pimpl->tail != nullptr)
                pimpl->tail->next = pboard;
            else
                pimpl->head = pboard;
            pimpl->tail = pboard;
            pb = pb->next;
        }
    }
    return *this;
}```

我尝试了各种解决方案，但我真的不知道是否有效，因为他不让我看他的主，我不得不用我自己的。这是他发给我的瓦尔格林的报告。

==235== Memcheck, a memory error detector
==235== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==235== Using Valgrind-3.15.0 and LibVEX; rerun with -h for copyright info
==235== Command: ../consegne/prova_decompressed/890361/benchmark /home/nico/progetti_PEL/scripts/testcase
==235== 
==235== Invalid read of size 8
==235==    at 0x1178E0: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDC2: check_copy_constr(double) (benchmark.cpp:132)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Address 0x4dc8a20 is 0 bytes inside a block of size 24 free'd
==235==    at 0x483D1CF: operator delete(void*, unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x117906: Player::~Player() (player_filt.cpp:39)
==235==    by 0x10CDB3: check_copy_constr(double) (benchmark.cpp:137)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Block was alloc'd at
==235==    at 0x483BE63: operator new(unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x117847: Player::Player(int) (player_filt.cpp:30)
==235==    by 0x10CB39: check_copy_constr(double) (benchmark.cpp:132)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235== 
==235== Invalid read of size 8
==235==    at 0x117778: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDC2: check_copy_constr(double) (benchmark.cpp:132)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Address 0x4dcae10 is 32 bytes inside a block of size 48 free'd
==235==    at 0x483D1CF: operator delete(void*, unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x1177A8: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDB3: check_copy_constr(double) (benchmark.cpp:137)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Block was alloc'd at
==235==    at 0x483BE63: operator new(unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x117FE3: Player::load_board(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (player_filt.cpp:119)
==235==    by 0x10CB69: check_copy_constr(double) (benchmark.cpp:133)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235== 
==235== Invalid read of size 8
==235==    at 0x117778: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDC2: check_copy_constr(double) (benchmark.cpp:132)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Address 0x4dcd270 is 32 bytes inside a block of size 48 free'd
==235==    at 0x483D1CF: operator delete(void*, unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x1177A8: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDB3: check_copy_constr(double) (benchmark.cpp:137)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Block was alloc'd at
==235==    at 0x483BE63: operator new(unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x117FE3: Player::load_board(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (player_filt.cpp:119)
==235==    by 0x10CBA5: check_copy_constr(double) (benchmark.cpp:134)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235== 
==235== Invalid read of size 8
==235==    at 0x117778: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDC2: check_copy_constr(double) (benchmark.cpp:132)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Address 0x4dcf6d0 is 32 bytes inside a block of size 48 free'd
==235==    at 0x483D1CF: operator delete(void*, unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x1177A8: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDB3: check_copy_constr(double) (benchmark.cpp:137)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Block was alloc'd at
==235==    at 0x483BE63: operator new(unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x117FE3: Player::load_board(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (player_filt.cpp:119)
==235==    by 0x10CBE1: check_copy_constr(double) (benchmark.cpp:135)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235== 
==235== Invalid read of size 8
==235==    at 0x499D5C4: std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::~basic_string() (in /usr/lib/x86_64-linux-gnu/libstdc++.so.6.0.28)
==235==    by 0x11B7DB: Board::~Board() (player_filt.cpp:3)
==235==    by 0x11779B: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDC2: check_copy_constr(double) (benchmark.cpp:132)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Address 0x4dcf6b0 is 0 bytes inside a block of size 48 free'd
==235==    at 0x483D1CF: operator delete(void*, unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x1177A8: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDB3: check_copy_constr(double) (benchmark.cpp:137)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Block was alloc'd at
==235==    at 0x483BE63: operator new(unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x117FE3: Player::load_board(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (player_filt.cpp:119)
==235==    by 0x10CBE1: check_copy_constr(double) (benchmark.cpp:135)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235== 
==235== Invalid free() / delete / delete[] / realloc()
==235==    at 0x483CFBF: operator delete(void*) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x11B7DB: Board::~Board() (player_filt.cpp:3)
==235==    by 0x11779B: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDC2: check_copy_constr(double) (benchmark.cpp:132)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Address 0x4dcf720 is 0 bytes inside a block of size 51 free'd
==235==    at 0x483CFBF: operator delete(void*) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x11B7DB: Board::~Board() (player_filt.cpp:3)
==235==    by 0x11779B: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDB3: check_copy_constr(double) (benchmark.cpp:137)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Block was alloc'd at
==235==    at 0x483BE63: operator new(unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x499D29E: std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_assign(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (in /usr/lib/x86_64-linux-gnu/libstdc++.so.6.0.28)
==235==    by 0x499D5FD: std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::operator=(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (in /usr/lib/x86_64-linux-gnu/libstdc++.so.6.0.28)
==235==    by 0x11801D: Player::load_board(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (player_filt.cpp:121)
==235==    by 0x10CBE1: check_copy_constr(double) (benchmark.cpp:135)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235== 
==235== Invalid free() / delete / delete[] / realloc()
==235==    at 0x483D1CF: operator delete(void*, unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x1177A8: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDC2: check_copy_constr(double) (benchmark.cpp:132)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Address 0x4dcf6b0 is 0 bytes inside a block of size 48 free'd
==235==    at 0x483D1CF: operator delete(void*, unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x1177A8: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDB3: check_copy_constr(double) (benchmark.cpp:137)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Block was alloc'd at
==235==    at 0x483BE63: operator new(unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x117FE3: Player::load_board(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (player_filt.cpp:119)
==235==    by 0x10CBE1: check_copy_constr(double) (benchmark.cpp:135)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235== 
==235== Invalid free() / delete / delete[] / realloc()
==235==    at 0x483D1CF: operator delete(void*, unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x1177A8: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDC2: check_copy_constr(double) (benchmark.cpp:132)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Address 0x4dcd250 is 0 bytes inside a block of size 48 free'd
==235==    at 0x483D1CF: operator delete(void*, unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x1177A8: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x11778A: Player::Impl::destroy(Board*) (player_filt.cpp:22)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDB3: check_copy_constr(double) (benchmark.cpp:137)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Block was alloc'd at
==235==    at 0x483BE63: operator new(unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x117FE3: Player::load_board(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (player_filt.cpp:119)
==235==    by 0x10CBA5: check_copy_constr(double) (benchmark.cpp:134)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235== 
==235== Invalid read of size 8
==235==    at 0x499D5C4: std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::~basic_string() (in /usr/lib/x86_64-linux-gnu/libstdc++.so.6.0.28)
==235==    by 0x11B7DB: Board::~Board() (player_filt.cpp:3)
==235==    by 0x11779B: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDC2: check_copy_constr(double) (benchmark.cpp:132)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Address 0x4dcadf0 is 0 bytes inside a block of size 48 free'd
==235==    at 0x483D1CF: operator delete(void*, unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x1177A8: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDB3: check_copy_constr(double) (benchmark.cpp:137)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Block was alloc'd at
==235==    at 0x483BE63: operator new(unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x117FE3: Player::load_board(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (player_filt.cpp:119)
==235==    by 0x10CB69: check_copy_constr(double) (benchmark.cpp:133)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235== 
==235== Invalid free() / delete / delete[] / realloc()
==235==    at 0x483CFBF: operator delete(void*) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x11B7DB: Board::~Board() (player_filt.cpp:3)
==235==    by 0x11779B: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDC2: check_copy_constr(double) (benchmark.cpp:132)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Address 0x4dcae60 is 0 bytes inside a block of size 54 free'd
==235==    at 0x483CFBF: operator delete(void*) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x11B7DB: Board::~Board() (player_filt.cpp:3)
==235==    by 0x11779B: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDB3: check_copy_constr(double) (benchmark.cpp:137)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Block was alloc'd at
==235==    at 0x483BE63: operator new(unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x499D29E: std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_assign(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (in /usr/lib/x86_64-linux-gnu/libstdc++.so.6.0.28)
==235==    by 0x499D5FD: std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::operator=(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (in /usr/lib/x86_64-linux-gnu/libstdc++.so.6.0.28)
==235==    by 0x11801D: Player::load_board(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (player_filt.cpp:121)
==235==    by 0x10CB69: check_copy_constr(double) (benchmark.cpp:133)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235== 
==235== Invalid free() / delete / delete[] / realloc()
==235==    at 0x483D1CF: operator delete(void*, unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x1177A8: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDC2: check_copy_constr(double) (benchmark.cpp:132)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Address 0x4dcadf0 is 0 bytes inside a block of size 48 free'd
==235==    at 0x483D1CF: operator delete(void*, unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x1177A8: Player::Impl::destroy(Board*) (player_filt.cpp:23)
==235==    by 0x1178ED: Player::~Player() (player_filt.cpp:38)
==235==    by 0x10CDB3: check_copy_constr(double) (benchmark.cpp:137)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Block was alloc'd at
==235==    at 0x483BE63: operator new(unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x117FE3: Player::load_board(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (player_filt.cpp:119)
==235==    by 0x10CB69: check_copy_constr(double) (benchmark.cpp:133)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235== 
==235== Invalid free() / delete / delete[] / realloc()
==235==    at 0x483D1CF: operator delete(void*, unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x117906: Player::~Player() (player_filt.cpp:39)
==235==    by 0x10CDC2: check_copy_constr(double) (benchmark.cpp:132)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Address 0x4dc8a20 is 0 bytes inside a block of size 24 free'd
==235==    at 0x483D1CF: operator delete(void*, unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x117906: Player::~Player() (player_filt.cpp:39)
==235==    by 0x10CDB3: check_copy_constr(double) (benchmark.cpp:137)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235==  Block was alloc'd at
==235==    at 0x483BE63: operator new(unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x117847: Player::Player(int) (player_filt.cpp:30)
==235==    by 0x10CB39: check_copy_constr(double) (benchmark.cpp:132)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235== 
==235== Invalid read of size 8
==235==    at 0x499FC08: std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::basic_string(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (in /usr/lib/x86_64-linux-gnu/libstdc++.so.6.0.28)
==235==    by 0x117C0D: Player::operator()(int, int, int) const (player_filt.cpp:91)
==235==    by 0x110A10: check_access2(double) (benchmark.cpp:504)
==235==    by 0x116916: main (benchmark.cpp:1004)
==235==  Address 0x8 is not stack'd, malloc'd or (recently) free'd
==235== 
==235== 
==235== Process terminating with default action of signal 11 (SIGSEGV)
==235==  Access not within mapped region at address 0x8
==235==    at 0x499FC08: std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::basic_string(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (in /usr/lib/x86_64-linux-gnu/libstdc++.so.6.0.28)
==235==    by 0x117C0D: Player::operator()(int, int, int) const (player_filt.cpp:91)
==235==    by 0x110A10: check_access2(double) (benchmark.cpp:504)
==235==    by 0x116916: main (benchmark.cpp:1004)
==235==  If you believe this happened as a result of a stack
==235==  overflow in your program's main thread (unlikely but
==235==  possible), you can try to increase the size of the
==235==  main thread stack using the --main-stacksize= flag.
==235==  The main thread stack size used in this run was 8388608.
==235== 
==235== HEAP SUMMARY:
==235==     in use at exit: 188 bytes in 5 blocks
==235==   total heap usage: 2,470 allocs, 2,472 frees, 6,973,158 bytes allocated
==235== 
==235== 24 bytes in 1 blocks are definitely lost in loss record 2 of 5
==235==    at 0x483BE63: operator new(unsigned long) (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==235==    by 0x117927: Player::Player(Player const&) (player_filt.cpp:44)
==235==    by 0x10CC06: check_copy_constr(double) (benchmark.cpp:137)
==235==    by 0x1168B9: main (benchmark.cpp:1001)
==235== 
==235== LEAK SUMMARY:
==235==    definitely lost: 24 bytes in 1 blocks
==235==    indirectly lost: 0 bytes in 0 blocks
==235==      possibly lost: 0 bytes in 0 blocks
==235==    still reachable: 164 bytes in 4 blocks
==235==         suppressed: 0 bytes in 0 blocks
==235== Reachable blocks (those to which a pointer was found) are not shown.
==235== To see them, rerun with: --leak-check=full --show-leak-kinds=all
==235== 
==235== For lists of detected and suppressed errors, rerun with: -s
==235== ERROR SUMMARY: 16 errors from 14 contexts (suppressed: 0 from 0)
Segmentation fault
STDERR.txt
Visualizzazione di STDERR.txt.
c++

1条答案

按热度按时间
rsl1atfo

rsl1atfo1#

玩家的复制构造函数是错误的。你创建了一个新的指针，把它赋给你的变量，然后立即告诉那个变量指向其他地方。这就是内存泄漏......当任何一个玩家被删除时，就会导致UB。

Player::Player(const Player& source)
{
    pimpl = new Impl; // dynamical allocation, assign to variable 
    pimpl = source.pimpl; // now changing where the pointer points to. Memory leak.
    // Both players also now point to the same memory location, so the destructor really causes issues. 
}
赞(0）分享  回复(0）举报  2023-01-18
我来回答

相关问题

    查看更多

    热门标签

    更多
    JavaquerypythonNode开发语言requestUtil数据库Table后端算法LoggerMessageElementParser

    最新问答

    更多
    • 蜀ICP备2022004421号-2 NULL123 https://www.null123.com © All rights reserved
    • 本站内容来源互联网,如果侵犯您的权益请联系我们删除, 联系方式：448109455@qq.com