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.

Sign up for GitHub

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

Jump to bottom

[privacy] Don't enable privacy when privacy is not supported. #22425

Merged
merged 1 commit into from
Sep 7, 2022
Merged

[privacy] Don't enable privacy when privacy is not supported. #22425

merged 1 commit into from
Sep 7, 2022

Conversation

turon
Copy link
Contributor

@turon turon commented Sep 6, 2022 •
edited
Loading

Problem

The SDK is setting the privacy flag from Group and MCSP messages even though support for privacy isn't enabled yet.
Fix #22426

Change overview

Correctly set privacy flag to reflect whether privacy is actually enabled on a message.

Testing

CI

Interoperability

This change will break interoperability of group messages between 1.0 and SVE2 + prior.
SVE2 + prior are erroneously sending group messages with privacy flag set but not applying any privacy. This behavior does not match the spec.

@github-actions
Copy link

github-actions bot commented Sep 6, 2022 •
edited
Loading

PR #22425: Size comparison from e535710 to 59d32d4

Increases (4 builds for cc13x2_26x2, qpg)
platform target config section e535710 59d32d4 change % change
cc13x2_26x2 lock-mtd LP_CC2652R7 (read/write) 178028 178036 8 0.0
pump-app LP_CC2652R7 (read/write) 157176 157184 8 0.0
pump-controller-app LP_CC2652R7 (read only) 669723 669731 8 0.0
.text 583616 583624 8 0.0
qpg lighting-app qpg6105+debug (read/write) 1129156 1129164 8 0.0
.text 576252 576260 8 0.0
Decreases (34 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, linux, nrfconnect, psoc6, telink)
platform target config section e535710 59d32d4 change % change
bl602 lighting-app bl602 (read/write) 1410254 1410198 -56 -0.0
.text 1065684 1065626 -58 -0.0
bl602+rpc (read/write) 1455450 1455394 -56 -0.0
.text 1097032 1096976 -56 -0.0
cc13x2_26x2 lock-mtd LP_CC2652R7 (read only) 659379 659371 -8 -0.0
.text 556576 556568 -8 -0.0
pump-app LP_CC2652R7 (read only) 685247 685239 -8 -0.0
.text 594684 594676 -8 -0.0
pump-controller-app LP_CC2652R7 (read/write) 172812 172804 -8 -0.0
cyw30739 lock cyw930739m2evb_01 (read/write) 592634 592626 -8 -0.0
.app_xip_area 464508 464500 -8 -0.0
efr32 lighting-app BRD4161A (read/write) 1108544 1108480 -64 -0.0
.text 970120 970056 -64 -0.0
BRD4161A+rpc (read/write) 972188 972172 -16 -0.0
.text 819072 819056 -16 -0.0
BRD4161A+rs911x (read/write) 1002048 1002000 -48 -0.0
.text 830796 830748 -48 -0.0
lock-app BRD4161A+wf200 (read/write) 1150116 1150100 -16 -0.0
.text 995776 995760 -16 -0.0
window-app BRD4161A (read/write) 1099784 1099720 -64 -0.0
.text 959896 959832 -64 -0.0
esp32 all-clusters-app c3devkit (read only) 1033856 1033842 -14 -0.0
.flash.text 1033856 1033842 -14 -0.0
m5stack (read only) 1086195 1086187 -8 -0.0
.flash.text 1080811 1080803 -8 -0.0
linux all-clusters-app debug (read only) 3046409 3046281 -128 -0.0
.text 2591314 2591186 -128 -0.0
all-clusters-minimal-app debug (read only) 2882273 2882129 -144 -0.0
.text 2429730 2429586 -144 -0.0
bridge-app debug+rpc (read only) 2379977 2379833 -144 -0.0
.text 2012578 2012434 -144 -0.0
chip-tool debug (read only) 10945849 10945721 -128 -0.0
.text 8855732 8855604 -128 -0.0
chip-tool-ipv6only arm64 (read only) 10320484 10320340 -144 -0.0
.text 8168820 8168676 -144 -0.0
lighting-app debug+rpc (read only) 2605401 2605273 -128 -0.0
.text 2212882 2212754 -128 -0.0
lock-app debug (read only) 2588385 2588241 -144 -0.0
.text 2183122 2182978 -144 -0.0
ota-provider-app debug (read only) 2365657 2365513 -144 -0.0
.text 1991570 1991426 -144 -0.0
ota-requestor-app debug (read only) 2530969 2530841 -128 -0.0
.text 2140946 2140818 -128 -0.0
shell debug (read only) 2614713 2614569 -144 -0.0
.text 2220242 2220098 -144 -0.0
thermostat-no-ble arm64 (read only) 2364132 2363988 -144 -0.0
.text 1984768 1984624 -144 -0.0
tv-app debug (read only) 3192969 3192825 -144 -0.0
.text 2742610 2742466 -144 -0.0
tv-casting-app debug (read only) 5511313 5511169 -144 -0.0
.text 4894018 4893874 -144 -0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 text 815072 815064 -8 -0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1160395 1160379 -16 -0.0
text 803460 803456 -4 -0.0
psoc6 all-clusters cy8ckit_062s2_43012 (read/write) 1742548 1742500 -48 -0.0
.debug_info 26715430 26715314 -116 -0.0
.debug_line 3657437 3657362 -75 -0.0
.debug_loc 3573305 3573219 -86 -0.0
.debug_ranges 338440 338360 -80 -0.0
.text 1542776 1542728 -48 -0.0
all-clusters-minimal cy8ckit_062s2_43012 (read/write) 1685748 1685700 -48 -0.0
.debug_info 26452052 26451937 -115 -0.0
.debug_line 3677953 3677878 -75 -0.0
.debug_loc 3560942 3560856 -86 -0.0
.debug_ranges 337056 336976 -80 -0.0
.text 1486712 1486664 -48 -0.0
light cy8ckit_062s2_43012 (read/write) 1603020 1602972 -48 -0.0
.debug_info 21914335 21914218 -117 -0.0
.debug_line 3248418 3248343 -75 -0.0
.debug_loc 3259284 3259198 -86 -0.0
.debug_ranges 302528 302448 -80 -0.0
.text 1412216 1412168 -48 -0.0
lock cy8ckit_062s2_43012 (read/write) 1640708 1640660 -48 -0.0
.debug_info 22294083 22293965 -118 -0.0
.debug_line 3257239 3257164 -75 -0.0
.debug_loc 3299137 3299051 -86 -0.0
.debug_ranges 305872 305792 -80 -0.0
.text 1444872 1444824 -48 -0.0
telink light-switch-app tlsr9518adk80d (read/write) 809096 809080 -16 -0.0
text 571518 571502 -16 -0.0
lighting-app tlsr9518adk80d (read/write) 831004 830988 -16 -0.0
text 589642 589626 -16 -0.0
Full report (45 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
platform target config section e535710 59d32d4 change % change
bl602 lighting-app bl602 (read/write) 1410254 1410198 -56 -0.0
.bss 121585 121585 0 0.0
.data 9816 9816 0 0.0
.text 1065684 1065626 -58 -0.0
bl602+rpc (read/write) 1455450 1455394 -56 -0.0
.bss 129017 129017 0 0.0
.data 10200 10200 0 0.0
.text 1097032 1096976 -56 -0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 675359 675359 0 0.0
(read/write) 176048 176048 0 0.0
.bss 74300 74300 0 0.0
.data 3380 3380 0 0.0
.rodata 89215 89215 0 0.0
.text 585832 585832 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 640071 640071 0 0.0
(read/write) 157868 157868 0 0.0
.bss 73572 73572 0 0.0
.data 3380 3380 0 0.0
.rodata 78367 78367 0 0.0
.text 561384 561384 0 0.0
lock-ftd LP_CC2652R7 (read only) 676419 676419 0 0.0
(read/write) 165300 165300 0 0.0
.bss 71500 71500 0 0.0
.data 3304 3304 0 0.0
.rodata 77067 77067 0 0.0
.text 598872 598872 0 0.0
lock-mtd LP_CC2652R7 (read only) 659379 659371 -8 -0.0
(read/write) 178028 178036 8 0.0
.bss 67188 67188 0 0.0
.data 3304 3304 0 0.0
.rodata 102323 102323 0 0.0
.text 556576 556568 -8 -0.0
pump-app LP_CC2652R7 (read only) 685247 685239 -8 -0.0
(read/write) 157176 157184 8 0.0
.bss 71436 71436 0 0.0
.data 3296 3296 0 0.0
.rodata 90079 90079 0 0.0
.text 594684 594676 -8 -0.0
pump-controller-app LP_CC2652R7 (read only) 669723 669731 8 0.0
(read/write) 172812 172804 -8 -0.0
.bss 71548 71548 0 0.0
.data 3292 3292 0 0.0
.rodata 85627 85627 0 0.0
.text 583616 583624 8 0.0
shell LP_CC2652R7 (read only) 666010 666010 0 0.0
(read/write) 180916 180916 0 0.0
.bss 76620 76620 0 0.0
.data 3376 3376 0 0.0
.rodata 85770 85770 0 0.0
.text 579924 579924 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 586882 586882 0 0.0
.app_xip_area 463540 463540 0 0.0
.bss 65776 65776 0 0.0
.data 744 744 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 592634 592626 -8 -0.0
.app_xip_area 464508 464500 -8 -0.0
.bss 70560 70560 0 0.0
.data 748 748 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 599810 599810 0 0.0
.app_xip_area 477188 477188 0 0.0
.bss 65088 65088 0 0.0
.data 716 716 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read/write) 1108544 1108480 -64 -0.0
.bss 136332 136332 0 0.0
.data 2072 2072 0 0.0
.text 970120 970056 -64 -0.0
BRD4161A+rpc (read/write) 972188 972172 -16 -0.0
.bss 150844 150844 0 0.0
.data 2252 2252 0 0.0
.text 819072 819056 -16 -0.0
BRD4161A+rs911x (read/write) 1002048 1002000 -48 -0.0
.bss 169168 169168 0 0.0
.data 2064 2064 0 0.0
.text 830796 830748 -48 -0.0
lock-app BRD4161A+wf200 (read/write) 1150116 1150100 -16 -0.0
.bss 152248 152248 0 0.0
.data 2072 2072 0 0.0
.text 995776 995760 -16 -0.0
window-app BRD4161A (read/write) 1099784 1099720 -64 -0.0
.bss 137772 137772 0 0.0
.data 2096 2096 0 0.0
.text 959896 959832 -64 -0.0
esp32 all-clusters-app c3devkit (read only) 1033856 1033842 -14 -0.0
(read/write) 1493494 1493494 0 0.0
.dram0.bss 71120 71120 0 0.0
.dram0.data 13696 13696 0 0.0
.flash.rodata 218008 218008 0 0.0
.flash.text 1033856 1033842 -14 -0.0
.iram0.text 65204 65204 0 0.0
m5stack (read only) 1086195 1086187 -8 -0.0
(read/write) 490772 490772 0 0.0
.dram0.bss 76640 76640 0 0.0
.dram0.data 34152 34152 0 0.0
.flash.rodata 247312 247312 0 0.0
.flash.text 1080811 1080803 -8 -0.0
.iram0.text 123939 123939 0 0.0
k32w light k32w0+release (read/write) 648204 648204 0 0.0
.bss 70712 70712 0 0.0
.data 2068 2068 0 0.0
.text 572696 572696 0 0.0
lock k32w0+release (read/write) 705216 705216 0 0.0
.bss 71160 71160 0 0.0
.data 2076 2076 0 0.0
.text 629252 629252 0 0.0
linux all-clusters-app debug (read only) 3046409 3046281 -128 -0.0
(read/write) 156064 156064 0 0.0
.bss 61824 61824 0 0.0
.data 2096 2096 0 0.0
.data.rel.ro 85768 85768 0 0.0
.dynamic 608 608 0 0.0
.got 4568 4568 0 0.0
.init 27 27 0 0.0
.init_array 1176 1176 0 0.0
.rodata 275659 275659 0 0.0
.text 2591314 2591186 -128 -0.0
all-clusters-minimal-app debug (read only) 2882273 2882129 -144 -0.0
(read/write) 147664 147664 0 0.0
.bss 61056 61056 0 0.0
.data 2064 2064 0 0.0
.data.rel.ro 78264 78264 0 0.0
.dynamic 608 608 0 0.0
.got 4488 4488 0 0.0
.init 27 27 0 0.0
.init_array 1160 1160 0 0.0
.rodata 275883 275883 0 0.0
.text 2429730 2429586 -144 -0.0
bridge-app debug+rpc (read only) 2379977 2379833 -144 -0.0
(read/write) 127752 127752 0 0.0
.bss 50656 50656 0 0.0
.data 3600 3600 0 0.0
.data.rel.ro 67640 67640 0 0.0
.dynamic 608 608 0 0.0
.got 4392 4392 0 0.0
.init 27 27 0 0.0
.init_array 832 832 0 0.0
.rodata 204488 204488 0 0.0
.text 2012578 2012434 -144 -0.0
chip-tool debug (read only) 10945849 10945721 -128 -0.0
(read/write) 657352 657352 0 0.0
.bss 25272 25272 0 0.0
.data 3266 3266 0 0.0
.data.rel.ro 622288 622288 0 0.0
.dynamic 608 608 0 0.0
.got 5096 5096 0 0.0
.init 27 27 0 0.0
.init_array 776 776 0 0.0
.rodata 569301 569301 0 0.0
.text 8855732 8855604 -128 -0.0
chip-tool-ipv6only arm64 (read only) 10320484 10320340 -144 -0.0
(read/write) 705185 705185 0 0.0
.bss 33313 33313 0 0.0
.data 3280 3280 0 0.0
.data.rel.ro 649784 649784 0 0.0
.dynamic 560 560 0 0.0
.got 13840 13840 0 0.0
.init 24 24 0 0.0
.init_array 200 200 0 0.0
.rodata 499252 499252 0 0.0
.text 8168820 8168676 -144 -0.0
lighting-app debug+rpc (read only) 2605401 2605273 -128 -0.0
(read/write) 130536 130536 0 0.0
.bss 49792 49792 0 0.0
.data 2096 2096 0 0.0
.data.rel.ro 72680 72680 0 0.0
.dynamic 608 608 0 0.0
.got 4392 4392 0 0.0
.init 27 27 0 0.0
.init_array 928 928 0 0.0
.rodata 221328 221328 0 0.0
.text 2212882 2212754 -128 -0.0
lock-app debug (read only) 2588385 2588241 -144 -0.0
(read/write) 125712 125712 0 0.0
.bss 48288 48288 0 0.0
.data 1712 1712 0 0.0
.data.rel.ro 69688 69688 0 0.0
.dynamic 608 608 0 0.0
.got 4464 4464 0 0.0
.init 27 27 0 0.0
.init_array 904 904 0 0.0
.rodata 238320 238320 0 0.0
.text 2183122 2182978 -144 -0.0
ota-provider-app debug (read only) 2365657 2365513 -144 -0.0
(read/write) 119144 119144 0 0.0
.bss 47808 47808 0 0.0
.data 1936 1936 0 0.0
.data.rel.ro 63512 63512 0 0.0
.dynamic 608 608 0 0.0
.got 4488 4488 0 0.0
.init 27 27 0 0.0
.init_array 768 768 0 0.0
.rodata 210328 210328 0 0.0
.text 1991570 1991426 -144 -0.0
ota-requestor-app debug (read only) 2530969 2530841 -128 -0.0
(read/write) 127552 127552 0 0.0
.bss 50368 50368 0 0.0
.data 2304 2304 0 0.0
.data.rel.ro 68920 68920 0 0.0
.dynamic 608 608 0 0.0
.got 4480 4480 0 0.0
.init 27 27 0 0.0
.init_array 856 856 0 0.0
.rodata 217120 217120 0 0.0
.text 2140946 2140818 -128 -0.0
shell debug (read only) 2614713 2614569 -144 -0.0
(read/write) 142184 142184 0 0.0
.bss 57704 57704 0 0.0
.data 1264 1264 0 0.0
.data.rel.ro 77376 77376 0 0.0
.dynamic 608 608 0 0.0
.got 4136 4136 0 0.0
.init 27 27 0 0.0
.init_array 1048 1048 0 0.0
.rodata 235762 235762 0 0.0
.text 2220242 2220098 -144 -0.0
thermostat-no-ble arm64 (read only) 2364132 2363988 -144 -0.0
(read/write) 141857 141857 0 0.0
.bss 55233 55233 0 0.0
.data 1680 1680 0 0.0
.data.rel.ro 76112 76112 0 0.0
.dynamic 560 560 0 0.0
.got 5056 5056 0 0.0
.init 24 24 0 0.0
.init_array 416 416 0 0.0
.rodata 141324 141324 0 0.0
.text 1984768 1984624 -144 -0.0
tv-app debug (read only) 3192969 3192825 -144 -0.0
(read/write) 258040 258040 0 0.0
.bss 167352 167352 0 0.0
.data 4752 4752 0 0.0
.data.rel.ro 79368 79368 0 0.0
.dynamic 608 608 0 0.0
.got 4856 4856 0 0.0
.init 27 27 0 0.0
.init_array 1080 1080 0 0.0
.rodata 260328 260328 0 0.0
.text 2742610 2742466 -144 -0.0
tv-casting-app debug (read only) 5511313 5511169 -144 -0.0
(read/write) 160536 160536 0 0.0
.bss 51352 51352 0 0.0
.data 2432 2432 0 0.0
.data.rel.ro 100304 100304 0 0.0
.dynamic 608 608 0 0.0
.got 4776 4776 0 0.0
.init 27 27 0 0.0
.init_array 1048 1048 0 0.0
.rodata 345393 345393 0 0.0
.text 4894018 4893874 -144 -0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2455032 2455032 0 0.0
.bss 215044 215044 0 0.0
.data 5872 5872 0 0.0
.text 1417676 1417676 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1181199 1181199 0 0.0
bss 143641 143641 0 0.0
rodata 143552 143552 0 0.0
text 815072 815064 -8 -0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1160395 1160379 -16 -0.0
bss 142868 142868 0 0.0
rodata 135140 135140 0 0.0
text 803460 803456 -4 -0.0
psoc6 all-clusters cy8ckit_062s2_43012 (read only) 841960 841960 0 0.0
(read/write) 1742548 1742500 -48 -0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 188720 188720 0 0.0
.comment 204 204 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 1221471 1221471 0 0.0
.debug_aranges 111728 111728 0 0.0
.debug_frame 372964 372964 0 0.0
.debug_info 26715430 26715314 -116 -0.0
.debug_line 3657437 3657362 -75 -0.0
.debug_loc 3573305 3573219 -86 -0.0
.debug_ranges 338440 338360 -80 -0.0
.debug_str 3427396 3427396 0 0.0
.heap 841960 841960 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 570587 570587 0 0.0
.symtab 421456 421456 0 0.0
.text 1542776 1542728 -48 -0.0
.zero.table 8 8 0 0.0
text 0 0 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 (read only) 842696 842696 0 0.0
(read/write) 1685748 1685700 -48 -0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 187984 187984 0 0.0
.comment 204 204 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 1213310 1213310 0 0.0
.debug_aranges 111200 111200 0 0.0
.debug_frame 376044 376044 0 0.0
.debug_info 26452052 26451937 -115 -0.0
.debug_line 3677953 3677878 -75 -0.0
.debug_loc 3560942 3560856 -86 -0.0
.debug_ranges 337056 336976 -80 -0.0
.debug_str 3416401 3416401 0 0.0
.heap 842696 842696 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 535061 535061 0 0.0
.symtab 408048 408048 0 0.0
.text 1486712 1486664 -48 -0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
light cy8ckit_062s2_43012 (read only) 850928 850928 0 0.0
(read/write) 1603020 1602972 -48 -0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 179960 179960 0 0.0
.comment 204 204 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 1048129 1048129 0 0.0
.debug_aranges 103376 103376 0 0.0
.debug_frame 346312 346312 0 0.0
.debug_info 21914335 21914218 -117 -0.0
.debug_line 3248418 3248343 -75 -0.0
.debug_loc 3259284 3259198 -86 -0.0
.debug_ranges 302528 302448 -80 -0.0
.debug_str 3221667 3221667 0 0.0
.heap 850928 850928 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 468360 468360 0 0.0
.symtab 375136 375136 0 0.0
.text 1412216 1412168 -48 -0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
lock cy8ckit_062s2_43012 (read only) 845896 845896 0 0.0
(read/write) 1640708 1640660 -48 -0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 184976 184976 0 0.0
.comment 204 204 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 1055564 1055564 0 0.0
.debug_aranges 104048 104048 0 0.0
.debug_frame 349140 349140 0 0.0
.debug_info 22294083 22293965 -118 -0.0
.debug_line 3257239 3257164 -75 -0.0
.debug_loc 3299137 3299051 -86 -0.0
.debug_ranges 305872 305792 -80 -0.0
.debug_str 3249088 3249088 0 0.0
.heap 845896 845896 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 474563 474563 0 0.0
.symtab 378320 378320 0 0.0
.text 1444872 1444824 -48 -0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1129156 1129164 8 0.0
.bss 106112 106112 0 0.0
.data 1028 1028 0 0.0
.text 576252 576260 8 0.0
lock-app qpg6105+debug (read/write) 1100176 1100176 0 0.0
.bss 102344 102344 0 0.0
.data 1032 1032 0 0.0
.text 547276 547276 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 809096 809080 -16 -0.0
bss 71344 71344 0 0.0
noinit 43488 43488 0 0.0
text 571518 571502 -16 -0.0
lighting-app tlsr9518adk80d (read/write) 831004 830988 -16 -0.0
bss 72200 72200 0 0.0
noinit 43488 43488 0 0.0
text 589642 589626 -16 -0.0

@turon turon marked this pull request as ready for review September 6, 2022 21:19
@turon turon added the spec Mismatch between spec and implementation label Sep 6, 2022
@andy31415
Copy link
Contributor

Accept for v1 master: spec alignment, fix invalid flag. If invalid flags start being used, it would be very hard to maintain compatibility.

@andy31415 andy31415 merged commit f8499f4 into project-chip:master Sep 7, 2022
github-actions bot pushed a commit that referenced this pull request Sep 7, 2022
@turon @web-flow
[privacy] Don't enable privacy when privacy is not supported. (#22425)
1a5834b
andy31415 added a commit that referenced this pull request Sep 8, 2022
@woody-apple @turon @andy31415
[privacy] Don't enable privacy when privacy is not supported. (#22425) (
45dde43 
#22452)

Co-authored-by: Martin Turon <mturon@google.com>
Co-authored-by: Andrei Litvin <andy314@gmail.com>
isiu-apple pushed a commit to isiu-apple/connectedhomeip that referenced this pull request Sep 16, 2022
@turon @isiu-apple
[privacy] Don't enable privacy when privacy is not supported. (projec…
575cf89 
…t-chip#22425)
This was referenced Oct 13, 2022
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmartinez-silabs jmartinez-silabs jmartinez-silabs approved these changes

@andy31415 andy31415 andy31415 approved these changes

@anush-apple anush-apple Awaiting requested review from anush-apple

@arkq arkq Awaiting requested review from arkq

@Byungjoo-Lee Byungjoo-Lee Awaiting requested review from Byungjoo-Lee

@bzbarsky-apple bzbarsky-apple Awaiting requested review from bzbarsky-apple

@carol-apple carol-apple Awaiting requested review from carol-apple

@chrisdecenzo chrisdecenzo Awaiting requested review from chrisdecenzo

@chshu chshu Awaiting requested review from chshu

@chulspro chulspro Awaiting requested review from chulspro

@Damian-Nordic Damian-Nordic Awaiting requested review from Damian-Nordic

@dhrishi dhrishi Awaiting requested review from dhrishi

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

@emargolis emargolis Awaiting requested review from emargolis

@franck-apple franck-apple Awaiting requested review from franck-apple

@gjc13 gjc13 Awaiting requested review from gjc13

@harsha-rajendran harsha-rajendran Awaiting requested review from harsha-rajendran

@hawk248 hawk248 Awaiting requested review from hawk248

@isiu-apple isiu-apple Awaiting requested review from isiu-apple

@jelderton jelderton Awaiting requested review from jelderton

@jepenven-silabs jepenven-silabs Awaiting requested review from jepenven-silabs

@jtung-apple jtung-apple Awaiting requested review from jtung-apple

@kghost kghost Awaiting requested review from kghost

@kpschoedel kpschoedel Awaiting requested review from kpschoedel

@lazarkov lazarkov Awaiting requested review from lazarkov

@LuDuda LuDuda Awaiting requested review from LuDuda

@mrjerryjohns mrjerryjohns Awaiting requested review from mrjerryjohns

@msandstedt msandstedt Awaiting requested review from msandstedt

@mspang mspang Awaiting requested review from mspang

@rgoliver rgoliver Awaiting requested review from rgoliver

@robszewczyk robszewczyk Awaiting requested review from robszewczyk

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@tcarmelveilleux tcarmelveilleux Awaiting requested review from tcarmelveilleux

@tecimovic tecimovic Awaiting requested review from tecimovic

@tehampson tehampson Awaiting requested review from tehampson

@vijs vijs Awaiting requested review from vijs

@vivien-apple vivien-apple Awaiting requested review from vivien-apple

@wbschiller wbschiller Awaiting requested review from wbschiller

@woody-apple woody-apple Awaiting requested review from woody-apple

@xylophone21 xylophone21 Awaiting requested review from xylophone21

@yunhanw-google yunhanw-google Awaiting requested review from yunhanw-google

Assignees
No one assigned
Labels
review - approved spec Mismatch between spec and implementation transport
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[1.0] [privacy] Messages should not enable P flag if privacy is not applied
3 participants
@turon @andy31415 @jmartinez-silabs