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 a startup crash when a node wants to advertise as a commissioner but not as commissionable. #24565

Merged

Conversation

bzbarsky-apple
Copy link
Contributor

In DnssdServer::Advertise we were trying to get things like a discriminator and other commissionee information even if we were just advertising as a commissioner (commissionableNode was false). This could lead to crashes due to calling things like GetCommissionableDataProvider() when we are not in fact commissionable and don't have one.

The fix is to reorder things a bit so we set up all the advertiseParameters state that both commissioners and commissionable share, and then condition all the commissionable-only bits on commissionableNode.

…but not as commissionable.

In DnssdServer::Advertise we were trying to get things like a discriminator and
other commissionee information even if we were just advertising as a
commissioner (commissionableNode was false).  This could lead to crashes due to
calling things like GetCommissionableDataProvider() when we are not in fact
commissionable and don't have one.

The fix is to reorder things a bit so we set up all the advertiseParameters
state that both commissioners and commissionable share, and then condition all
the commissionable-only bits on commissionableNode.
@github-actions
Copy link

github-actions bot commented Jan 21, 2023

PR #24565: Size comparison from aedd1de to 4e76323

Increases (36 builds for bl702, cc13x2_26x2, cyw30739, esp32, k32w, linux, nrfconnect, psoc6, qpg, telink)
platform target config section aedd1de 4e76323 change % change
bl702 lighting-app bl702 .debug_info 39912007 39912009 2 0.0
.debug_line 5252223 5252235 12 0.0
.debug_ranges 372056 372176 120 0.0
bl702+rpc .debug_info 44301935 44301937 2 0.0
.debug_line 5649999 5650011 12 0.0
.debug_ranges 395752 395872 120 0.0
cc13x2_26x2 pump-app LP_CC2652R7 (read only) 686535 686543 8 0.0
.text 596000 596008 8 0.0
pump-controller-app LP_CC2652R7 (read only) 671971 671979 8 0.0
.text 585352 585360 8 0.0
shell LP_CC2652R7 (read only) 667186 667194 8 0.0
.text 583180 583188 8 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 585122 585130 8 0.0
.app_xip_area 461148 461156 8 0.0
lock cyw930739m2evb_01 (read/write) 588846 588854 8 0.0
.app_xip_area 459576 459584 8 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 549814 549822 8 0.0
.app_xip_area 431328 431336 8 0.0
esp32 all-clusters-app c3devkit (read only) 1040746 1040752 6 0.0
.flash.text 1040746 1040752 6 0.0
k32w contact k32w0+release (read/write) 660252 660268 16 0.0
.text 561684 561700 16 0.0
linux all-clusters-app debug (read only) 3121393 3121409 16 0.0
.text 2654690 2654706 16 0.0
all-clusters-minimal-app debug (read only) 2933585 2933601 16 0.0
.text 2471506 2471522 16 0.0
bridge-app debug (read only) 2497233 2497249 16 0.0
.text 2112818 2112834 16 0.0
lighting-app debug+rpc (read only) 2664809 2664825 16 0.0
.text 2259458 2259474 16 0.0
lock-app debug (read only) 2623537 2623553 16 0.0
.text 2206866 2206882 16 0.0
ota-provider-app debug (read only) 2197433 2197449 16 0.0
.text 1861426 1861442 16 0.0
ota-requestor-app debug (read only) 2358897 2358913 16 0.0
.text 2007170 2007186 16 0.0
shell debug (read only) 2653961 2653977 16 0.0
.text 2254050 2254066 16 0.0
tv-app debug (read only) 3288081 3288097 16 0.0
.text 2824578 2824594 16 0.0
tv-casting-app debug (read only) 5639393 5639409 16 0.0
.text 4999826 4999842 16 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1163400 1163416 16 0.0
text 803564 803576 12 0.0
nrf7002dk_nrf5340_cpuapp (read/write) 1365104 1365120 16 0.0
text 762800 762808 8 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1109240 1109256 16 0.0
text 772960 772976 16 0.0
psoc6 all-clusters cy8ckit_062s2_43012 .debug_info 27324910 27324913 3 0.0
.debug_line 3753136 3753191 55 0.0
.debug_loc 3643491 3643535 44 0.0
.debug_ranges 356352 356504 152 0.0
all-clusters-minimal cy8ckit_062s2_43012 .debug_info 26886483 26886485 2 0.0
.debug_line 3761933 3761988 55 0.0
.debug_loc 3626891 3626935 44 0.0
.debug_ranges 354320 354472 152 0.0
light cy8ckit_062s2_43012 .debug_info 22362394 22362398 4 0.0
.debug_line 3328440 3328495 55 0.0
.debug_loc 3328766 3328810 44 0.0
.debug_ranges 318544 318696 152 0.0
lock cy8ckit_062s2_43012 .debug_info 22580798 22580801 3 0.0
.debug_line 3329542 3329597 55 0.0
.debug_loc 3350279 3350323 44 0.0
.debug_ranges 320400 320552 152 0.0
qpg lighting-app qpg6105+debug (read/write) 1147692 1147700 8 0.0
.text 594792 594800 8 0.0
lock-app qpg6105+debug (read/write) 1115044 1115052 8 0.0
.text 562140 562148 8 0.0
telink all-clusters-app tlsr9518adk80d (read/write) 1025096 1025112 16 0.0
text 687954 687966 12 0.0
all-clusters-minimal-app tlsr9518adk80d (read/write) 963436 963444 8 0.0
text 651244 651254 10 0.0
contact-sensor-app tlsr9518adk80d (read/write) 868068 868076 8 0.0
text 581742 581752 10 0.0
light-switch-app tlsr9518adk80d (read/write) 884516 884524 8 0.0
text 596344 596352 8 0.0
lighting-app tlsr9518adk80d (read/write) 962860 962868 8 0.0
text 662348 662356 8 0.0
ota-requestor-app tlsr9518adk80d (read/write) 898332 898340 8 0.0
text 607440 607448 8 0.0
thermostat tlsr9518adk80d (read/write) 887740 887748 8 0.0
text 597106 597114 8 0.0
Decreases (8 builds for bl602, bl702, cc13x2_26x2, esp32, linux)
platform target config section aedd1de 4e76323 change % change
bl602 lighting-app bl602 (read/write) 1343978 1343970 -8 -0.0
.text 1021490 1021488 -2 -0.0
bl702 lighting-app bl702 (read/write) 1199315 1199299 -16 -0.0
.debug_loc 3413033 3413016 -17 -0.0
.text 954388 954384 -4 -0.0
bl702+rpc (read/write) 1292119 1292103 -16 -0.0
.debug_loc 3608981 3608964 -17 -0.0
.text 1031540 1031536 -4 -0.0
cc13x2_26x2 pump-app LP_CC2652R7 (read/write) 163520 163512 -8 -0.0
pump-controller-app LP_CC2652R7 (read/write) 178196 178188 -8 -0.0
shell LP_CC2652R7 (read/write) 186676 186668 -8 -0.0
esp32 all-clusters-app m5stack (read only) 1092823 1092815 -8 -0.0
.flash.text 1087439 1087431 -8 -0.0
linux chip-tool-ipv6only arm64 (read only) 10879828 10879812 -16 -0.0
.text 8672772 8672756 -16 -0.0
Full report (54 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
platform target config section aedd1de 4e76323 change % change
bl602 lighting-app bl602 (read/write) 1343978 1343970 -8 -0.0
.bss 94730 94730 0 0.0
.data 9728 9728 0 0.0
.text 1021490 1021488 -2 -0.0
bl602+rpc (read/write) 1389146 1389146 0 0.0
.bss 102778 102778 0 0.0
.data 10120 10120 0 0.0
.text 1052164 1052164 0 0.0
bl702 lighting-app bl702 (read/write) 1199315 1199299 -16 -0.0
.bleromro 6342 6342 0 0.0
.bleromrw 124 124 0 0.0
.boot2 292 292 0 0.0
.bss 71181 71181 0 0.0
.bss_psram 30048 30048 0 0.0
.comment 48 48 0 0.0
.data 4048 4048 0 0.0
.debug_abbrev 1538897 1538897 0 0.0
.debug_aranges 133880 133880 0 0.0
.debug_frame 490720 490720 0 0.0
.debug_info 39912007 39912009 2 0.0
.debug_line 5252223 5252235 12 0.0
.debug_loc 3413033 3413016 -17 -0.0
.debug_ranges 372056 372176 120 0.0
.debug_str 3518605 3518605 0 0.0
.hbn 536 536 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 144 144 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 115328 115328 0 0.0
.rsvd 2960 2960 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 570083 570083 0 0.0
.symtab 172576 172576 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3338 3338 0 0.0
.text 0 0 0 0.0
954388 954384 -4 -0.0
bl702+rpc (read/write) 1292119 1292103 -16 -0.0
.bleromro 6342 6342 0 0.0
.bleromrw 124 124 0 0.0
.boot2 292 292 0 0.0
.bss 79229 79229 0 0.0
.bss_psram 30304 30304 0 0.0
.comment 48 48 0 0.0
.data 4600 4600 0 0.0
.debug_abbrev 1687303 1687303 0 0.0
.debug_aranges 142096 142096 0 0.0
.debug_frame 518384 518384 0 0.0
.debug_info 44301935 44301937 2 0.0
.debug_line 5649999 5650011 12 0.0
.debug_loc 3608981 3608964 -17 -0.0
.debug_ranges 395752 395872 120 0.0
.debug_str 3921739 3921739 0 0.0
.hbn 536 536 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 160 160 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 129824 129824 0 0.0
.rsvd 2960 2960 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 630557 630557 0 0.0
.symtab 190896 190896 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3338 3338 0 0.0
.text 0 0 0 0.0
1031540 1031536 -4 -0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 675915 675915 0 0.0
(read/write) 175876 175876 0 0.0
.bss 81484 81484 0 0.0
.data 3380 3380 0 0.0
.rodata 86947 86947 0 0.0
.text 588656 588656 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 639555 639555 0 0.0
(read/write) 158220 158220 0 0.0
.bss 80724 80724 0 0.0
.data 3380 3380 0 0.0
.rodata 77035 77035 0 0.0
.text 562196 562196 0 0.0
lock-ftd LP_CC2652R7 (read only) 673631 673631 0 0.0
(read/write) 175696 175696 0 0.0
.bss 79108 79108 0 0.0
.data 3304 3304 0 0.0
.rodata 76015 76015 0 0.0
.text 597136 597136 0 0.0
lock-mtd LP_CC2652R7 (read only) 659563 659563 0 0.0
(read/write) 185028 185028 0 0.0
.bss 74372 74372 0 0.0
.data 3304 3304 0 0.0
.rodata 102443 102443 0 0.0
.text 556640 556640 0 0.0
pump-app LP_CC2652R7 (read only) 686535 686543 8 0.0
(read/write) 163520 163512 -8 -0.0
.bss 79068 79068 0 0.0
.data 3268 3268 0 0.0
.rodata 90055 90055 0 0.0
.text 596000 596008 8 0.0
pump-controller-app LP_CC2652R7 (read only) 671971 671979 8 0.0
(read/write) 178196 178188 -8 -0.0
.bss 79180 79180 0 0.0
.data 3292 3292 0 0.0
.rodata 86139 86139 0 0.0
.text 585352 585360 8 0.0
shell LP_CC2652R7 (read only) 667186 667194 8 0.0
(read/write) 186676 186668 -8 -0.0
.bss 83556 83556 0 0.0
.data 3376 3376 0 0.0
.rodata 83690 83690 0 0.0
.text 583180 583188 8 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 585122 585130 8 0.0
.app_xip_area 461148 461156 8 0.0
.bss 66432 66432 0 0.0
.data 728 728 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 588846 588854 8 0.0
.app_xip_area 459576 459584 8 0.0
.bss 71720 71720 0 0.0
.data 736 736 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 549814 549822 8 0.0
.app_xip_area 431328 431336 8 0.0
.bss 60984 60984 0 0.0
.data 684 684 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A+rpc (read/write) 973160 973160 0 0.0
.bss 147144 147144 0 0.0
.data 2188 2188 0 0.0
.text 823804 823804 0 0.0
BRD4161A+rs911x (read/write) 1036096 1036096 0 0.0
.bss 181632 181632 0 0.0
.data 2032 2032 0 0.0
.text 852412 852412 0 0.0
BRD4187C (read/write) 1150108 1150108 0 0.0
.bss 133644 133644 0 0.0
.data 2536 2536 0 0.0
.text 989332 989332 0 0.0
lock-app BRD4161A+wf200 (read/write) 1064316 1064316 0 0.0
.bss 153152 153152 0 0.0
.data 2040 2040 0 0.0
.text 909104 909104 0 0.0
window-app BRD4187C (read/write) 1142808 1142808 0 0.0
.bss 135060 135060 0 0.0
.data 2560 2560 0 0.0
.text 980592 980592 0 0.0
esp32 all-clusters-app c3devkit (read only) 1040746 1040752 6 0.0
(read/write) 1515146 1515146 0 0.0
.dram0.bss 75848 75848 0 0.0
.dram0.data 13784 13784 0 0.0
.flash.rodata 219280 219280 0 0.0
.flash.text 1040746 1040752 6 0.0
.iram0.text 72896 72896 0 0.0
m5stack (read only) 1092823 1092815 -8 -0.0
(read/write) 496807 496807 0 0.0
.dram0.bss 80888 80888 0 0.0
.dram0.data 34072 34072 0 0.0
.flash.rodata 248076 248076 0 0.0
.flash.text 1087439 1087431 -8 -0.0
.iram0.text 124855 124855 0 0.0
k32w contact k32w0+release (read/write) 660252 660268 16 0.0
.bss 77352 77352 0 0.0
.data 2104 2104 0 0.0
.text 561684 561700 16 0.0
light k32w0+release (read/write) 671776 671776 0 0.0
.bss 75144 75144 0 0.0
.data 2060 2060 0 0.0
.text 591844 591844 0 0.0
lock k32w0+release (read/write) 631652 631652 0 0.0
.bss 75904 75904 0 0.0
.data 2080 2080 0 0.0
.text 550940 550940 0 0.0
linux all-clusters-app debug (read only) 3121393 3121409 16 0.0
(read/write) 154032 154032 0 0.0
.bss 63488 63488 0 0.0
.data 2272 2272 0 0.0
.data.rel.ro 81688 81688 0 0.0
.dynamic 608 608 0 0.0
.got 4776 4776 0 0.0
.init 27 27 0 0.0
.init_array 1192 1192 0 0.0
.rodata 281835 281835 0 0.0
.text 2654690 2654706 16 0.0
all-clusters-minimal-app debug (read only) 2933585 2933601 16 0.0
(read/write) 147440 147440 0 0.0
.bss 62624 62624 0 0.0
.data 2240 2240 0 0.0
.data.rel.ro 76104 76104 0 0.0
.dynamic 608 608 0 0.0
.got 4688 4688 0 0.0
.init 27 27 0 0.0
.init_array 1160 1160 0 0.0
.rodata 280491 280491 0 0.0
.text 2471506 2471522 16 0.0
bridge-app debug (read only) 2497233 2497249 16 0.0
(read/write) 132000 132000 0 0.0
.bss 53536 53536 0 0.0
.data 3632 3632 0 0.0
.data.rel.ro 68712 68712 0 0.0
.dynamic 608 608 0 0.0
.got 4608 4608 0 0.0
.init 27 27 0 0.0
.init_array 856 856 0 0.0
.rodata 212544 212544 0 0.0
.text 2112818 2112834 16 0.0
chip-tool debug (read only) 11486321 11486321 0 0.0
(read/write) 644144 644144 0 0.0
.bss 26168 26168 0 0.0
.data 3042 3042 0 0.0
.data.rel.ro 608176 608176 0 0.0
.dynamic 608 608 0 0.0
.got 5352 5352 0 0.0
.init 27 27 0 0.0
.init_array 752 752 0 0.0
.rodata 638277 638277 0 0.0
.text 9355684 9355684 0 0.0
chip-tool-ipv6only arm64 (read only) 10879828 10879812 -16 -0.0
(read/write) 693400 693400 0 0.0
.bss 34168 34168 0 0.0
.data 3008 3008 0 0.0
.data.rel.ro 636976 636976 0 0.0
.dynamic 560 560 0 0.0
.got 14048 14048 0 0.0
.init 24 24 0 0.0
.init_array 200 200 0 0.0
.rodata 565028 565028 0 0.0
.text 8672772 8672756 -16 -0.0
lighting-app debug+rpc (read only) 2664809 2664825 16 0.0
(read/write) 131336 131336 0 0.0
.bss 50976 50976 0 0.0
.data 2288 2288 0 0.0
.data.rel.ro 71848 71848 0 0.0
.dynamic 608 608 0 0.0
.got 4640 4640 0 0.0
.init 27 27 0 0.0
.init_array 928 928 0 0.0
.rodata 228512 228512 0 0.0
.text 2259458 2259474 16 0.0
lock-app debug (read only) 2623537 2623553 16 0.0
(read/write) 126384 126384 0 0.0
.bss 49440 49440 0 0.0
.data 1904 1904 0 0.0
.data.rel.ro 68824 68824 0 0.0
.dynamic 608 608 0 0.0
.got 4672 4672 0 0.0
.init 27 27 0 0.0
.init_array 904 904 0 0.0
.rodata 244680 244680 0 0.0
.text 2206866 2206882 16 0.0
ota-provider-app debug (read only) 2197433 2197449 16 0.0
(read/write) 106128 106128 0 0.0
.bss 47616 47616 0 0.0
.data 2080 2080 0 0.0
.data.rel.ro 51320 51320 0 0.0
.dynamic 608 608 0 0.0
.got 3736 3736 0 0.0
.init 27 27 0 0.0
.init_array 744 744 0 0.0
.rodata 196008 196008 0 0.0
.text 1861426 1861442 16 0.0
ota-requestor-app debug (read only) 2358897 2358913 16 0.0
(read/write) 113824 113824 0 0.0
.bss 50080 50080 0 0.0
.data 2448 2448 0 0.0
.data.rel.ro 56088 56088 0 0.0
.dynamic 608 608 0 0.0
.got 3736 3736 0 0.0
.init 27 27 0 0.0
.init_array 824 824 0 0.0
.rodata 202736 202736 0 0.0
.text 2007170 2007186 16 0.0
shell debug (read only) 2653961 2653977 16 0.0
(read/write) 138528 138528 0 0.0
.bss 58888 58888 0 0.0
.data 1264 1264 0 0.0
.data.rel.ro 72568 72568 0 0.0
.dynamic 608 608 0 0.0
.got 4136 4136 0 0.0
.init 27 27 0 0.0
.init_array 1040 1040 0 0.0
.rodata 240018 240018 0 0.0
.text 2254050 2254066 16 0.0
thermostat-no-ble arm64 (read only) 2479452 2479452 0 0.0
(read/write) 144040 144040 0 0.0
.bss 56360 56360 0 0.0
.data 1816 1816 0 0.0
.data.rel.ro 76632 76632 0 0.0
.dynamic 560 560 0 0.0
.got 5192 5192 0 0.0
.init 24 24 0 0.0
.init_array 432 432 0 0.0
.rodata 149832 149832 0 0.0
.text 2069024 2069024 0 0.0
tv-app debug (read only) 3288081 3288097 16 0.0
(read/write) 259248 259248 0 0.0
.bss 170776 170776 0 0.0
.data 3904 3904 0 0.0
.data.rel.ro 77856 77856 0 0.0
.dynamic 608 608 0 0.0
.got 5008 5008 0 0.0
.init 27 27 0 0.0
.init_array 1072 1072 0 0.0
.rodata 269208 269208 0 0.0
.text 2824578 2824594 16 0.0
tv-casting-app debug (read only) 5639393 5639409 16 0.0
(read/write) 161816 161816 0 0.0
.bss 53368 53368 0 0.0
.data 1936 1936 0 0.0
.data.rel.ro 99880 99880 0 0.0
.dynamic 608 608 0 0.0
.got 4928 4928 0 0.0
.init 27 27 0 0.0
.init_array 1048 1048 0 0.0
.rodata 361433 361433 0 0.0
.text 4999826 4999842 16 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2461952 2461952 0 0.0
.bss 215956 215956 0 0.0
.data 5872 5872 0 0.0
.text 1424596 1424596 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read only) 4 4 0 0.0
(read/write) 1163400 1163416 16 0.0
bss 147056 147056 0 0.0
rodata 132632 132632 0 0.0
text 803564 803576 12 0.0
nrf7002dk_nrf5340_cpuapp (read only) 4 4 0 0.0
(read/write) 1365104 1365120 16 0.0
bss 106356 106356 0 0.0
rodata 210116 210116 0 0.0
text 762800 762808 8 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read only) 4 4 0 0.0
(read/write) 1109240 1109256 16 0.0
bss 146259 146259 0 0.0
rodata 109912 109912 0 0.0
text 772960 772976 16 0.0
psoc6 all-clusters cy8ckit_062s2_43012 (read only) 841008 841008 0 0.0
(read/write) 1748420 1748420 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 189672 189672 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2664 2664 0 0.0
.debug_abbrev 1247001 1247001 0 0.0
.debug_aranges 111184 111184 0 0.0
.debug_frame 373384 373384 0 0.0
.debug_info 27324910 27324913 3 0.0
.debug_line 3753136 3753191 55 0.0
.debug_loc 3643491 3643535 44 0.0
.debug_ranges 356352 356504 152 0.0
.debug_str 3470157 3470157 0 0.0
.heap 841008 841008 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 575120 575120 0 0.0
.symtab 423200 423200 0 0.0
.text 0 0 0 0.0
1547696 1547696 0 0.0
.zero.table 8 8 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 (read only) 841776 841776 0 0.0
(read/write) 1689852 1689852 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 188912 188912 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2656 2656 0 0.0
.debug_abbrev 1232808 1232808 0 0.0
.debug_aranges 110448 110448 0 0.0
.debug_frame 375828 375828 0 0.0
.debug_info 26886483 26886485 2 0.0
.debug_line 3761933 3761988 55 0.0
.debug_loc 3626891 3626935 44 0.0
.debug_ranges 354320 354472 152 0.0
.debug_str 3454963 3454963 0 0.0
.heap 841776 841776 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 535500 535500 0 0.0
.symtab 408160 408160 0 0.0
.text 1489896 1489896 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
light cy8ckit_062s2_43012 (read only) 849952 849952 0 0.0
(read/write) 1609948 1609948 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 180936 180936 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2456 2456 0 0.0
.debug_abbrev 1067806 1067806 0 0.0
.debug_aranges 102736 102736 0 0.0
.debug_frame 346404 346404 0 0.0
.debug_info 22362394 22362398 4 0.0
.debug_line 3328440 3328495 55 0.0
.debug_loc 3328766 3328810 44 0.0
.debug_ranges 318544 318696 152 0.0
.debug_str 3260760 3260760 0 0.0
.heap 849952 849952 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 473441 473441 0 0.0
.symtab 377392 377392 0 0.0
.text 1418168 1418168 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
lock cy8ckit_062s2_43012 (read only) 844960 844960 0 0.0
(read/write) 1644012 1644012 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 185912 185912 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2472 2472 0 0.0
.debug_abbrev 1069852 1069852 0 0.0
.debug_aranges 103120 103120 0 0.0
.debug_frame 348204 348204 0 0.0
.debug_info 22580798 22580801 3 0.0
.debug_line 3329542 3329597 55 0.0
.debug_loc 3350279 3350323 44 0.0
.debug_ranges 320400 320552 152 0.0
.debug_str 3277167 3277167 0 0.0
.heap 844960 844960 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 476376 476376 0 0.0
.symtab 379216 379216 0 0.0
.text 1447240 1447240 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1147692 1147700 8 0.0
.bss 100668 100668 0 0.0
.data 840 840 0 0.0
.text 594792 594800 8 0.0
lock-app qpg6105+debug (read/write) 1115044 1115052 8 0.0
.bss 97188 97188 0 0.0
.data 852 852 0 0.0
.text 562140 562148 8 0.0
telink all-clusters-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 1025096 1025112 16 0.0
bss 98492 98492 0 0.0
text 687954 687966 12 0.0
all-clusters-minimal-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 963436 963444 8 0.0
bss 97680 97680 0 0.0
text 651244 651254 10 0.0
contact-sensor-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 868068 868076 8 0.0
bss 89844 89844 0 0.0
text 581742 581752 10 0.0
light-switch-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 884516 884524 8 0.0
bss 89940 89940 0 0.0
text 596344 596352 8 0.0
lighting-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 962860 962868 8 0.0
bss 97996 97996 0 0.0
text 662348 662356 8 0.0
ota-requestor-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 898332 898340 8 0.0
bss 90888 90888 0 0.0
text 607440 607448 8 0.0
thermostat tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 887740 887748 8 0.0
bss 91328 91328 0 0.0
text 597106 597114 8 0.0

@andy31415 andy31415 merged commit 164d7cd into project-chip:master Jan 23, 2023
@bzbarsky-apple bzbarsky-apple deleted the fix-linux-startup-crash branch January 23, 2023 16:23
lecndav pushed a commit to lecndav/connectedhomeip that referenced this pull request Mar 22, 2023
…but not as commissionable. (project-chip#24565)

In DnssdServer::Advertise we were trying to get things like a discriminator and
other commissionee information even if we were just advertising as a
commissioner (commissionableNode was false).  This could lead to crashes due to
calling things like GetCommissionableDataProvider() when we are not in fact
commissionable and don't have one.

The fix is to reorder things a bit so we set up all the advertiseParameters
state that both commissioners and commissionable share, and then condition all
the commissionable-only bits on commissionableNode.
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.

3 participants