[Home]

Summary:ASTERISK-04214: port in sip.conf is now searched as bindport
Reporter:Tomislav Petrovic (tomy)Labels:
Date Opened:2005-05-18 04:09:40Date Closed:2011-06-07 14:11:57
Priority:TrivialRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:
Description:When reading settings from sip.conf port is read from "bindport" setting
instead of "port" setting which was used up to now.
Sampe sip.conf.sample is not updated to reflect this change.

Cut from chan_sip.c
               } else if (!strcasecmp(v->name, "bindport")) {
                       if (sscanf(v->value, "%d", &ourport) == 1) {
                               bindaddr.sin_port = htons(ourport);
                       } else {
                               ast_log(LOG_WARNING, "Invalid port number '%s' at line %d of %s\n", v->value, v->lineno, config);
                       }


****** ADDITIONAL INFORMATION ******

I also think this change is not necessary and only confuses.
Comments:By: Russell Bryant (russell) 2005-05-18 20:05:57

This is not a bug.  It was changed to be more consistent with the rest of the channel drivers.  Also, it is correct in the sample config.