alex/esp-idf
alex/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf.git synced 2024-10-05 20:47:46 -04:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

Merge branch 'bugfix/clang_tidy_and_nvs_memory_issue' into 'release/v5.0'

Browse Source 
clang tidy add C++ and nvs memory issue (backports 5.0)

See merge request espressif/esp-idf!22666
This commit is contained in:
Marius Vikhammer 2023-03-10 11:42:02 +08:00
commit 319208c3d1
3 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
.gitlab/ci/rules.yml
View File

@ -7,8 +7,8 @@
- "tools/ci/static-analysis-rules.yml"
- "tools/ci/clang_tidy_dirs.txt"
- "**/*.{c,C}"
- "**/*.{h,H}"
- "**/*.{c,C,cpp}"
- "**/*.{h,H,hpp}"
- "components/**/Kconfig"
- "components/**/CMakeList.txt"

4
components/nvs_flash/src/nvs_storage.cpp
View File

@ -112,12 +112,14 @@ esp_err_t Storage::init(uint32_t baseSector, uint32_t sectorCount)
item.getKey(entry->mName, sizeof(entry->mName));
err = item.getValue(entry->mIndex);
if (err != ESP_OK) {
delete entry;
return err;
}
mNamespaces.push_back(entry);
if (mNamespaceUsage.set(entry->mIndex, true) != ESP_OK) {
delete entry;
return ESP_FAIL;
}
mNamespaces.push_back(entry);
itemIndex += item.span;
}
}

3
tools/ci/static-analysis-rules.yml
View File

@ -38,4 +38,7 @@ skip:
- "components/bt/host/bluedroid/stack/btm/btm_dev.c"
- "components/bt/host/bluedroid/stack/gatt/att_protocol.c"
- "components/bt/host/bluedroid/stack/gatt/gatt_db.c"
- "components/bt/host/bluedroid/stack/l2cap/l2c_utils.c"
- "components/wifi_provisioning/src/scheme_ble.c"
- "components/wpa_supplicant/src/rsn_supp/wpa.c"
- "components/nvs_flash/src/nvs_storage.cpp"