Skip to content

fix(zigbeeEP): review of names and memory allocation #11199

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

Merged
merged 17 commits into from
Apr 5, 2025
Merged

fix(zigbeeEP): review of names and memory allocation #11199

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
d3a44fc
fix(zigbeeEP): review of names and memory allocation
SuGlider Mar 28, 2025
7ad636f
fix(zigbeeEP): destructor shall free any allocated memory
SuGlider Mar 28, 2025
2c44063
fix(zigbee_ep): forgotten var name change
SuGlider Mar 28, 2025
83d0f42
Merge branch 'master' into zigbee_memory_review
SuGlider Mar 29, 2025
49d983a
feat(zigbee_ep): use static heap memory for manufacturer and model names
SuGlider Mar 30, 2025
bf3e5c1
feat(zigbee_ep): changed model and manufacturer to heap
SuGlider Mar 30, 2025
6c6b980
feat(zigbee_ep): use static heap memory allocation
SuGlider Mar 30, 2025
c15fbf3
fix(zigbee_ep): using stack only for adding attribute
SuGlider Mar 31, 2025
b6cb584
Merge branch 'master' into zigbee_memory_review
SuGlider Mar 31, 2025
0763c2a
feat(zigbee_ep): reverting back read data type
SuGlider Mar 31, 2025
cfe4457
fix(zigbee_ep): rooling back to use malloc for remote attr reading
SuGlider Mar 31, 2025
a7c7713
feat(zigbee_ep): check malloc return for null
SuGlider Mar 31, 2025
48e5f17
fix(zigbee_ep): replace nullptr by NULL after C malloc()
SuGlider Mar 31, 2025
828d911
Merge branch 'master' into zigbee_memory_review
P-R-O-C-H-Y Apr 1, 2025
4681be4
ci(pre-commit): Apply automatic fixes
pre-commit-ci-lite[bot] Apr 2, 2025
d113ba2
fix(zigbee_ep): fix variable scope
SuGlider Apr 3, 2025
8e03dd0
fix(zigbee_ep): fix variable scope
SuGlider Apr 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix(zigbeeEP): destructor shall free any allocated memory
  • Loading branch information
@SuGlider
SuGlider authored Mar 28, 2025
commit 7ad636f33cf420bc86c43c53dfbda8d4e272f962
5 changes: 4 additions & 1 deletion libraries/Zigbee/src/ZigbeeEP.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,10 @@ typedef enum {
class ZigbeeEP {
public:
ZigbeeEP(uint8_t endpoint = 10);
~ZigbeeEP() {}
~ZigbeeEP() {
free(_read_manufacturer);
free(_read_model);
}

// Set ep config and cluster list
void setEpConfig(esp_zb_endpoint_config_t ep_config, esp_zb_cluster_list_t *cluster_list) {
Expand Down
Loading