Skip to content

Fix: get_tcp_option() and get_tcp_element()#2261

Open
jjnicola wants to merge 2 commits into
mainfrom
fix-tcp-option
Open

Fix: get_tcp_option() and get_tcp_element()#2261
jjnicola wants to merge 2 commits into
mainfrom
fix-tcp-option

Conversation

@jjnicola

@jjnicola jjnicola commented Jun 24, 2026

Copy link
Copy Markdown
Member

Avoid infinit loop and OOB.
Adds a buffer size check before memcpy()'ing
Other minor bugs and code improvement (remove dead code)

Jira: SC-1639
Jira: SC-1640

@jjnicola jjnicola requested a review from a team as a code owner June 24, 2026 15:57
@github-actions

github-actions Bot commented Jun 24, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 7bcc1e8.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@github-actions github-actions Bot added the patch_release creates a patch release label Jun 24, 2026
@github-actions github-actions Bot added patch_release creates a patch release and removed patch_release creates a patch release labels Jun 24, 2026
avoid infinit loop and OOB. Also, adds a buffer size check before memcpy()'ing
@github-actions github-actions Bot added patch_release creates a patch release and removed patch_release creates a patch release labels Jun 24, 2026
@jjnicola jjnicola enabled auto-merge (squash) June 24, 2026 17:45
@github-actions github-actions Bot added patch_release creates a patch release and removed patch_release creates a patch release labels Jun 24, 2026
@jjnicola jjnicola changed the title Fix: get_tcp_option() Fix: get_tcp_option() and get_tcp_element() Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

patch_release creates a patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant