抛出异常会导致分段错误

时间:2010-02-09 19:31:50

标签: c++

Collection CollectionFactory::createFromMap(const std::string& name,
        const DataMap& dm) const
{
    if (!Collection::isNameValid(name))
    {
        const std::string error = "invalid collection name";
        throw std::invalid_argument(error);
    }
    Collection c(name, dm);
    dm.initDataCollection(&c, true);
    return c;
}

每当执行throw语句时,我都会遇到分段错误。这是Valgrind输出的原因。我不知道发生了什么。

==21124== Invalid read of size 1
==21124==    at 0x41D2190: parse_lsda_header(_Unwind_Context*, unsigned char const*, lsda_header_info*) (eh_personality.cc:62)
==21124==    by 0x41D24A9: __gxx_personality_v0 (eh_personality.cc:228)
==21124==    by 0x4200220: _Unwind_RaiseException (unwind.inc:109)
==21124==    by 0x41D2C9C: __cxa_throw (eh_throw.cc:75)
==21124==    by 0x4079BFB: corestore::CollectionFactory::createFromMap(std::string const&, corestore::DataMap const&) const (CollectionFactory.C:43)
==21124==    by 0x8188F86: CollectionFactoryTest::testCreateNewFromMap_InvalidName() (CollectionFactoryTest.C:91)
==21124==    by 0x81895D3: CppUnit::TestCaller<CollectionFactoryTest>::runTest() (TestCaller.h:166)
==21124==    by 0x40D1BB5: CppUnit::TestCaseMethodFunctor::operator()() const (TestCase.cpp:34)
==21124==    by 0x40C18E3: CppUnit::DefaultProtector::protect(CppUnit::Functor const&, CppUnit::ProtectorContext const&) (DefaultProtector.cpp:15)
==21124==    by 0x40CD0FC: CppUnit::ProtectorChain::ProtectFunctor::operator()() const (ProtectorChain.cpp:20)
==21124==    by 0x40CCA65: CppUnit::ProtectorChain::protect(CppUnit::Functor const&, CppUnit::ProtectorContext const&) (ProtectorChain.cpp:77)
==21124==    by 0x40DC6C4: CppUnit::TestResult::protect(CppUnit::Functor const&, CppUnit::Test*, std::string const&) (TestResult.cpp:178)
==21124==  Address 0xc82f is not stack'd, malloc'd or (recently) free'd

我已经进行了几次单元测试的重复轰炸,但是现在这个是与所有其他人一样出现同样错误的错误:

void CollectionFactoryTest::testCreateNewFromMap_InvalidName()
{
    const char* MAP_FILE =
            "smallMapWithThreeSets.xml";
    const char* NAME1 = "name/invalidname";
    const char* NAME2 = "name/invalidname";

    DataMapReader dmr;
    DataMap dm = dmr.getDataMapFromFile(MAP_FILE);

    CollectionFactory cf;
    try
    {
        cf.createFromMap(NAME1, dm);
    }
    catch (std::exception const& e)
    {
        std::cerr << e.what() << std::endl;
    }

    /*CPPUNIT_ASSERT_THROW(cf.createFromMap(NAME1, dm), std::invalid_argument);
    CPPUNIT_ASSERT_THROW(cf.createFromMap(NAME2, dm), std::invalid_argument);*/
}

每个请求,isNameValid:

的内容
bool Collection::isNameValid(const std::string& name)
{
    /* can't be blank */
    if(name.length() == 0)
    {
        return false;
    }
    /* Can't contain '/' */
    if(name.find('/') != std::string::npos)
    {
        return false;
    }
    return true;
}

3 个答案:

答案 0 :(得分:2)

戴夫,你的执行路径中缺少代码,这可以防止解决问题:

  • DataMapReader构造函数;
  • DataMapReader :: getDataMapFromFile();
  • 的实现
  • DataMap构造函数;
  • CollectionFactory构造函数;

我建议尝试尽可能地剥离测试用例,同时分段错误仍然可以重现。

我对此问题有以下猜测:

  • DataMapReader :: getDataMapFromFile()接受对std :: string的引用,然后将其存储在静态内存或DataMapReader实例或DataMap实例中。这会导致未定义的行为,因为引用的std :: string对象将在退出DataMapReader :: getDataMapFromFile()后立即销毁,因此所有剩余的引用将自动变为无效。
  • DataMapReader :: getDataMapFromFile()返回对存储在堆栈中的DataMap实例的引用。一旦DataMapReader :: getDataMapFromFile()返回并且在调用dm的复制构造函数之前,这样的实例将被销毁,这会导致未定义的行为。这种情况不太可能,因为编译器通常会警告返回指针或对存储在堆栈中的对象的引用。
  • 如果DataMapReader :: getMapFromFile()按值返回DataMap,并且DataMap没有明确定义的复制构造函数和/或复制赋值运算符,或者它对那些构造函数和/或运算符有不正确的实现,这些实现不正确地处理指针的复制和/或者对DataMap拥有的成员的引用(或者由value存储的其他成员递归地具有相同的问题)。在这种情况下,引用的成员可以在从DataMapReader :: getMapFromFile()返回之前在存储在堆栈上的实例调用的DataMap析构函数中销毁,这会导致未定义的行为。

虽然这与问题中提到的分段错误无关,但同样的潜在问题可能适用于下面的代码,在CollectionFactory :: createFromMap()中抛出异常:

  • 如果生命周期小于Collection实例的生命周期,则集合构造函数不应存储对输入参数(std :: string和DataMap)的引用。
  • DataMap.initDataCollection()不应存储指向Collection实例的指针,因为在堆栈上创建的Collection实例的生命周期小于DataMap实例的生命周期。在从CollectionFactory :: createFromMap()返回之前,集合实例将被销毁。
  • 集合应该有正确实现的复制构造函数和/或复制赋值运算符,否则会遇到上述问题。

答案 1 :(得分:1)

这是第一个Valgrind错误还是以前的错误?

我的猜测是有以前的那些,其中一个是破坏记忆并导致爆发。

答案 2 :(得分:1)

你是如何链接代码的?例如,如果要创建共享库,则可能需要为与位置无关的代码指定编译标志,例如-fPIC(gcc / g ++)。