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

BLEEndPoint ref count overflow check #16046

Merged
merged 2 commits into from
Mar 15, 2022

Conversation

lzgrablic02
Copy link
Contributor

@lzgrablic02 lzgrablic02 commented Mar 10, 2022

Problem

What is being fixed? Examples:

Change overview

Check for ref count over- and underflow.

Testing

How was this tested? (at least one bullet point required)

  • WiFi-commissioned node via BLE.

@github-actions
Copy link

github-actions bot commented Mar 10, 2022

PR #16046: Size comparison from 190520d to e8cd248

Decreases (1 build for esp32)
platform target config section 190520d e8cd248 change % change
esp32 all-clusters-app m5stack (read only) 1012887 1012883 -4 -0.0
.flash.text 1007503 1007499 -4 -0.0
Full report (26 builds for cyw30739, efr32, esp32, k32w, linux, mbed, p6, telink)
platform target config section 190520d e8cd248 change % change
cyw30739 light cyw930739m2evb_01 (read/write) 598058 598058 0 0.0
.app_xip_area 505640 505640 0 0.0
.bss 75108 75108 0 0.0
.data 652 652 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 555926 555926 0 0.0
.app_xip_area 465044 465044 0 0.0
.bss 73612 73612 0 0.0
.data 616 616 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor cyw930739m2evb_01 (read/write) 566030 566030 0 0.0
.app_xip_area 465444 465444 0 0.0
.bss 82996 82996 0 0.0
.data 552 552 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 919872 919872 0 0.0
(read/write) 128264 128264 0 0.0
.bss 126224 126224 0 0.0
.data 2040 2040 0 0.0
.text 919864 919864 0 0.0
BRD4161A+rpc (read only) 948700 948700 0 0.0
(read/write) 144220 144220 0 0.0
.bss 142000 142000 0 0.0
.data 2220 2220 0 0.0
.text 948692 948692 0 0.0
window-app BRD4161A (read only) 852348 852348 0 0.0
(read/write) 126220 126220 0 0.0
.bss 124304 124304 0 0.0
.data 1916 1916 0 0.0
.text 852340 852340 0 0.0
esp32 all-clusters-app c3devkit (read only) 957736 957736 0 0.0
(read/write) 1393778 1393778 0 0.0
.dram0.bss 63576 63576 0 0.0
.dram0.data 14220 14220 0 0.0
.flash.rodata 197064 197064 0 0.0
.flash.text 957736 957736 0 0.0
.iram0.text 62016 62016 0 0.0
m5stack (read only) 1012887 1012883 -4 -0.0
(read/write) 461028 461028 0 0.0
.dram0.bss 69096 69096 0 0.0
.dram0.data 34064 34064 0 0.0
.flash.rodata 226032 226032 0 0.0
.flash.text 1007503 1007499 -4 -0.0
.iram0.text 123107 123107 0 0.0
k32w light k32w061+release (read/write) 694492 694492 0 0.0
.bss 77160 77160 0 0.0
.data 1924 1924 0 0.0
.text 609608 609608 0 0.0
lock k32w061+release (read/write) 695292 695292 0 0.0
.bss 77072 77072 0 0.0
.data 1964 1964 0 0.0
.text 610456 610456 0 0.0
linux all-clusters-app debug (read only) 2430321 2430321 0 0.0
(read/write) 145872 145872 0 0.0
.bss 60352 60352 0 0.0
.data 1360 1360 0 0.0
.data.rel.ro 78424 78424 0 0.0
.dynamic 592 592 0 0.0
.got 4176 4176 0 0.0
.init 27 27 0 0.0
.init_array 936 936 0 0.0
.rodata 205637 205637 0 0.0
.text 2061698 2061698 0 0.0
bridge-app debug+rpc (read only) 1735493 1735493 0 0.0
(read/write) 92784 92784 0 0.0
.bss 47624 47624 0 0.0
.data 2016 2016 0 0.0
.data.rel.ro 38064 38064 0 0.0
.dynamic 592 592 0 0.0
.got 3928 3928 0 0.0
.init 27 27 0 0.0
.init_array 544 544 0 0.0
.rodata 141868 141868 0 0.0
.text 1477653 1477653 0 0.0
chip-tool debug (read only) 10093045 10093045 0 0.0
(read/write) 358168 358168 0 0.0
.bss 25640 25640 0 0.0
.data 1152 1152 0 0.0
.data.rel.ro 325352 325352 0 0.0
.dynamic 608 608 0 0.0
.got 4776 4776 0 0.0
.init 27 27 0 0.0
.init_array 616 616 0 0.0
.rodata 514196 514196 0 0.0
.text 8796165 8796165 0 0.0
chip-tool-ipv6only arm64 (read only) 9729052 9729052 0 0.0
(read/write) 475969 475969 0 0.0
.bss 43953 43953 0 0.0
.data 1200 1200 0 0.0
.data.rel.ro 371712 371712 0 0.0
.dynamic 560 560 0 0.0
.got 55304 55304 0 0.0
.init 24 24 0 0.0
.init_array 192 192 0 0.0
.rodata 491908 491908 0 0.0
.text 8185956 8185956 0 0.0
door-lock-app debug (read only) 1949953 1949953 0 0.0
(read/write) 119296 119296 0 0.0
.bss 50656 50656 0 0.0
.data 1056 1056 0 0.0
.data.rel.ro 62184 62184 0 0.0
.dynamic 592 592 0 0.0
.got 4112 4112 0 0.0
.init 27 27 0 0.0
.init_array 664 664 0 0.0
.rodata 173756 173756 0 0.0
.text 1627554 1627554 0 0.0
lighting-app debug+rpc (read only) 2082121 2082121 0 0.0
(read/write) 124616 124616 0 0.0
.bss 51456 51456 0 0.0
.data 1536 1536 0 0.0
.data.rel.ro 66104 66104 0 0.0
.dynamic 608 608 0 0.0
.got 4144 4144 0 0.0
.init 27 27 0 0.0
.init_array 736 736 0 0.0
.rodata 165948 165948 0 0.0
.text 1760514 1760514 0 0.0
ota-provider-app debug (read only) 1883193 1883193 0 0.0
(read/write) 114896 114896 0 0.0
.bss 50464 50464 0 0.0
.data 1320 1320 0 0.0
.data.rel.ro 57464 57464 0 0.0
.dynamic 608 608 0 0.0
.got 4368 4368 0 0.0
.init 27 27 0 0.0
.init_array 616 616 0 0.0
.rodata 158963 158963 0 0.0
.text 1573506 1573506 0 0.0
ota-requestor-app debug (read only) 1910801 1910801 0 0.0
(read/write) 117864 117864 0 0.0
.bss 51584 51584 0 0.0
.data 1480 1480 0 0.0
.data.rel.ro 59368 59368 0 0.0
.dynamic 592 592 0 0.0
.got 4160 4160 0 0.0
.init 27 27 0 0.0
.init_array 640 640 0 0.0
.rodata 154924 154924 0 0.0
.text 1605362 1605362 0 0.0
shell debug (read only) 2396513 2396513 0 0.0
(read/write) 150088 150088 0 0.0
.bss 70472 70472 0 0.0
.data 864 864 0 0.0
.data.rel.ro 73064 73064 0 0.0
.dynamic 592 592 0 0.0
.got 4160 4160 0 0.0
.init 27 27 0 0.0
.init_array 920 920 0 0.0
.rodata 205586 205586 0 0.0
.text 2034354 2034354 0 0.0
thermostat-no-ble arm64 (read only) 2178620 2178620 0 0.0
(read/write) 148817 148817 0 0.0
.bss 65393 65393 0 0.0
.data 1048 1048 0 0.0
.data.rel.ro 75160 75160 0 0.0
.dynamic 560 560 0 0.0
.got 4240 4240 0 0.0
.init 24 24 0 0.0
.init_array 352 352 0 0.0
.rodata 133116 133116 0 0.0
.text 1826912 1826912 0 0.0
tv-app debug (read only) 2622009 2622009 0 0.0
(read/write) 250360 250360 0 0.0
.bss 167520 167520 0 0.0
.data 3168 3168 0 0.0
.data.rel.ro 73624 73624 0 0.0
.dynamic 592 592 0 0.0
.got 4544 4544 0 0.0
.init 27 27 0 0.0
.init_array 880 880 0 0.0
.rodata 199293 199293 0 0.0
.text 2248626 2248626 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2351356 2351356 0 0.0
.bss 186212 186212 0 0.0
.data 5792 5792 0 0.0
.text 1313956 1313956 0 0.0
p6 all-clusters-app default (read/write) 2494216 2494216 0 0.0
.bss 119640 119640 0 0.0
.data 2680 2680 0 0.0
.text 1452480 1452480 0 0.0
light-app default (read/write) 2398296 2398296 0 0.0
.bss 113104 113104 0 0.0
.data 2536 2536 0 0.0
.text 1356560 1356560 0 0.0
lock-app default (read/write) 2361856 2361856 0 0.0
.bss 112848 112848 0 0.0
.data 2496 2496 0 0.0
.text 1320120 1320120 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 888882 888882 0 0.0
bss 86968 86968 0 0.0
noinit 37160 37160 0 0.0
text 628184 628184 0 0.0

@bzbarsky-apple bzbarsky-apple merged commit ca52c2e into project-chip:master Mar 15, 2022
@lzgrablic02 lzgrablic02 deleted the bleep branch March 15, 2022 20:52
andrei-menzopol pushed a commit to andrei-menzopol/connectedhomeip that referenced this pull request Apr 14, 2022
* BLEEndPoint ref count overflow check

* Assert preconditions for BLEEndPoint ref counting
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.

BLEEndPoint possible refcount overflow
4 participants