You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@mesos.apache.org by "Joseph Wu (JIRA)" <ji...@apache.org> on 2016/02/10 00:09:18 UTC
[jira] [Updated] (MESOS-4634) Tests will dereference stack
allocated master objects upon assertion/expectation failure.
[ https://issues.apache.org/jira/browse/MESOS-4634?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Joseph Wu updated MESOS-4634:
-----------------------------
Description:
Tests that use the {{StartMaster}} test helper are generally fragile when the test fails an assert/expect in the middle of the test. This is because the {{StartMaster}} helper takes raw pointer arguments, which may be stack-allocated.
In case of an assert failure, the test immediately exits (destroying stack allocated objects) and proceeds onto test cleanup. The test cleanup may dereference some of these destroyed objects, leading to a test crash like:
{code}
[18:27:36][Step 8/8] F0204 18:27:35.981302 23085 logging.cpp:64] RAW: Pure virtual method called
[18:27:36][Step 8/8] @ 0x7f7077055e1c google::LogMessage::Fail()
[18:27:36][Step 8/8] @ 0x7f707705ba6f google::RawLog__()
[18:27:36][Step 8/8] @ 0x7f70760f76c9 __cxa_pure_virtual
[18:27:36][Step 8/8] @ 0xa9423c mesos::internal::tests::Cluster::Slaves::shutdown()
[18:27:36][Step 8/8] @ 0x1074e45 mesos::internal::tests::MesosTest::ShutdownSlaves()
[18:27:36][Step 8/8] @ 0x1074de4 mesos::internal::tests::MesosTest::Shutdown()
[18:27:36][Step 8/8] @ 0x1070ec7 mesos::internal::tests::MesosTest::TearDown()
{code}
The {{StartMaster}} helper should take {{shared_ptr}} arguments instead.
This also means that we can remove the {{Shutdown}} helper from most of these tests.
was:
Tests that use the {{StartSlave}} test helper are generally fragile when the test fails an assert/expect in the middle of the test. This is because the {{StartSlave}} helper takes raw pointer arguments, which may be stack-allocated.
In case of an assert failure, the test immediately exits (destroying stack allocated objects) and proceeds onto test cleanup. The test cleanup may dereference some of these destroyed objects, leading to a test crash like:
{code}
[18:27:36][Step 8/8] F0204 18:27:35.981302 23085 logging.cpp:64] RAW: Pure virtual method called
[18:27:36][Step 8/8] @ 0x7f7077055e1c google::LogMessage::Fail()
[18:27:36][Step 8/8] @ 0x7f707705ba6f google::RawLog__()
[18:27:36][Step 8/8] @ 0x7f70760f76c9 __cxa_pure_virtual
[18:27:36][Step 8/8] @ 0xa9423c mesos::internal::tests::Cluster::Slaves::shutdown()
[18:27:36][Step 8/8] @ 0x1074e45 mesos::internal::tests::MesosTest::ShutdownSlaves()
[18:27:36][Step 8/8] @ 0x1074de4 mesos::internal::tests::MesosTest::Shutdown()
[18:27:36][Step 8/8] @ 0x1070ec7 mesos::internal::tests::MesosTest::TearDown()
{code}
The {{StartSlave}} helper should take {{shared_ptr}} arguments instead.
This also means that we can remove the {{Shutdown}} helper from most of these tests.
> Tests will dereference stack allocated master objects upon assertion/expectation failure.
> -----------------------------------------------------------------------------------------
>
> Key: MESOS-4634
> URL: https://issues.apache.org/jira/browse/MESOS-4634
> Project: Mesos
> Issue Type: Bug
> Reporter: Joseph Wu
> Assignee: Joseph Wu
> Labels: flaky, mesosphere, test
>
> Tests that use the {{StartMaster}} test helper are generally fragile when the test fails an assert/expect in the middle of the test. This is because the {{StartMaster}} helper takes raw pointer arguments, which may be stack-allocated.
> In case of an assert failure, the test immediately exits (destroying stack allocated objects) and proceeds onto test cleanup. The test cleanup may dereference some of these destroyed objects, leading to a test crash like:
> {code}
> [18:27:36][Step 8/8] F0204 18:27:35.981302 23085 logging.cpp:64] RAW: Pure virtual method called
> [18:27:36][Step 8/8] @ 0x7f7077055e1c google::LogMessage::Fail()
> [18:27:36][Step 8/8] @ 0x7f707705ba6f google::RawLog__()
> [18:27:36][Step 8/8] @ 0x7f70760f76c9 __cxa_pure_virtual
> [18:27:36][Step 8/8] @ 0xa9423c mesos::internal::tests::Cluster::Slaves::shutdown()
> [18:27:36][Step 8/8] @ 0x1074e45 mesos::internal::tests::MesosTest::ShutdownSlaves()
> [18:27:36][Step 8/8] @ 0x1074de4 mesos::internal::tests::MesosTest::Shutdown()
> [18:27:36][Step 8/8] @ 0x1070ec7 mesos::internal::tests::MesosTest::TearDown()
> {code}
> The {{StartMaster}} helper should take {{shared_ptr}} arguments instead.
> This also means that we can remove the {{Shutdown}} helper from most of these tests.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)