Skip to content

Coverity/v1#14839

Closed
victorjulien wants to merge 2 commits intoOISF:mainfrom
victorjulien:coverity/v1
Closed

Coverity/v1#14839
victorjulien wants to merge 2 commits intoOISF:mainfrom
victorjulien:coverity/v1

Conversation

@victorjulien
Copy link
Member

2 coverity warning fixes

CID 1667318: (#1 of 1): Structurally dead code (UNREACHABLE)
unreachable: This code cannot be reached: hwloc_obj_t non_io_ancestor....
CID 1638288: (#1 of 1): Missing unlock (LOCK)
4. missing_unlock: Returning without unlocking xsk_protect.queue_protect.
Copy link
Contributor

@jufajardini jufajardini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

Copy link
Member

@inashivb inashivb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@victorjulien victorjulien added this to the 9.0 milestone Feb 19, 2026
@suricata-qa
Copy link

Information: QA ran without warnings.

Pipeline = 29775

@victorjulien
Copy link
Member Author

Merged in #14856, thanks!

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants