#### Pre-Submission Checklist - [x] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [x] Each component has a single commit (if not, squash them into one commit) - [x] No commits to README files for modules (changes must be done to docbook files in doc/ subfolder, the README file is autogenerated)
#### Type Of Change - [x] Small bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist: - [x] PR should be backported to stable branches - [x] Tested changes locally - [x] Related to issue #2459
#### Description There's a small regression after #2497.
If SDP didn't have an "a=rtcp" header (RFC1889 behavior), Kamailio had thrown an Error `can't extract 'a=rtcp' IP from the SDP` on every INVITE. After the PR Kamailio does not flood into log.
Please take into account IPv6 address parser after the commit.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2737
-- Commit Summary --
* nathelper: fix_nated_sdp added ignoring RFC3605-param if omitted
-- File Changes --
M src/modules/nathelper/nathelper.c (22)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2737.patch https://github.com/kamailio/kamailio/pull/2737.diff
Thanks for contributing!
Can you elaborate more on your remark `Please take into account IPv6 address parser after the commit.`? Your fix does not take care of IPv6, or the existing code does not do it?
@miconda the fix takes care IPv4 (AF_INET socket type). We've tested it on IPv4, but we cannot test it on IPv6. :(
The previous version only worked with IPv4. I've fixed it, but haven't test it.
@oej - is there a chance that you can test this PR? I see you discussed #2497, maybe you have some IPv6 environment at hand.
If not and no other comments, then it can be merged soon.
Let me check if I can set up tests.
Hello @dwagin
Could you please check the case:
if 'a=rtcp' is presents, but contains only port value, exactly as mentioned in #2768
`rtcp-attribute = "a=rtcp:" port [nettype space addrtype space connection-address] CRLF`
It seems that in this case the fix you've provided will not rewrite IP address in either "c=" or "o=" if fix_nated_sdp used with 2 or 8 flags.
Hello All, I'll check it out soon.
Please also have a look to #2769 - it was considered to first merge this pull request, as its the easier fix. Then this request could be updated, if necessary to fix other eventually remaining issues.
Merged #2769 in preparation to v5.5.1. This PR has now conflicts, it will be closed. If there is anything else that it fixes compared to #2769, please make a new PR for master branch.
Closed #2737.