Commit Graph

30265 Commits

Author SHA1 Message Date
Corey Farrell
5ca2b36510 Add defaultbranch to .gitreview.
This will cause `git review` to post changes to the branch it is based
on instead of always using master.  The defaultbranch setting should be
updated when new major branches are created.

Change-Id: I3db009217c5ae399fb84bee95076f4dbb7fa52d2
2017-11-22 19:53:03 -05:00
Kevin Harwell
260ade8be8 pjsip: 183 without To tag does not negotiate media
If a 183 with sdp response is receive without a To tag the sdp is not
negotiated. According to RFC 3261 section 12.1.2 while a To tag is required,
the client needs to still be able to handle the missing tag case for
backwards compatibility.

This patch, accepted by and applied to pjproject, makes it so if an incoming
180/183 with SDP comes in without a To tag it gets appropriately handled.

ASTERISK-27442 #close

Change-Id: Ic9d6b01e05e8f4874eebbd7adfe05d932025d203
2017-11-22 11:00:09 -06:00
Alexander Traud
73a9b88110 res_rtp_asterisk: ICE server-reflexive candidates (srflx) with Dual-Stack.
Previously, Asterisk sent srflx only when configured exclusively for IPv4. Now,
srflx is gathered and sent via SDP, even when Asterisk is enabled for
Dual Stack (IPv4+IPv6) and an IPv4 interface is available/used.

ASTERISK-27437

Change-Id: Ie07d8e2bfa7b6fe06fcdc73d390a7a9a4d8c0bc1
2017-11-22 03:07:04 -06:00
Joshua Colp
3e62546511 Merge "aoc: Remove silly usage of RAII_VAR." into 15 2017-11-21 16:59:51 -06:00
Joshua Colp
1e2799f491 Merge "abstract_jb: Remove silly usage of RAII_VAR." into 15 2017-11-21 10:53:59 -06:00
Corey Farrell
c784dd2b96 res_parking: Make load_pri explicit.
res_parking has an implicit load_pri of 0 meaining it's one of the very
first modules loaded after modules with global symbols.  Set it
explicitly in the AST_MODULE_INFO block.

Change-Id: I297b6fb3ff6993ec004e667b22a74f5925906259
2017-11-21 10:34:39 -06:00
Joshua Colp
69fcd96e72 Merge "BuildSystem: pjsip_evsub_set_uas_timeout was not used." into 15 2017-11-21 10:17:50 -06:00
Joshua Colp
48ec7e49e8 Merge "app: Remove silly usage of RAII_VAR." into 15 2017-11-21 09:27:31 -06:00
Corey Farrell
ce5fb5b149 README: Convert to README.md.
Convert the README file to markdown format, remove the old README.  This
causes websites like github to display the README in a much nicer
format with live links.  The raw file is still very readable from
plain text editors and terminals.

Change-Id: I7d13131764a9a9026e5f8a6ddb245a01bbd788e7
2017-11-21 09:11:54 -06:00
Corey Farrell
d643fbda49 CLI: Finish conversion of completion handling to vectors.
Change-Id: Ib81318f4ee52a5e73b003316e13fe9be1dd897a1
2017-11-21 09:55:29 -05:00
Corey Farrell
7c9d73615c CLI: Refactor cli_complete.
* Stop using "_COMMAND NUMMATCHES" on remote consoles.  Using this
  command had doubled the amount of work needed from the Asterisk
  daemon for each completion request.
* Fix code formatting.
* Remove static buffer used to send the command, use the same buffer
  that will receive the results.
* Move sort from ast_cli_display_match_list.

Change-Id: Ie2211b519a3d4bec45bf46e0095bdd01d384cb69
2017-11-21 09:55:27 -05:00
Corey Farrell
b8a735e9f4 CLI: Rewrite ast_el_strtoarr to use vector's internally.
This rewrites ast_el_strtoarr to use vector's internally, but still
return the original NULL terminated array of strings.

Change-Id: Ibfe776cbe14f750effa9ca360930acaccc02e957
2017-11-21 09:55:24 -05:00
Corey Farrell
0ca507ba67 CLI: Refactor ast_cli_display_match_list.
* Stop estimating line count, just print until we run out of matches.
* Stop freeing entries, the caller does that anyways.
* Stop calculating / returning numoutput, it was ignored.

Change-Id: I7f92afa8bea92241a95227587367424c8c32a5cb
2017-11-21 09:55:22 -05:00
Joshua Colp
d92b786359 Merge "ccss: Remove silly usage of RAII_VAR." into 15 2017-11-21 08:52:20 -06:00
Corey Farrell
6748f690f5 CLI: Create ast_cli_completion_add function.
Some completion generators are very inefficent due to the way CLI
requests matches one at a time.  ast_cli_completion_add can be called
multiple times during one invokation of a CLI generator to add all
results without having to reinitialize the search state for each match.

Change-Id: I73d26d270bbbe1e3e6390799cfc1b639e39cceec
2017-11-21 09:41:21 -05:00
Corey Farrell
369e77d68b CLI: Remove calls to ast_cli_generator.
The ability to add to localized storage cannot be supported by
ast_cli_generator.  The only calls to ast_cli_generator should be by
functions that need to proxy the CLI generator, for example 'cli check
permissions' or 'core show help'.

* ast_cli_generatornummatches now retrieves the vector of matches and
  reports the number of elements (not including 'best' match).
* test_substitution retrieves and iterates the vector.

Change-Id: I8cd6b93905363cf7a33a2d2b0e2a8f8446d9f248
2017-11-21 09:41:17 -05:00
Joshua Colp
65606471dd Merge "cli: Remove silly usage of RAII_VAR." into 15 2017-11-21 08:17:27 -06:00
Joshua Colp
e472aee810 Merge "loader: Fix comments in struct ast_module." into 15 2017-11-21 07:46:16 -06:00
Joshua Colp
e7f0281472 Merge "menuselect: Remove ineffective weak attribute detection." into 15 2017-11-21 06:51:00 -06:00
Joshua Colp
5de5f10a5e Merge "README: Send people to secure websites where available." into 15 2017-11-21 05:23:53 -06:00
Joshua Colp
ebee67dca1 Merge "Build: Fix issues building without SSL." into 15 2017-11-21 05:23:34 -06:00
Joshua Colp
051744c4b9 Merge "Build: Fix OSX build issues." into 15 2017-11-21 05:22:47 -06:00
Alexander Traud
c4f07a9bd4 chan_sip: ICE contained square brackets around IPv6 addresses.
ASTERISK-27434

Change-Id: Iaeed89b4fa05d94c5f0ec2d3b7cd6e93d2d5a8f7
2017-11-21 03:53:39 -06:00
Corey Farrell
50e0e000c6 loader: Fix comments in struct ast_module.
Make the comments follow doxygen format, move comments to the line
before each field they describe.

Change-Id: Ic445468398b5e88f13910f7c2f70bd15aad33a27
2017-11-20 18:21:39 -06:00
Joshua Colp
a881c52c78 Merge "res_pjsip: Use sorcery prefix operation for contact lookup" into 15 2017-11-20 17:27:07 -06:00
Joshua Colp
67f5875969 Merge "CLI: Create ast_cli_completion_vector." into 15 2017-11-20 17:26:59 -06:00
Joshua Colp
d8965d9bde Merge "acl: Fix allocation related issues." into 15 2017-11-20 16:48:04 -06:00
Joshua Colp
534637e91b Merge "bridge_basic: Ignore answer from transfer target when they've timed out." into 15 2017-11-20 14:46:38 -06:00
Corey Farrell
69e3fb73d0 cli: Remove silly usage of RAII_VAR.
Change-Id: I81aacfee7cd26e4fc5eef07bca582700c2975bd7
2017-11-20 14:32:38 -06:00
Corey Farrell
9f460c8efd ccss: Remove silly usage of RAII_VAR.
Change-Id: I5ce40035e0a940e4e56f6322c1dcd47fbd509b98
2017-11-20 14:25:58 -06:00
Corey Farrell
f893822a8b app: Remove silly usage of RAII_VAR.
Change-Id: Ideb594f7aae134974fb78d5477ba0853b97b8625
2017-11-20 14:21:22 -06:00
Corey Farrell
93d979d520 aoc: Remove silly usage of RAII_VAR.
Change-Id: I07907f833b81aeb0128bc9442a2abb52679c7511
2017-11-20 14:05:10 -06:00
Corey Farrell
3c3c0140a3 abstract_jb: Remove silly usage of RAII_VAR.
Change-Id: I9d56175369363d1dc735504cf78a3a5577069f49
2017-11-20 14:03:14 -06:00
Joshua Colp
62001f0221 Merge "doxygen: Remove obsolete contents." into 15 2017-11-20 13:33:29 -06:00
Corey Farrell
e8c9cb6c40 res_mwi_external_ami: Remove incorrect load priority.
res_mwi_external_ami specified AST_MODFLAG_LOAD_ORDER but didn't set
load_pri, resulting in an actual load priority of 0.  This module only
provides AMI actions so it has no reason to load early.

Change-Id: I82987fcf10d3ea42716b2f9df915b16687fd5839
2017-11-20 13:18:57 -06:00
Corey Farrell
0e2385581b Loader: Remove unneeded load_pri declarations.
Instead of specifying AST_MODFLAG_LOAD_ORDER with load_pri
AST_MODPRI_DEFAULT just use AST_MODFLAG_DEFAULT.

Change-Id: I0123258eafce324249433a69df15a85cc16e509f
2017-11-20 14:16:05 -05:00
Joshua Colp
8090aaf823 Merge "res_fax: Remove checks for unsigned values being >= 0." into 15 2017-11-20 13:04:28 -06:00
Joshua Colp
882bdcdd60 Merge "res_snmp: Declare RONLY if net-snmp headers do not." into 15 2017-11-20 12:28:30 -06:00
Joshua Colp
6506cb9d21 Merge "iostream: Fix ast_iostream_printf declaration." into 15 2017-11-20 11:49:47 -06:00
Joshua Colp
241e00e310 Merge "tests: Fix warnings found on Mac." into 15 2017-11-20 11:32:42 -06:00
Joshua Colp
86cba949a5 Merge "app_minivm: Fix possible uninitialized return value." into 15 2017-11-20 10:22:06 -06:00
Joshua Colp
489823e3dc Merge "res_pjsip: Fix warning by deferring implicit type cast." into 15 2017-11-20 10:21:48 -06:00
Alexander Traud
43dad7f669 BuildSystem: pjsip_evsub_set_uas_timeout was not used.
ASTERISK-27435

Change-Id: Id318a7ae6d7d69b53f911d30bf3eece64852f15c
2017-11-20 09:56:41 -06:00
Corey Farrell
1346e1e758 Build: Fix OSX build issues.
OSX does not support 'readlink -f' or 'sed -r'.  Replace readlink with
the GNU make macro 'realpath'.  Replace sed with grep in one place, cut
in the other.

ASTERISK-27332

Change-Id: I5d34ecca905384decb22ead45c913ae5e8aff748
2017-11-19 14:33:12 -06:00
Corey Farrell
0b4fe4a0c7 Build: Fix issues building without SSL.
* Fix conditional in libasteriskssl.
* Use variables produced by configure to link the SSL and uuid libraries
  into libasteriskpj.so instead of hard-coding them.

ASTERISK-27431

Change-Id: I3977931fd3ef8c4e4376349ccddb354eb839b58d
2017-11-19 15:25:22 -05:00
Corey Farrell
65ff827751 res_pjsip: Fix warning by deferring implicit type cast.
Mac doesn't like the comparison of -1 to an enum, so store the result of
ast_sip_str_to_dtmf to an int so we can check for the negative return
value.  ast_sip_str_to_dtmf returns an int so this is only delaying the
implicit type cast.

Change-Id: I0c262c1719ee951aae1f437d733a301cf5f8ad29
2017-11-19 13:31:50 -06:00
Corey Farrell
ef4dc43a75 tests: Fix warnings found on Mac.
test_pbx used raise without explicitly including signal.h.  On Mac for
some reason nothing else includes it.

test_logger checked if an unsigned int was negative.  Switch the
variable to 'int' so that error check can be effective.

Change-Id: Ie1db5dd1818ac25cc2ae41b644f848b5865b1362
2017-11-18 21:15:56 -06:00
Corey Farrell
85f468cd64 res_snmp: Declare RONLY if net-snmp headers do not.
Some net-snmp builds do not provide the RONLY declare, only
NETSNMP_OLDAPI_RONLY.  Map RONLY to NETSNMP_OLDAPI_RONLY to get around
this error.

Change-Id: Ida5c7ad9406515825485c4d3b4a34fd6ad0da577
2017-11-18 20:28:23 -06:00
Corey Farrell
85d675b14c res_fax: Remove checks for unsigned values being >= 0.
It's impossible for gwtimeout or fdtimeout to be less than 0 because
they are unsigned int's.  Remove checks and unreachable branches.

Change-Id: Ib2286960621e6ee245e40013c84986143302bc78
2017-11-18 20:05:03 -06:00
Corey Farrell
9da69ac6c1 iostream: Fix ast_iostream_printf declaration.
This adds the printf attribute and changes 'fmt' from 'const void *' to
'const char *'.  This resolves a warning from some compiler for
vsnprintf needing a literal string for format.

Change-Id: I71c33a8262590042ee451e1146760c10bb22fb78
2017-11-18 19:55:20 -06:00