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

Fix 12692: Chiptool cannot commission tv-app on same machine #12693

Merged
merged 2 commits into from
Dec 8, 2021

Conversation

chrisdecenzo
Copy link
Contributor

Problem

Change overview

  • Problem is that both tv-app and chip-tool are configured to listen on the same default commissioner port (5542)
  • Solution is to modify tv-app configuration to not use the default commissioner port

Testing

  • Commission tv-app from chiptool running on same host:

./out/host/chip-tv-app --secured-device-port 5640
./out/host/chip-tool pairing onnetwork-long 1234 20202021 3840

@todo
Copy link

todo bot commented Dec 7, 2021

Replace testingRootStore with a AttestationTrustStore that has the necessary official PAA roots available

// TODO: Replace testingRootStore with a AttestationTrustStore that has the necessary official PAA roots available
const chip::Credentials::AttestationTrustStore * testingRootStore = chip::Credentials::GetTestAttestationTrustStore();
SetDeviceAttestationVerifier(GetDefaultDACVerifier(testingRootStore));
chip::Platform::ScopedMemoryBuffer<uint8_t> noc;
VerifyOrReturnError(noc.Alloc(chip::Controller::kMaxCHIPDERCertLength), CHIP_ERROR_NO_MEMORY);


This comment was generated by todo based on a TODO comment in a33c897 in #12693. cc @project-chip.

@github-actions
Copy link

github-actions bot commented Dec 7, 2021

PR #12693: Size comparison from 051685b to 97075ca

Full report (18 builds for esp32, k32w, nrfconnect, qpg)
platform target config section 051685b 97075ca change % change
esp32 all-clusters-app c3devkit (read only) 856796 856796 0 0.0
(read/write) 1298938 1298938 0 0.0
.dram0.bss 61096 61096 0 0.0
.dram0.data 14132 14132 0 0.0
.flash.rodata 170664 170664 0 0.0
.flash.text 856796 856796 0 0.0
.iram0.text 62028 62028 0 0.0
m5stack (read only) 960115 960115 0 0.0
(read/write) 443244 443244 0 0.0
.dram0.bss 68448 68448 0 0.0
.dram0.data 34056 34056 0 0.0
.flash.rodata 208952 208952 0 0.0
.flash.text 954731 954731 0 0.0
.iram0.text 123451 123451 0 0.0
k32w lighting-app k32w061+se05x+release (read/write) 732484 732484 0 0.0
.bss 79296 79296 0 0.0
.data 1860 1860 0 0.0
.text 645528 645528 0 0.0
lock-app k32w061+debug (read/write) 623164 623164 0 0.0
.bss 69960 69960 0 0.0
.data 1828 1828 0 0.0
.text 545576 545576 0 0.0
shell k32w061+debug (read/write) 689084 689084 0 0.0
.bss 81596 81596 0 0.0
.data 1800 1800 0 0.0
.text 599888 599888 0 0.0
nrfconnect lighting-app nrf52840dk_nrf52840 (read/write) 896707 896707 0 0.0
bss 113724 113724 0 0.0
rodata 100308 100308 0 0.0
text 607112 607112 0 0.0
nrf52840dk_nrf52840+rpc (read/write) 859643 859643 0 0.0
bss 110072 110072 0 0.0
rodata 91500 91500 0 0.0
text 581848 581848 0 0.0
nrf5340dk_nrf5340_cpuapp (read/write) 822638 822638 0 0.0
bss 115100 115100 0 0.0
rodata 95568 95568 0 0.0
text 537480 537480 0 0.0
lock-app nrf52840dk_nrf52840 (read/write) 869275 869275 0 0.0
bss 110988 110988 0 0.0
rodata 96428 96428 0 0.0
text 586468 586468 0 0.0
nrf5340dk_nrf5340_cpuapp (read/write) 795458 795458 0 0.0
bss 112396 112396 0 0.0
rodata 91716 91716 0 0.0
text 516932 516932 0 0.0
pigweed-app nrf52840dk_nrf52840 (read/write) 497463 497463 0 0.0
bss 51820 51820 0 0.0
rodata 45852 45852 0 0.0
text 339492 339492 0 0.0
pump-app nrf52840dk_nrf52840 (read/write) 874027 874027 0 0.0
bss 110900 110900 0 0.0
rodata 97780 97780 0 0.0
text 589884 589884 0 0.0
pump-controller-app nrf52840dk_nrf52840 (read/write) 867235 867235 0 0.0
bss 110776 110776 0 0.0
rodata 95916 95916 0 0.0
text 585092 585092 0 0.0
shell nrf52840dk_nrf52840 (read/write) 781643 781643 0 0.0
bss 109552 109552 0 0.0
rodata 74296 74296 0 0.0
text 523284 523284 0 0.0
nrf5340dk_nrf5340_cpuapp (read/write) 696718 696718 0 0.0
bss 110532 110532 0 0.0
rodata 68936 68936 0 0.0
text 443924 443924 0 0.0
qpg lighting-app qpg6100+debug (read only) 514144 514144 0 0.0
(read/write) 122332 122332 0 0.0
.bss 80168 80168 0 0.0
.data 964 964 0 0.0
.text 508824 508824 0 0.0
lock-app qpg6100+debug (read only) 488448 488448 0 0.0
(read/write) 122336 122336 0 0.0
.bss 79304 79304 0 0.0
.data 920 920 0 0.0
.text 483128 483128 0 0.0
persistent-storage-app qpg6100+debug (read only) 108104 108104 0 0.0
(read/write) 122336 122336 0 0.0
.bss 36152 36152 0 0.0
.data 288 288 0 0.0
.text 102784 102784 0 0.0

@woody-apple woody-apple merged commit d0eb6dd into master Dec 8, 2021
@woody-apple woody-apple deleted the tv-app-broken branch December 8, 2021 00:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Chiptool cannot commission tv-app on same machine
5 participants