Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: zero-out URI in case of parse ERROR #3557

Closed
wants to merge 2 commits into from
Closed

fix: zero-out URI in case of parse ERROR #3557

wants to merge 2 commits into from

Conversation

vladpaiu
Copy link
Member

Summary
zero-out URI in case of parse ERROR

Details
zero-out URI in case of parse ERROR

Solution
zero-out URI in case of parse ERROR

Compatibility
Backwards compatible

@vladpaiu vladpaiu requested a review from bogdan-iancu January 14, 2025 16:30
@vladpaiu vladpaiu added this to the 3.6-dev milestone Jan 14, 2025
@vladpaiu vladpaiu added the bug label Jan 14, 2025
bogdan-iancu added a commit that referenced this pull request Jan 23, 2025
@bogdan-iancu
Fix dangling parsed_uri structure in to_body
dba13a4 
Whenever the SIP URI parsing fails, be sure you reset to_body->parsed_uri, as other functions may later try to use it and we want to have it in a consistent format.
The issue was reported by @vladpaiu
Closes #3557

(cherry picked from commit 021172b)
@bogdan-iancu bogdan-iancu self-assigned this Jan 23, 2025
bogdan-iancu added a commit that referenced this pull request Jan 23, 2025
@bogdan-iancu
Fix dangling parsed_uri structure in to_body
4093083 
Whenever the SIP URI parsing fails, be sure you reset to_body->parsed_uri, as other functions may later try to use it and we want to have it in a consistent format.
The issue was reported by @vladpaiu
Closes #3557

(cherry picked from commit 021172b)
(cherry picked from commit dba13a4)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bogdan-iancu bogdan-iancu Awaiting requested review from bogdan-iancu

Assignees

@bogdan-iancu bogdan-iancu

Labels
bug fixed
Projects
None yet
Milestone
3.6-dev
Development

Successfully merging this pull request may close these issues.
2 participants
@vladpaiu @bogdan-iancu