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

MTRSetupPayload initWithSetupPasscode should not produce invalid payloads. #31132

Conversation

bzbarsky-apple
Copy link
Contributor

It's using a long discriminator, so has to have discovery capabilities available.

Fixes #31129

…oads.

It's using a long discriminator, so has to have discovery capabilities
available.

Fixes project-chip#31129
Copy link

github-actions bot commented Dec 20, 2023

PR #31132: Size comparison from 19e202e to 0c218cc

Decreases (1 build for efr32)
platform target config section 19e202e 0c218cc change % change
efr32 lock-app BRD4161A+wf200 (read/write) 1114848 1114840 -8 -0.0
.text 923300 923292 -8 -0.0
Full report (72 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
platform target config section 19e202e 0c218cc change % change
bl602 lighting-app bl602 (read/write) 1417794 1417794 0 0.0
.bss 85160 85160 0 0.0
.data 9416 9416 0 0.0
.rodata 156912 156912 0 0.0
.text 1085664 1085664 0 0.0
bl602+mfd (read/write) 1432522 1432522 0 0.0
.bss 85328 85328 0 0.0
.data 9384 9384 0 0.0
.rodata 155872 155872 0 0.0
.text 1101288 1101288 0 0.0
bl602+rpc (read/write) 1465578 1465578 0 0.0
.bss 93192 93192 0 0.0
.data 9784 9784 0 0.0
.rodata 164536 164536 0 0.0
.text 1117408 1117408 0 0.0
bl702 lighting-app bl702 (read only) 3478 3478 0 0.0
(read/write) 1182811 1182811 0 0.0
.bss 11197 11197 0 0.0
.data 3584 3584 0 0.0
.rodata 106288 106288 0 0.0
.text 955366 955366 0 0.0
bl702+mfd (read only) 3478 3478 0 0.0
(read/write) 1193943 1193943 0 0.0
.bss 11373 11373 0 0.0
.data 3568 3568 0 0.0
.rodata 105228 105228 0 0.0
.text 967430 967430 0 0.0
bl702+rpc (read only) 3478 3478 0 0.0
(read/write) 1274823 1274823 0 0.0
.bss 19677 19677 0 0.0
.data 4144 4144 0 0.0
.rodata 121708 121708 0 0.0
.text 1030428 1030428 0 0.0
bl706-eth (read/write) 1002437 1002437 0 0.0
.bss 23772 23772 0 0.0
.data 3160 3160 0 0.0
.rodata 99620 99620 0 0.0
.text 748262 748262 0 0.0
bl706-wifi (read/write) 1240574 1240574 0 0.0
.bss 10641 10641 0 0.0
.data 3600 3600 0 0.0
.rodata 120600 120600 0 0.0
.text 975558 975558 0 0.0
bl702l lighting-app bl702l (read only) 512 512 0 0.0
(read/write) 1152516 1152516 0 0.0
.bss 16392 16392 0 0.0
.data 4952 4952 0 0.0
.rodata 100292 100292 0 0.0
.text 948492 948492 0 0.0
bl702l+mfd (read only) 512 512 0 0.0
(read/write) 1163968 1163968 0 0.0
.bss 16568 16568 0 0.0
.data 4928 4928 0 0.0
.rodata 99232 99232 0 0.0
.text 960868 960868 0 0.0
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 (read only) 771712 771712 0 0.0
(read/write) 168100 168100 0 0.0
.bss 90436 90436 0 0.0
.data 3460 3460 0 0.0
.rodata 89536 89536 0 0.0
.text 681908 681908 0 0.0
lock-ftd LP_EM_CC1354P10_6 (read only) 787816 787816 0 0.0
(read/write) 178328 178328 0 0.0
.bss 100612 100612 0 0.0
.data 3488 3488 0 0.0
.rodata 83792 83792 0 0.0
.text 703760 703760 0 0.0
lock-mtd LP_EM_CC1354P10_6 (read only) 776032 776032 0 0.0
(read/write) 172784 172784 0 0.0
.bss 95068 95068 0 0.0
.data 3488 3488 0 0.0
.rodata 110392 110392 0 0.0
.text 665376 665376 0 0.0
pump-app LP_EM_CC1354P10_6 (read only) 728648 728648 0 0.0
(read/write) 167096 167096 0 0.0
.bss 89208 89208 0 0.0
.data 3448 3448 0 0.0
.rodata 85112 85112 0 0.0
.text 643268 643268 0 0.0
pump-controller-app LP_EM_CC1354P10_6 (read only) 714184 714184 0 0.0
(read/write) 167276 167276 0 0.0
.bss 89392 89392 0 0.0
.data 3444 3444 0 0.0
.rodata 80944 80944 0 0.0
.text 632972 632972 0 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL (read only) 566994 566994 0 0.0
(read/write) 207652 207652 0 0.0
.bss 201116 201116 0 0.0
.data 1412 1412 0 0.0
.rodata 83938 83938 0 0.0
.text 480932 480932 0 0.0
lock CC3235SF_LAUNCHXL (read only) 620322 620322 0 0.0
(read/write) 208048 208048 0 0.0
.bss 201480 201480 0 0.0
.data 1448 1448 0 0.0
.rodata 105810 105810 0 0.0
.text 512388 512388 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 561023 561023 0 0.0
.app_xip_area 451273 451273 0 0.0
.bss 64664 64664 0 0.0
.data 660 660 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 543247 543247 0 0.0
.app_xip_area 428761 428761 0 0.0
.bss 69360 69360 0 0.0
.data 700 700 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor cyw930739m2evb_01 (read/write) 561983 561983 0 0.0
.app_xip_area 453689 453689 0 0.0
.bss 63280 63280 0 0.0
.data 588 588 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
switch cyw930739m2evb_01 (read/write) 557423 557423 0 0.0
.app_xip_area 445905 445905 0 0.0
.bss 66400 66400 0 0.0
.data 696 696 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A+rs9116 (read/write) 1017020 1017020 0 0.0
.bss 172600 172600 0 0.0
.data 2688 2688 0 0.0
.text 841712 841712 0 0.0
BRD4187C (read/write) 1100980 1100980 0 0.0
.bss 196204 196204 0 0.0
.data 3296 3296 0 0.0
.text 901460 901460 0 0.0
lock-app BRD4161A+wf200 (read/write) 1114848 1114840 -8 -0.0
.bss 188816 188816 0 0.0
.data 2712 2712 0 0.0
.text 923300 923292 -8 -0.0
window-app BRD4187C (read/write) 1135744 1135744 0 0.0
.bss 168236 168236 0 0.0
.data 3168 3168 0 0.0
.text 964320 964320 0 0.0
esp32 all-clusters-app c3devkit (read only) 1137758 1137758 0 0.0
(read/write) 1671564 1671564 0 0.0
.dram0.bss 72872 72872 0 0.0
.dram0.data 12988 12988 0 0.0
.flash.rodata 241832 241832 0 0.0
.flash.text 1137758 1137758 0 0.0
.iram0.text 75694 75694 0 0.0
m5stack (read only) 1182763 1182763 0 0.0
(read/write) 520756 520756 0 0.0
.dram0.bss 79520 79520 0 0.0
.dram0.data 34004 34004 0 0.0
.flash.rodata 272868 272868 0 0.0
.flash.text 1177379 1177379 0 0.0
.iram0.text 123907 123907 0 0.0
k32w contact k32w0+release (read only) 605208 605208 0 0.0
(read/write) 81896 81896 0 0.0
.bss 69744 69744 0 0.0
.data 2168 2168 0 0.0
.text 604672 604672 0 0.0
k32w1+release (read only) 1024 1024 0 0.0
(read/write) 690680 690680 0 0.0
.bss 67600 67600 0 0.0
.data 2716 2716 0 0.0
.text 576888 576888 0 0.0
light k32w0+release (read only) 594912 594912 0 0.0
(read/write) 79112 79112 0 0.0
.bss 67008 67008 0 0.0
.data 2120 2120 0 0.0
.text 594376 594376 0 0.0
k32w1+release (read only) 1024 1024 0 0.0
(read/write) 782484 782484 0 0.0
.bss 77276 77276 0 0.0
.data 1928 1928 0 0.0
.text 659824 659824 0 0.0
lock k32w0+release (read only) 576312 576312 0 0.0
(read/write) 79376 79376 0 0.0
.bss 67296 67296 0 0.0
.data 2096 2096 0 0.0
.text 575776 575776 0 0.0
linux air-purifier-app debug (read only) 2525829 2525829 0 0.0
(read/write) 127208 127208 0 0.0
.bss 46944 46944 0 0.0
.data 1472 1472 0 0.0
.data.rel.ro 72824 72824 0 0.0
.dynamic 608 608 0 0.0
.got 4552 4552 0 0.0
.init 27 27 0 0.0
.init_array 800 800 0 0.0
.rodata 179240 179240 0 0.0
.text 2171285 2171285 0 0.0
all-clusters-app debug (read only) 5415297 5415297 0 0.0
(read/write) 456232 456232 0 0.0
.bss 131632 131632 0 0.0
.data 3840 3840 0 0.0
.data.rel.ro 313512 313512 0 0.0
.dynamic 624 624 0 0.0
.got 5336 5336 0 0.0
.init 27 27 0 0.0
.init_array 1264 1264 0 0.0
.rodata 312540 312540 0 0.0
.text 4692131 4692131 0 0.0
all-clusters-minimal-app debug (read only) 4952889 4952889 0 0.0
(read/write) 234688 234688 0 0.0
.bss 127856 127856 0 0.0
.data 3776 3776 0 0.0
.data.rel.ro 95960 95960 0 0.0
.dynamic 624 624 0 0.0
.got 5296 5296 0 0.0
.init 27 27 0 0.0
.init_array 1160 1160 0 0.0
.rodata 272749 272749 0 0.0
.text 4447779 4447779 0 0.0
bridge-app debug (read only) 4416625 4416625 0 0.0
(read/write) 215928 215928 0 0.0
.bss 119224 119224 0 0.0
.data 4704 4704 0 0.0
.data.rel.ro 85280 85280 0 0.0
.dynamic 624 624 0 0.0
.got 5264 5264 0 0.0
.init 27 27 0 0.0
.init_array 824 824 0 0.0
.rodata 219061 219061 0 0.0
.text 3979091 3979091 0 0.0
chip-tool debug (read only) 1172507 1172507 0 0.0
(read/write) 505840 505840 0 0.0
.bss 94200 94200 0 0.0
.data 4994 4994 0 0.0
.data.rel.ro 399408 399408 0 0.0
.dynamic 624 624 0 0.0
.got 5784 5784 0 0.0
.init 27 27 0 0.0
.init_array 784 784 0 0.0
.rodata 425833 425833 0 0.0
.text 10606835 10606835 0 0.0
chip-tool-ipv6only arm64 (read only) 11448848 11448848 0 0.0
(read/write) 574664 574664 0 0.0
.bss 103448 103448 0 0.0
.data 4288 4288 0 0.0
.data.rel.ro 438304 438304 0 0.0
.dynamic 512 512 0 0.0
.got 19856 19856 0 0.0
.init 24 24 0 0.0
.init_array 248 248 0 0.0
.rodata 346152 346152 0 0.0
.text 10232056 10232056 0 0.0
lighting-app debug+rpc+ui (read only) 5302441 5302441 0 0.0
(read/write) 225000 225000 0 0.0
.bss 119360 119360 0 0.0
.data 4416 4416 0 0.0
.data.rel.ro 93696 93696 0 0.0
.dynamic 672 672 0 0.0
.got 5880 5880 0 0.0
.init 27 27 0 0.0
.init_array 960 960 0 0.0
.rodata 340276 340276 0 0.0
.text 4717475 4717475 0 0.0
lock-app debug (read only) 4501649 4501649 0 0.0
(read/write) 203648 203648 0 0.0
.bss 113760 113760 0 0.0
.data 3648 3648 0 0.0
.data.rel.ro 79512 79512 0 0.0
.dynamic 624 624 0 0.0
.got 5216 5216 0 0.0
.init 27 27 0 0.0
.init_array 856 856 0 0.0
.rodata 246901 246901 0 0.0
.text 4044819 4044819 0 0.0
ota-provider-app debug (read only) 4165817 4165817 0 0.0
(read/write) 192728 192728 0 0.0
.bss 113632 113632 0 0.0
.data 3904 3904 0 0.0
.data.rel.ro 69296 69296 0 0.0
.dynamic 624 624 0 0.0
.got 4528 4528 0 0.0
.init 27 27 0 0.0
.init_array 720 720 0 0.0
.rodata 201205 201205 0 0.0
.text 3772835 3772835 0 0.0
ota-requestor-app debug (read only) 4255305 4255305 0 0.0
(read/write) 197008 197008 0 0.0
.bss 114656 114656 0 0.0
.data 4176 4176 0 0.0
.data.rel.ro 72288 72288 0 0.0
.dynamic 624 624 0 0.0
.got 4464 4464 0 0.0
.init 27 27 0 0.0
.init_array 776 776 0 0.0
.rodata 205045 205045 0 0.0
.text 3854083 3854083 0 0.0
shell debug (read only) 2788297 2788297 0 0.0
(read/write) 152272 152272 0 0.0
.bss 62312 62312 0 0.0
.data 768 768 0 0.0
.data.rel.ro 83400 83400 0 0.0
.dynamic 592 592 0 0.0
.got 4088 4088 0 0.0
.init 27 27 0 0.0
.init_array 1088 1088 0 0.0
.rodata 181856 181856 0 0.0
.text 2435474 2435474 0 0.0
thermostat-no-ble arm64 (read only) 4405248 4405248 0 0.0
(read/write) 245216 245216 0 0.0
.bss 122232 122232 0 0.0
.data 2784 2784 0 0.0
.data.rel.ro 102352 102352 0 0.0
.dynamic 512 512 0 0.0
.got 12008 12008 0 0.0
.init 24 24 0 0.0
.init_array 424 424 0 0.0
.rodata 162236 162236 0 0.0
.text 3902712 3902712 0 0.0
tv-app debug (read only) 5398473 5398473 0 0.0
(read/write) 349280 349280 0 0.0
.bss 244704 244704 0 0.0
.data 4992 4992 0 0.0
.data.rel.ro 92280 92280 0 0.0
.dynamic 624 624 0 0.0
.got 5512 5512 0 0.0
.init 27 27 0 0.0
.init_array 1144 1144 0 0.0
.rodata 273461 273461 0 0.0
.text 4887171 4887171 0 0.0
tv-casting-app debug (read only) 9603105 9603105 0 0.0
(read/write) 339008 339008 0 0.0
.bss 156352 156352 0 0.0
.data 2464 2464 0 0.0
.data.rel.ro 173336 173336 0 0.0
.dynamic 624 624 0 0.0
.got 5064 5064 0 0.0
.init 27 27 0 0.0
.init_array 1144 1144 0 0.0
.rodata 363592 363592 0 0.0
.text 8774099 8774099 0 0.0
mbed lock-app-release cy8cproto_062_4343w (read only) 6224 6224 0 0.0
(read/write) 2525992 2525992 0 0.0
.bss 220216 220216 0 0.0
.data 5144 5144 0 0.0
.text 1488676 1488676 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read only) 4 4 0 0.0
(read/write) 1067836 1067836 0 0.0
bss 129393 129393 0 0.0
rodata 105436 105436 0 0.0
text 785436 785436 0 0.0
nrf7002dk_nrf5340_cpuapp (read only) 4 4 0 0.0
(read/write) 1193944 1193944 0 0.0
bss 121603 121603 0 0.0
rodata 139196 139196 0 0.0
text 782816 782816 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read only) 4 4 0 0.0
(read/write) 1040168 1040168 0 0.0
bss 129348 129348 0 0.0
rodata 98872 98872 0 0.0
text 764324 764324 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 (read only) 835864 835864 0 0.0
(read/write) 1751132 1751132 0 0.0
.bss 195004 195004 0 0.0
.data 2472 2472 0 0.0
.text 1545268 1545268 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 (read only) 837912 837912 0 0.0
(read/write) 1705940 1705940 0 0.0
.bss 192956 192956 0 0.0
.data 2472 2472 0 0.0
.text 1502124 1502124 0 0.0
light cy8ckit_062s2_43012 (read only) 844600 844600 0 0.0
(read/write) 1630924 1630924 0 0.0
.bss 186380 186380 0 0.0
.data 2360 2360 0 0.0
.text 1433796 1433796 0 0.0
lock cy8ckit_062s2_43012 (read only) 817480 817480 0 0.0
(read/write) 1657812 1657812 0 0.0
.bss 213476 213476 0 0.0
.data 2384 2384 0 0.0
.text 1433564 1433564 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1116596 1116596 0 0.0
.bss 101216 101216 0 0.0
.data 740 740 0 0.0
.text 628972 628972 0 0.0
lock-app qpg6105+debug (read/write) 1080588 1080588 0 0.0
.bss 97512 97512 0 0.0
.data 784 784 0 0.0
.text 592968 592968 0 0.0
telink air-quality-sensor-app tlsr9528a_retention (read only) 51774 51774 0 0.0
(read/write) 821378 821378 0 0.0
bss 55064 55064 0 0.0
text 613736 613736 0 0.0
all-clusters-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 1086744 1086744 0 0.0
bss 100404 100404 0 0.0
text 767230 767230 0 0.0
all-clusters-minimal-app tlsr9528a (read only) 4 4 0 0.0
(read/write) 1084036 1084036 0 0.0
bss 109764 109764 0 0.0
text 753368 753368 0 0.0
bridge-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 934668 934668 0 0.0
bss 94272 94272 0 0.0
text 646786 646786 0 0.0
contact-sensor-app tlsr9528a_retention (read only) 51774 51774 0 0.0
(read/write) 821614 821614 0 0.0
bss 55108 55108 0 0.0
text 614106 614106 0 0.0
light-switch-app-ota-shell-factory-data tlsr9528a (read only) 4 4 0 0.0
(read/write) 978700 978700 0 0.0
bss 79092 79092 0 0.0
text 694196 694196 0 0.0
lighting-app-ota-rpc-factory-data-4mb tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 1093016 1093016 0 0.0
bss 99836 99836 0 0.0
text 769678 769678 0 0.0
lock-app-dfu tlsr9528a (read only) 4 4 0 0.0
(read/write) 965332 965332 0 0.0
bss 97356 97356 0 0.0
text 647872 647872 0 0.0
ota-requestor-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 948344 948344 0 0.0
bss 92380 92380 0 0.0
text 662894 662894 0 0.0
pump-app tlsr9528a (read only) 4 4 0 0.0
(read/write) 880412 880412 0 0.0
bss 69084 69084 0 0.0
text 622116 622116 0 0.0
pump-controller-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 821024 821024 0 0.0
bss 57320 57320 0 0.0
text 588992 588992 0 0.0
shell tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 705600 705600 0 0.0
bss 73588 73588 0 0.0
text 462094 462094 0 0.0
smoke_co_alarm-app tlsr9528a_retention (read only) 51774 51774 0 0.0
(read/write) 823350 823350 0 0.0
bss 55244 55244 0 0.0
text 616850 616850 0 0.0
temperature-measurement-app-mars-ota tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 878117 878117 0 0.0
bss 60796 60796 0 0.0
text 630732 630732 0 0.0
thermostat tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 845396 845396 0 0.0
bss 57616 57616 0 0.0
text 607458 607458 0 0.0
window-covering tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 839672 839672 0 0.0
bss 57968 57968 0 0.0
text 601492 601492 0 0.0

@mergify mergify bot merged commit d0d95d3 into project-chip:master Dec 22, 2023
57 checks passed
@bzbarsky-apple bzbarsky-apple deleted the fix-payload-init-discovery-capabilities branch January 2, 2024 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Platform] Failed encoding invalid payload with setupCommissioningSession
3 participants