Skip to content

nimble/host: fix compilation error with clang 20.1.1#125

Open
Throows wants to merge 1 commit into
espressif:nimble-1.6.0-idffrom
Throows:nimble-1.6.0-idf
Open

nimble/host: fix compilation error with clang 20.1.1#125
Throows wants to merge 1 commit into
espressif:nimble-1.6.0-idffrom
Throows:nimble-1.6.0-idf

Conversation

@Throows

@Throows Throows commented May 15, 2026

Copy link
Copy Markdown

Description

I have an compilation error with clang 20.1.1
image

I applied a fix, but as I don't know the working of nimble a review is asked 😅.
I just initialised a variable to NULL and added a null check before freeing the variable.

~Romain

Related

Testing


Checklist

Before submitting a Pull Request, please ensure the following:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

@CLAassistant

CLAassistant commented May 15, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants