[asterisk-bugs] [JIRA] (ASTERISK-21225) Setting nat=force_rport in [general] sip.conf will never work

[Alexandre Vezina (JIRA)]

    [ https://issues.asterisk.org/jira/browse/ASTERISK-21225?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=204165#comment-204165 ] 

Alexandre Vezina edited comment on ASTERISK-21225 at 3/13/13 10:26 AM:
-----------------------------------------------------------------------

The test worked. directmedia=nonat gives Yes upon restart, module reload, config reload.

Though I had to edit chan_sip.c to make it compile. This one (v2) works for me, but ouputs

chan_sip.c: In function ‘reload_config’:
chan_sip.c:29375:2: warning: array subscript is above array bounds [-Warray-bounds]
chan_sip.c:29375:2: warning: array subscript is above array bounds [-Warray-bounds]
chan_sip.c:29375:2: warning: array subscript is above array bounds [-Warray-bounds]

I tried to find the reason of this, but didn't get a clue after 10 minutes. I guess you will find it faster than I will.
                
      was (Author: avezina at ubikvoip.com):
    This one compiles against 1.8.15-cert1
                  
> Setting nat=force_rport in [general] sip.conf will never work
> -------------------------------------------------------------
>
>                 Key: ASTERISK-21225
>                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-21225
>             Project: Asterisk
>          Issue Type: Bug
>      Security Level: None
>          Components: Channels/chan_sip/General
>    Affects Versions: 11.2.1
>            Reporter: Alexandre Vezina
>            Assignee: Alexandre Vezina
>            Severity: Minor
>         Attachments: asterisk-21225-handle-options-default-prob_1.8.diff, asterisk-21225-handle-options-default-prob_1.8_v2.diff, asterisk-21225-handle-options-default-prob_v3.diff
>
>
> I noticed that it was impossible to set nat=force_rport in the general section of sip.conf. It actually has no effect when issuing cli command "sip show settings" (result is Force rport: Auto (No).
> The function to reload configurations in chan_sip.c sets the flag SIP_PAGE3_NAT_AUTO_RPORT in the global flags and the function sip_parse_nat_option in config_parser.c will only add the flag SIP_NAT_FORCE_RPORT if SIP_PAGE3_NAT_AUTO_RPORT is not present.
> So, it seems impossible to raise the flag SIP_NAT_FORCE_RPORT in the [general] section of sip.conf.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.asterisk.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira