Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ExchangeContext leak in Darwin unit test #6668

Closed
kghost opened this issue May 11, 2021 · 0 comments · Fixed by #10359
Closed

ExchangeContext leak in Darwin unit test #6668

kghost opened this issue May 11, 2021 · 0 comments · Fixed by #10359
Assignees

Comments

@kghost
Copy link
Contributor

kghost commented May 11, 2021

Problem

https://github.com/project-chip/connectedhomeip/pull/5389/checks?check_run_id=2554419054

2021-05-11 10:19:12.280249+0000 xctest[9231:42649] Thermostat SystemMode Error: (null)
Test Case '-[CHIPClustersTests testSendClusterThermostatWriteAttributeSystemModeWithValue]' passed (0.004 seconds).
Test Case '-[CHIPClustersTests testShutdownStack]' started.
2021-05-11 10:19:12.281295+0000 xctest[9231:42649] PersistentStorageDelegate Delete Key: PairedDevice1
2021-05-11 10:19:12.281862+0000 xctest[9231:42649] PersistentStorageDelegate Set Key ListPairedDevices0
2021-05-11 10:19:12.282071+0000 xctest[9231:42649] Shutting down the CHIP Stack
[1620728352282] [0xa699] CHIP: [CTL] Shutting down the commissioner
[1620728352282] [0xa699] CHIP: [CTL] Shutting down the controller
Assertion failed: (ec.GetReferenceCount() == 0), function Shutdown, file ../../../../../../../../../../../work/connectedhomeip/connectedhomeip/src/messaging/ExchangeMgr.cpp, line 104.
2021-05-11 10:19:33.043 xcodebuild[8074:43675]  IDETestOperationsObserverDebug: Writing diagnostic log for test session to:
/Users/runner/Library/Developer/Xcode/DerivedData/CHIP-dhyjrdlylemwdfesoclvgithlmec/Logs/Test/Test-CHIP Framework Tests-2021.05.11_10-17-34-+0000.xcresult/Staging/1_Test/Diagnostics/CHIPTests-DBE91E36-1FF3-4813-9B0B-6A4014870F80/CHIPTests-3324E36C-FE7B-4E61-A3B2-2681A842D48A/Session-CHIPTests-2021-05-11_101933-rd5sts.log
2021-05-11 10:19:33.043 xcodebuild[8074:38898] [MT] IDETestOperationsObserverDebug: (36B55EE0-E532-4205-BEAF-BF1EA7CE8F6E) Beginning test session CHIPTests-36B55EE0-E532-4205-BEAF-BF1EA7CE8F6E at 2021-05-11 10:19:33.043 with Xcode 12D4e on target <DVTLocalComputer: 0x7fe6e9ca61c0 (My Mac | x86_64)> (10.15.7 (19H1030))

There is an assert fail while shutting down the exchange manager.

@kghost kghost self-assigned this May 11, 2021
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Oct 8, 2021
The leak that caused us to not do this before seems to be fixed with
the exchange lifetime work that has happened since.

Fixes project-chip#6668
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Oct 8, 2021
The leak that caused us to not do this before seems to be fixed with
the exchange lifetime work that has happened since.

Fixes project-chip#6668
bzbarsky-apple added a commit that referenced this issue Oct 11, 2021
…te. (#10359)

The leak that caused us to not do this before seems to be fixed with
the exchange lifetime work that has happened since.

Fixes #6668
mrjerryjohns pushed a commit to mrjerryjohns/connectedhomeip that referenced this issue Oct 11, 2021
…te. (project-chip#10359)

The leak that caused us to not do this before seems to be fixed with
the exchange lifetime work that has happened since.

Fixes project-chip#6668
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant