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.

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

[maxfwd]: Fix bug is_maxfwd_lt function limit value #3480

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

ar45
Copy link
Contributor

@ar45 ar45 commented Sep 25, 2024

Summary
is_maxfwd_lt( val ) was using a random value, the ptr address.

Solution
Correctly use the int value.

Compatibility
Should be backported to all supported versions of opensips.

Was using the address pointer as value.
Copy link

Any updates here? No progress has been made in the last 30 days, marking as stale.

@github-actions github-actions bot added the stale label Oct 26, 2024
@ar45
Copy link
Contributor Author

ar45 commented Oct 27, 2024

Still active

@stale stale bot removed the stale label Oct 27, 2024
Copy link

Any updates here? No progress has been made in the last 30 days, marking as stale.

@github-actions github-actions bot added the stale label Nov 26, 2024
@ar45
Copy link
Contributor Author

ar45 commented Nov 26, 2024

active

@github-actions github-actions bot removed the stale label Nov 27, 2024
@bogdan-iancu bogdan-iancu self-assigned this Dec 16, 2024
@bogdan-iancu
Copy link
Member

Great finding and fix @ar45 - thanks a lot and sorry for taking longer to merge this PR.

@bogdan-iancu bogdan-iancu reopened this Dec 16, 2024
@bogdan-iancu bogdan-iancu merged commit e6a0c98 into OpenSIPS:master Dec 16, 2024
53 checks passed
bogdan-iancu added a commit that referenced this pull request Dec 16, 2024
[maxfwd]: Fix bug `is_maxfwd_lt` function limit value

(cherry picked from commit e6a0c98)
bogdan-iancu added a commit that referenced this pull request Dec 16, 2024
[maxfwd]: Fix bug `is_maxfwd_lt` function limit value

(cherry picked from commit e6a0c98)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants