Skip to content

[nrf fromlist] net: openthread: Fix ignored return values in OT utils #2738

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

Merged

Conversation

rlubos
Copy link
Contributor

@rlubos rlubos commented Apr 9, 2025

Pull more fixes for OT clang builds from upstream

rlubos added 2 commits April 9, 2025 14:27
Upstream PR #: 88359

Do not ignore return values of OT functions registering IPv6 addresses.
As this is called from the net_mgmt callback context, which returns
void, there's not much that can be done, other than logging the error.

To simplify the logging code and avoid the need to use extra buffer,
use net_sprint_ipv6_addr() networking utility function within the log
message to convert binary address to string.

Signed-off-by: Robert Lubos <[email protected]>
Upstream PR #: 88359

clear_pending_events() was not used anywhere.

Signed-off-by: Robert Lubos <[email protected]>
Copy link

sonarqubecloud bot commented Apr 9, 2025

@rlubos rlubos added the backport v4.0.99-ncs1-branch Relates to NCS v3.0-branch label Apr 9, 2025
@rlubos rlubos requested review from LuDuda, tejlmand and carlescufi April 9, 2025 13:04
@carlescufi carlescufi merged commit 8e6457e into nrfconnect:main Apr 9, 2025
26 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport v4.0.99-ncs1-branch Relates to NCS v3.0-branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants