Project

General

Profile

Actions

Bug #3322

closed

Face/TestTcp/FaceClosing fails assertion in Signal::~Signal()

Added by Davide Pesavento over 8 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Faces
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:

Description

$ ./build/unit-tests-daemon -l all -t Face/TestTcp/FaceClosing
Running 1 test case...
Entering test suite "Master Test Suite"
Entering test suite "Face"
Entering test suite "TestTcp"
Entering test case "FaceClosing"
1446740427.944282 INFO: [TcpTransport] [id=0,local=tcp4://127.0.0.1:20070,remote=tcp4://127.0.0.1:39728] Creating transport
../tests/daemon/face/tcp.t.cpp(138): info: check !static_cast<bool>(face1) passed
1446740427.944647 INFO: [TcpTransport] [id=0,local=tcp4://127.0.0.1:39728,remote=tcp4://127.0.0.1:20070] Creating transport
../tests/daemon/face/tcp.t.cpp(181): info: check !static_cast<bool>(face2) passed
../tests/daemon/face/tcp.t.cpp(474): info: check 'TcpChannel error: cannot connect or cannot accept connection' passed
../tests/daemon/face/tcp.t.cpp(476): info: check channel1->size() == 1 passed
../tests/daemon/face/tcp.t.cpp(477): info: check channel2->size() == 1 passed
../tests/daemon/face/tcp.t.cpp(479): info: check static_cast<bool>(face1) passed
../tests/daemon/face/tcp.t.cpp(480): info: check static_cast<bool>(face2) passed
1446740428.045295 INFO: [Transport] [id=0,local=tcp4://127.0.0.1:20070,remote=tcp4://127.0.0.1:39728] setState UP -> CLOSING
1446740428.045657 INFO: [Transport] [id=0,local=tcp4://127.0.0.1:20070,remote=tcp4://127.0.0.1:39728] setState CLOSING -> CLOSED
unit-tests-daemon: /usr/local/include/ndn-cxx/util/signal-signal.hpp:170: ndn::util::signal::Signal<Owner, TArgs>::~Signal() [with Owner = nfd::face::Transport; TArgs = {nfd::face::TransportState, nfd::face::TransportState}]: Assertion `!m_isExecuting' failed.
unknown location(0): fatal error in "FaceClosing": signal: SIGABRT (application abort requested)
../tests/daemon/face/tcp.t.cpp(486): last checkpoint
Leaving test case "FaceClosing"; testing time: 1706mks
Leaving test suite "TestTcp"
Leaving test suite "Face"
Leaving test suite "Master Test Suite"

*** 1 failure detected in test suite "Master Test Suite"
Actions #1

Updated by Davide Pesavento over 8 years ago

This is probably due to EndToEndFixture immediately resetting the shared_ptr in the onFail handler.

Actions #2

Updated by Junxiao Shi over 8 years ago

  • Status changed from New to Resolved
Actions #3

Updated by Davide Pesavento over 8 years ago

I get the same assertion failure in Face/TestWebSocket

Actions #4

Updated by Davide Pesavento over 8 years ago

  • Status changed from Resolved to Code review
  • Assignee set to Junxiao Shi
  • % Done changed from 0 to 100

Patch is not merged so "resolved" is incorrect.

Also, I strongly suggest to fix this bug with a separate commit.

Actions #6

Updated by Junxiao Shi over 8 years ago

  • Status changed from Code review to Closed
Actions

Also available in: Atom PDF