Skip to content

libsecutils CONN_is_IP_address(): prevent crash freeing uninitialized 'res' on unsuccessful getaddrinfo() call#107

Merged
DDvO merged 1 commit intomasterfrom
fix_crash_CONN_is_IP_address
Dec 19, 2025
Merged

libsecutils CONN_is_IP_address(): prevent crash freeing uninitialized 'res' on unsuccessful getaddrinfo() call#107
DDvO merged 1 commit intomasterfrom
fix_crash_CONN_is_IP_address

Conversation

@DDvO
Copy link
Member

@DDvO DDvO commented Dec 19, 2025

This takes over the fix from siemens/libsecutils#69 to genCMPClient.

@DDvO DDvO requested review from Copilot and rajeev-0 December 19, 2025 11:29
@DDvO DDvO added the bug Something isn't working label Dec 19, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the libsecutils submodule to incorporate a critical bug fix that prevents a crash in CONN_is_IP_address() when getaddrinfo() fails, by ensuring the res pointer is properly initialized before attempting to free it.

Key Changes:


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@DDvO DDvO force-pushed the fix_crash_CONN_is_IP_address branch from 877003f to 4e391ef Compare December 19, 2025 11:34
@sonarqubecloud
Copy link

Copy link
Collaborator

@rajeev-0 rajeev-0 left a comment

Choose a reason for hiding this comment

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

LGTM

@DDvO DDvO merged commit f7902e3 into master Dec 19, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants