28661 Commits (c7511029c339542391f7612bb07de2ff3e4c23a3)

 

All Branches   

Search

Author	SHA1	Message	Date
zuul	5417fc9f65	Merge "bundled pjproject: Fix DNS write to freed memory." into 14	9 years ago
Joshua Colp	3b909277d6	Merge "pjproject_bundled: Fix compile of pjsua so it handles audio" into 14	9 years ago
zuul	0e55ec619e	Merge "codecs.conf.sample: Add sample and option descriptions for codec_opus" into 14	9 years ago
Joshua Colp	a7e22cf192	Merge "res/stasis: Add CLI commands for displaying/debugging ARI apps" into 14	9 years ago
Richard Mudgett	4fda9e7b0b	bundled pjproject: Fix DNS write to freed memory. ... PJPROJECT 2.5.5 introduced a race condition with the -r5349 IPv6 DNS patch. The patch below fixes a write to freed memory under cartain DNS lookup conditions. 0006-r5477-svn-backport-Fix-DNS-write-on-freed-memory.patch ASTERISK-26516 Reported by: Richard Mudgett Change-Id: Ifdfae9ecf1e41b53080f33aab44ce1a220f349c5	9 years ago
mkrokosz	d3e0d5d40b	res_pjsip_outbound_publish: Fix crash when publishing device state. ... While publishing device state between multiple instances of Asterisk, a crash will sporadically occur under high CPS which looks to be a race condition operating on the publisher queue. ASTERISK-26506 Change-Id: I28da25d346deb358eff1d563485cabc433ce1ed6	9 years ago
Tzafrir Cohen	2897fc9ab0	netsock.c: fix includes for HURD ... ASTERISK-25070 Change-Id: I43bf94d2d36d3d8a8d0df40cd6c027d65a462814	9 years ago
Tzafrir Cohen	3748e336ac	define PATH_MAX for HURD ... PATH_MAX is not guaranteed to be defined. In parctice, all but the HURD define it to a constant. It is indeed not safe to assume there won't be longer paths and Asterisk generally does err safely on such cases. So even for HURD we'll just pretend PATH_MAX is 4096. ASTERISK-25070 #close Change-Id: I53d10ba18c34c132bcb640a5fd8e0da1d9b22db3	9 years ago
Kevin Harwell	35f9d472ba	codecs.conf.sample: Add sample and option descriptions for codec_opus ... codecs.conf.sample was missing codec opus's configuration options, descriptions, and examples. This patch adds the configuration options and examples to codecs.conf.sample that can be used with codec_opus. ASTERISK-26538 #close Change-Id: I1d89bb5e01d3e3b5bd78951b8dd0ff077a83dc8b	9 years ago
Matt Jordan	1a3e699316	res/stasis: Add CLI commands for displaying/debugging ARI apps ... This patch adds three new CLI commands: - ari show apps: list the registered ARI applications - ari show app: show detailed information about an ARI application - ari set debug: dump events being sent to an ARI application Note that while these CLI commands live in the res_stasis module, we use the 'ari' family for these commands. This was done as most users of Asterisk aren't aware of the semantic differences between ARI and res_stasis, and some 'ari' CLI commands already exist. ASTERISK-26488 #close Change-Id: I51ad6ff0cabee0d69db06858c13f18b1c513c9f5	9 years ago
Grachev Sergey	782dfa09a8	chan_sip: Incorrect display option Outbound reg. retry 403 ... If in sip.conf (general section) set option register_retry_403=no, the command "sip show settings" return value: Outbound reg. retry 403:0 If in sip.conf (general section) set option register_retry_403=yes, the command "sip show settings" return value: Outbound reg. retry 403:-1 * In static char "sip show settings" for "Outbound.reg. retry 403" option use AST_CLI_YESNO ASTERISK-26476 #close Change-Id: I3c14272f05f1067bd2aeaa8b3ef9cf8fcb12dcf9	9 years ago
Joshua Colp	69196a8db4	res_pjsip_sdp_rtp: Limit number of formats to defined maximum. ... The res_pjsip_sdp_rtp module did not restrict the number of formats added to a media stream in the SDP to the defined limit. If allow=all was used with additional loaded codecs this could result in the next media stream being overwritten some. This change restricts the module to limit it to the defined maximum and also increases the maximum in our bundled pjproject. ASTERISK-26541 #close Change-Id: I0dc5f59d3891246cafa2f3df5ec406f088559ee8	9 years ago
George Joseph	23812d60b9	pjproject_bundled: Fix compile of pjsua so it handles audio ... In order for pjsua and its python binding to actually negotiate audio for the testsuite tests, it needs g711 and resample. The pj* libraries themselves do not. Unfortunately, pjproject relies on a brand new libresample that most distros don't ship so we need to use the libresample already bundled with pjproject. Only the pjsua executable and the _pjsua.so python library are linked with it so it shouldn't interfere with asterisk itself. Also it was pointed out that apply_patches couldn't handle multiple patches that depended on each other during the dry-run, so the dry-run was removed. Change-Id: I24f397462b486dcdde0dcafe40e6c55a6593f098	9 years ago
Etienne Lessard	d9f9691d31	manager: Add documentation for NewConnectedLine event. ... The NewConnectedLine event has been added by commit fe7671f, but the documentation was missing. ASTERISK-26537 #close Change-Id: I7fc331f18caa28492da9303e576f70884ca8c9e6	9 years ago
zuul	201f21e0e5	Merge "bundled pjproject: Crashes while resolving DNS names." into 14	9 years ago
Joshua Colp	1926e555ef	Merge "astobj2: Declare private variable data_size for AO2_DEBUG only." into 14	9 years ago
Corey Farrell	b2bf6c4c22	vector: Prevent NULL argument to memcpy. ... Headers declare that memcpy does not accept NULL argument for the first two parameters. Add a conditional block to prevent memcpy and ast_free from running on vectors with NULL element array. ASTERISK-26526 #close Change-Id: I988a476bb5fcfcbd3f6d6c6b3e7769e4f9629b71	9 years ago
Corey Farrell	39ba7aa91f	astobj2: Declare private variable data_size for AO2_DEBUG only. ... Every ao2 object contains storage for a private variable data_size, though the value is never read if AO2_DEBUG is disabled. This change makes the variable conditional, reducing memory usage. ASTERISK-26524 #close Change-Id: If859929e507676ebc58b0f84247a4231e11da07f	9 years ago
Richard Mudgett	79ac79ab03	bundled pjproject: Crashes while resolving DNS names. ... PJPROJECT 2.5.5 introduced a race condition with the -r5349 IPv6 DNS patch. The patches below fix the DNS lookup race condition crash caused by attempting to send the same message twice for the single DNS lookup. 0006-r5471-svn-backport-Various-fixes-for-DNS-IPv6.patch 0006-r5473-svn-backport-Fix-pending-query.patch The patch below removes a cached DNS response from the hash table when another thread is referencing the old entry. The table still contained the entry when it was destroyed which can result in inexplicable crashes. 0006-r5475-svn-backport-Remove-DNS-cache-entry.patch ASTERISK-26344 #close Reported by: Ian Gilmour ASTERISK-26387 #close Reported by: Harley Peters Change-Id: I17fde80359e66f65a91341ceca58d914d0f61cc4	9 years ago
George Joseph	d84eaa46fa	pjproject_bundled: Fix issue where "/version.mak" wasn't found ... main/Makefile includes third-party/pjproject/build.mak but doesn't set PJDIR beforehand so "include $(PJDIR)/version.mak" evaluates to "/version.mak". Fix is to set PJDIR in main/Makefile before the include. Change-Id: I0f7c67d60209049056fe9c4b041bf0463aa95604	9 years ago
zuul	8b3b9a7b7a	Merge "Fix shutdown crash caused by modules being left open." into 14	9 years ago
Corey Farrell	b76afa5e4f	Fix shutdown crash caused by modules being left open. ... It is only safe to run ast_register_cleanup callbacks when all modules have been unloaded. Previously these callbacks were run during graceful shutdown, making it possible to crash during shutdown. ASTERISK-26513 #close Change-Id: Ibfa635bb688d1227ec54aa211d90d6bd45052e21	9 years ago
Rusty Newton	c2a2643c69	SAC documentation: don't specify transports for endpoints and registrations ... Removing explicit transport definition for endpoints and registrations. It isn't necessary and isn't generally advised. ASTERISK-26514 #close Change-Id: Ifdec5e631962438a4683600968dfa4bfd15909fb	9 years ago
zuul	bfbf41c33b	Merge "res_pjsip_sdp_rtp: Fix address family of explicit media_address." into 14	9 years ago
Joshua Colp	2a6d5fb271	Merge "pjsip: Fix a few media bugs with reinvites and asymmetric payloads." into 14	9 years ago
Joshua Colp	67e5d618b0	Merge "res_pjsip_caller_id: Fix crash on session timers UPDATE on inbound calls." into 14	9 years ago
Joshua Colp	eeeb27e4f9	Merge "pjproject_bundled: Remove usage of tar's --strip-components option" into 14	9 years ago
zuul	56876dbd21	Merge "app_voicemail: Clear voice mailbox in MailboxExists and MAILBOX_EXISTS." into 14	9 years ago
George Joseph	162bb27cfb	pjproject_bundled: Remove usage of tar's --strip-components option ... Older versions of tar don't support the --strip-components option so instead of doing 'tar --strip-components=1 -C source', we now just untar to the tarball's root directory (pjproject-<version>) and rename that directory to 'source'. Also fixed an issue where the pjproject source directory is a hard coded absolute pathname. ASTERISK-26510 #close ASTERISK-22480 #close Change-Id: I9ec92952507a91ff4e4d01e0149e09fd8e8f32b0	9 years ago
Joshua Colp	82ef2bb69d	res_pjsip_caller_id: Fix crash on session timers UPDATE on inbound calls. ... The res_pjsip_caller_id module wrongly assumed that a saved From header would always exist on sessions. This is true until an inbound call is received and a session timer causes an UPDATE to be sent. In this case there will be no saved From header and a crash will occur. This change makes it fall back to the From header of the outgoing request if no saved From header is present. ASTERISK-26307 #close Change-Id: Iccc3bc8d243b5ede9b81abf960292930c908d4fa	9 years ago
Joshua Colp	893afdac1a	Merge "test_astobj2_thrash: Fix multithreaded issues" into 14	9 years ago
Joshua Colp	ebc293e609	app_voicemail: Clear voice mailbox in MailboxExists and MAILBOX_EXISTS. ... When executing the MailboxExists dialplan application and MAILBOX_EXISTS dialplan function the passed in temporary voice mailbox was not cleared, causing it to try to free garbage. ASTERISK-26503 #close Change-Id: Ie21ccfa1b80b9c59318e596f6b8e17da2b5a7cb3	9 years ago
Joshua Colp	791d2319ce	pjsip: Fix a few media bugs with reinvites and asymmetric payloads. ... When channel format changes occurred as a result of an RTP re-negotiation the bridge was not informed this had happened. As a result the bridge technology was not re-evaluated and the channel may have been in a bridge technology that was incompatible with its formats. The bridge is now unbridged and the technology re-evaluated when this occurs. The chan_pjsip module also allowed asymmetric codecs for sending and receiving. This did not work with all devices and caused one way audio problems. The default has been changed to NOT do this but to match the sending codec to the receiving codec. For users who want asymmetric codecs an option has been added, asymmetric_rtp_codec, which will return chan_pjsip to the previous behavior. The codecs returned by the chan_pjsip module when queried by the bridge_native_rtp module were also not reflective of the actual negotiated codecs. The nativeformats are now returned as they reflect the actual negotiated codecs. ASTERISK-26423 #close Change-Id: I6ec88c6e3912f52c334f1a26983ccb8f267020dc	9 years ago
Joshua Colp	110c18f413	res_pjsip_sdp_rtp: Fix address family of explicit media_address. ... When an explicit media_address is provided the address family in the SDP needs to be set to reflect it. ASTERISK-26309 Change-Id: Ib9350cc91c120eb2f96f0623d3907d12af67eb79	9 years ago
George Joseph	f2d406ced8	test_astobj2_thrash: Fix multithreaded issues ... The test uses 4 threads to grow, count, lookup and shrink 15K objects in a container. If there's only 1 execution engine available, the test will complete in <50ms. If each threads gets its own execution engine, the test may timeout after 60 seconds because the count thread does a locked ao2_callback on the whole container in a tight loop with only a sched_yield to give up time. The lock contention makes the test execution times wildly variable and mostly timeout. 2 execution engines are OK, 3 results in about 33% failure rate and >=4 causes a 80% failure rate. To fix, the sched_yield was changed to a usleep(500). Also, the number of buckets specified for the container was an even number so that was changed to the next prime number greater than (MAX_HASH_ENTRIES / 100). That's 151 currently. Change-Id: I50cd2344161ea61bfe4b96d2a29a6ccf88385c77	9 years ago
Alexei Gradinari	3e040685c7	chan_pjsip: segfault on already disconnected session ... On heavy loaded system the TCP/TLS incoming calls could be disconnected by pjproject while these calls are being processed by asterisk. This patch uses functions pjsip_inv_add_ref/pjsip_inv_dec_ref to inform pjproject that an INVITE session is in use. ASTERISK-26482 #close Change-Id: Ia2e3e2f75358cdb530252a9ce158af3d5d9fdf33	9 years ago
zuul	7c3cc1b605	Merge "pjsip: Support dual stack automatically." into 14	9 years ago
zuul	8d91bfa3aa	Merge "pjproject_bundled: Fixed various build issues" into 14	9 years ago
Joshua Colp	1381b1d3fd	Merge "ARI: Add duplicate channel ID checking for channel creation." into 14	9 years ago
Joshua Colp	1567ccfc82	Merge "ARI: Detect duplicate channel IDs" into 14	9 years ago
Pascal Cadotte Michaud	cb342298cd	typo: s/paranthesis/parenthesis/ in a comment ... Change-Id: I7c1f4eb051177ee22cbe97e063d4a3effe29be30	9 years ago
George Joseph	0836b617e5	pjproject_bundled: Fixed various build issues ... * CFLAGS is now properly set when using older gcc. * All third-party pjproject targets have been removed. This fixes an issue with older libsrtp in some distros. * Manually removing the source directory now causes a rebuild. * EXTERNALS_CACHE_DIR is now properly checked. * Whitespace fixes. Change-Id: I98fec6847efc5602a9f41cb95096fd660a49fa60	9 years ago
Joshua Colp	47de0ee4f6	pjsip: Support dual stack automatically. ... This change adds support for dual stack automatically. No configuration is required and the IP address and version in the SIP messages and SDP will be automatically changed based on the transport over which the message is being sent. RTP usage has also been changed to listen on both IPv4 and IPv6 simultaneously to allow media to flow, and to allow ICE support on both simultaneously. This also allows failover between IPv6 and IPv4 to work as expected. ASTERISK-26309 #close Change-Id: I235a421d8f9a326606d861b449fa6fe3a030572d	9 years ago
Mark Michelson	ba145944a0	ARI: Add duplicate channel ID checking for channel creation. ... This is similar to what is done for origination, but for the 14 and up channel creation method. When attempting to create a channel, if a channel ID is specified and a channel already exists with that ID, then a 409 is returned. Change-Id: I77f9253278c6947939c418073b6b31065489187c	9 years ago
Mark Michelson	98378133c0	ARI: Detect duplicate channel IDs ... ARI and AMI allow for an explicit channel ID to be specified when originating channels. Unfortunately, there is nothing in place to prevent someone from using the same ID for multiple channels. Further complicating things, adding ID validation to channel allocation makes it impossible for ARI to discern why channel allocation failed, resulting in a vague error code being returned. The fix for this is to institute a new method for channel errors to be discerned. The method mirrors errno, in that when an error occurs, the caller can consult the channel errno value to determine what the error was. This initial iteration of the feature only introduces "unknown" and "channel ID exists" errors. However, it's possible to add more errors as needed. ARI uses this feature to determine why channel allocation failed and can return a 409 error during origination to show that a channel with the given ID already exists. ASTERISK-26421 Change-Id: Ibba7ae68842dab6df0c2e9c45559208bc89d3d06	9 years ago
snuffy	8e3aee2036	Fix issue with CLI not returning to prompt after running "features show" ... ASTERISK-26444 #close Change-Id: I91d645b7e6e5dba35f8c410df2be77a8c0e3acb8	9 years ago
zuul	1ecb4f9e2b	Merge "utils.c: Fix ast_set_default_eid for multiple platforms" into 14	9 years ago
zuul	63f2a7d007	Merge "res_rtp_asterisk: Add ice_blacklist option" into 14	9 years ago
zuul	e2cf981a1c	Merge "chan_sip: Support nat=auto_comedia or nat=force_rport,auto_comedia." into 14	9 years ago
Michael Walton	494bebeb6f	res_rtp_asterisk: Add ice_blacklist option ... Introduces ice_blacklist configuration in rtp.conf. Subnets listed in the form ice_blacklist = <subnet spec>, e.g. ice_blacklist = 192.168.1.0/255.255.255.0, are excluded from ICE host, srflx and relay discovery. This is useful for optimizing the ICE process where a system has multiple host address ranges and/or physical interfaces and certain of them are not expected to be used for RTP. Multiple ice_blacklist configuration lines may be used. If left unconfigured, all discovered host addresses are used, as per previous behavior. Documention in rtp.conf.sample. ASTERISK-26418 #close Change-Id: Ibee88f80d7693874fda1cceaef94a03bd86012c9	9 years ago