aboutsummaryrefslogtreecommitdiffstats
path: root/codecs
AgeCommit message (Collapse)AuthorFilesLines
2010-11-05Merged revisions 293968 via svnmerge from sruffell1-4/+4
https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r293968 | sruffell | 2010-11-04 19:02:53 -0500 (Thu, 04 Nov 2010) | 17 lines codecs/codec_dahdi: Prevent "choppy" audio when receiving unexpected frame sizes. dahdi-linux 2.4.0 (specifically commit 9034) added the capability for the wctc4xxp to return more than a single packet of data in response to a read. However, when decoding packets, codec_dahdi was still assuming that the default number of samples was in each read. In other words, each packet your provider sent you, regardless of size, would result in 20 ms of decoded data (30 ms if decoding G723). If your provider was sending 60 ms packets then codec_dahdi would end up stripping 40 ms of data from each transcoded frame resulting in "choppy" audio. This would only affect systems where G729 packets are arriving in sizes greater than 20ms or G723 packets arriving in sizes greater than 30ms. DAHDI-744. ........ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@293969 f38db490-d61c-443f-a65b-d21fe96a405b
2010-09-09Merged revisions 285817 via svnmerge from pabelanger1-0/+8
https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r285817 | pabelanger | 2010-09-09 18:34:35 -0400 (Thu, 09 Sep 2010) | 8 lines GCC 4.2.x optimizations result in improper behavior of GSM codec (closes issue #17688) Reported by: pprindeville Patches: asterisk-trunk-bugid11243.patch uploaded by pprindeville (license 347) Tested by: mkeuter, pprindeville ........ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@285818 f38db490-d61c-443f-a65b-d21fe96a405b
2010-06-03Merged revisions 267622 via svnmerge from rmudgett1-2/+0
https://origsvn.digium.com/svn/asterisk/trunk ........ r267622 | rmudgett | 2010-06-03 13:38:00 -0500 (Thu, 03 Jun 2010) | 1 line Make compile again. ........ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@267623 f38db490-d61c-443f-a65b-d21fe96a405b
2010-06-03Merged revisions 267492 via svnmerge from mmichelson8-214/+0
https://origsvn.digium.com/svn/asterisk/trunk ........ r267492 | mmichelson | 2010-06-03 12:09:11 -0500 (Thu, 03 Jun 2010) | 6 lines Remove unnecessary code relating to PLC. The logic for handling generic PLC is now handled in ast_write in channel.c instead of in translation code. ........ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@267507 f38db490-d61c-443f-a65b-d21fe96a405b
2010-05-15Merged revisions 252488 via svnmerge from pabelanger1-10/+12
https://origsvn.digium.com/svn/asterisk/trunk ........ r252488 | tilghman | 2010-03-15 12:27:08 -0400 (Mon, 15 Mar 2010) | 9 lines Make the Makefile logic more explicit and move the Snow Leopard logic down to where it's not executed on non-Darwin systems. (closes issue #17028) Reported by: pabelanger Patches: issue17028_20100315.patch uploaded by seanbright (license 71) 20100315__issue17028.diff.txt uploaded by tilghman (license 14) Tested by: tilghman, pabelanger ........ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@263202 f38db490-d61c-443f-a65b-d21fe96a405b
2010-03-09Merged revisions 251475 via svnmerge from tilghman1-0/+5
https://origsvn.digium.com/svn/asterisk/trunk ........ r251475 | tilghman | 2010-03-09 14:30:03 -0600 (Tue, 09 Mar 2010) | 8 lines Build system modifications to ensure that Asterisk properly builds on Mac OS X 10.6. (closes issue #16997) Reported by: jquinn Patches: 20100309__issue16997__2.diff.txt uploaded by tilghman (license 14) Tested by: tilghman, russell ........ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@251496 f38db490-d61c-443f-a65b-d21fe96a405b
2009-11-10Merged revisions 229282 via svnmerge from file1-42/+0
https://origsvn.digium.com/svn/asterisk/trunk ................ r229282 | file | 2009-11-10 16:06:13 -0400 (Tue, 10 Nov 2009) | 15 lines Merged revisions 229281 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r229281 | file | 2009-11-10 16:03:14 -0400 (Tue, 10 Nov 2009) | 8 lines Remove broken support for direct transcoding between G.726 RFC3551 and G.726 AAL2. On some systems the translation core would actually consider g726aal2 -> g726 -> signed linear to be a quicker path then g726aal2 -> signed linear which exposed this problem. (closes issue #15504) Reported by: globalnetinc ........ ................ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@229285 f38db490-d61c-443f-a65b-d21fe96a405b
2009-11-06Merged revisions 228441 via svnmerge from dvossel1-1/+1
https://origsvn.digium.com/svn/asterisk/trunk ........ r228441 | dvossel | 2009-11-06 11:22:31 -0600 (Fri, 06 Nov 2009) | 3 lines Fixes merging issue from 1.4, frame data is held in data.ptr in trunk ........ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@228447 f38db490-d61c-443f-a65b-d21fe96a405b
2009-11-06Merged revisions 228420 via svnmerge from dvossel1-0/+5
https://origsvn.digium.com/svn/asterisk/trunk ................ r228420 | dvossel | 2009-11-06 11:09:01 -0600 (Fri, 06 Nov 2009) | 19 lines Merged revisions 228418 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r228418 | dvossel | 2009-11-06 11:07:13 -0600 (Fri, 06 Nov 2009) | 13 lines fixes segfault in iLBC For reasons not yet known, it appears possible for an ast_frame to have a datalen greater than zero while the actual data is NULL during Packet Loss Concealment. Most codecs don't support PLC so this doesn't affect them. This patch catches the malformed frame and prevents the crash from occuring. Additional efforts to determine why it is possible for a frame to look like this are still being investigated. (issue #16979) ........ ................ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@228421 f38db490-d61c-443f-a65b-d21fe96a405b
2009-08-10AST-2009-005tilghman1-4/+4
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@211580 f38db490-d61c-443f-a65b-d21fe96a405b
2009-07-30Merged revisions 209554 via svnmerge from dbrooks1-1/+1
https://origsvn.digium.com/svn/asterisk/trunk ........ r209554 | dbrooks | 2009-07-30 11:07:05 -0500 (Thu, 30 Jul 2009) | 6 lines Fixes numerous spelling errors. Patch submitted by alecdavis. (closes issue #15595) Reported by: alecdavis ........ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@209594 f38db490-d61c-443f-a65b-d21fe96a405b
2009-07-21Merged revisions 207680 via svnmerge from kpfleming3-3/+3
https://origsvn.digium.com/svn/asterisk/trunk ................ r207680 | kpfleming | 2009-07-21 08:28:04 -0500 (Tue, 21 Jul 2009) | 18 lines Merged revisions 207647 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r207647 | kpfleming | 2009-07-21 08:04:44 -0500 (Tue, 21 Jul 2009) | 12 lines Ensure that user-provided CFLAGS and LDFLAGS are honored. This commit changes the build system so that user-provided flags (in ASTCFLAGS and ASTLDFLAGS) are supplied to the compiler/linker *after* all flags provided by the build system itself, so that the user can effectively override the build system's flags if desired. In addition, ASTCFLAGS and ASTLDFLAGS can now be provided *either* in the environment before running 'make', or as variable assignments on the 'make' command line. As a result, the use of COPTS and LDOPTS is no longer necessary, so they are no longer documented, but are still supported so as not to break existing build systems that supply them when building Asterisk. ........ ................ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@207685 f38db490-d61c-443f-a65b-d21fe96a405b
2009-07-15Merged revisions 206636 via svnmerge from seanbright1-3/+3
https://origsvn.digium.com/svn/asterisk/trunk ................ r206636 | seanbright | 2009-07-15 12:00:24 -0400 (Wed, 15 Jul 2009) | 9 lines Merged revisions 206635 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r206635 | seanbright | 2009-07-15 11:57:51 -0400 (Wed, 15 Jul 2009) | 1 line Only print debug info in codec_dahdi if we are asking for it. ........ ................ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@206639 f38db490-d61c-443f-a65b-d21fe96a405b
2009-06-18Merged revisions 201678 via svnmerge from dvossel1-1/+1
https://origsvn.digium.com/svn/asterisk/trunk ........ r201678 | dvossel | 2009-06-18 11:37:42 -0500 (Thu, 18 Jun 2009) | 11 lines fixes some memory leaks and redundant conditions (closes issue #15269) Reported by: contactmayankjain Patches: patch.txt uploaded by contactmayankjain (license 740) memory_leak_stuff.trunk.diff uploaded by dvossel (license 671) Tested by: contactmayankjain, dvossel ........ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@201679 f38db490-d61c-443f-a65b-d21fe96a405b
2009-05-15Merged revisions 194722 via svnmerge from russell2-6/+6
https://origsvn.digium.com/svn/asterisk/trunk ........ r194722 | russell | 2009-05-15 12:59:08 -0500 (Fri, 15 May 2009) | 4 lines Shuttle some bits around to address some gain issues with G.722. (closes AST-209) ........ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@194725 f38db490-d61c-443f-a65b-d21fe96a405b
2009-05-15Merged revisions 194718 via svnmerge from russell2-26/+2
https://origsvn.digium.com/svn/asterisk/trunk ........ r194718 | russell | 2009-05-15 12:37:12 -0500 (Fri, 15 May 2009) | 2 lines Further simplify codec_g722 build. ........ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@194721 f38db490-d61c-443f-a65b-d21fe96a405b
2009-05-15Merged revisions 194714 via svnmerge from russell1-3/+3
https://origsvn.digium.com/svn/asterisk/trunk ........ r194714 | russell | 2009-05-15 12:24:39 -0500 (Fri, 15 May 2009) | 2 lines Actually force running make for g722. ........ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@194717 f38db490-d61c-443f-a65b-d21fe96a405b
2009-02-17Several changes to codec_dahdi to play nice with G723.sruffell1-84/+325
This commit brings in the changes that were living out on the svn/asterisk/team/sruffell/asterisk-trunk-transcoder branch. codec_dahdi.c now always uses signed linear as the simple codec so that a soft g729 codec will not end up being preferred to the hardware codec. There are also changes to allow codec_dahdi.c to feed packets to the hardware in the native sample size of the codec. This solves problems with choppy audio when using G723. git-svn-id: http://svn.digium.com/svn/asterisk/trunk@176760 f38db490-d61c-443f-a65b-d21fe96a405b
2008-11-20Merged revisions 157859 via svnmerge from kpfleming1-1/+1
https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r157859 | kpfleming | 2008-11-19 15:34:47 -0600 (Wed, 19 Nov 2008) | 7 lines the gcc optimizer frequently finds broken code (use of uninitalized variables, unreachable code, etc.), which is good. however, developers usually compile with the optimizer turned off, because if they need to debug the resulting code, optimized code makes that process very difficult. this means that we get code changes committed that weren't adequately checked over for these sorts of problems. with this build system change, if (and only if) --enable-dev-mode was used and DONT_OPTIMIZE is turned on, when a source file is compiled it will actually be preprocessed (into a .i or .ii file), then compiled once with optimization (with the result sent to /dev/null) and again without optimization (but only if the first compile succeeded, of course). while making these changes, i did some cleanup work in Makefile.rules to move commonly-used combinations of flag variables into their own variables, to make the file easier to read and maintain ........ git-svn-id: http://svn.digium.com/svn/asterisk/trunk@157974 f38db490-d61c-443f-a65b-d21fe96a405b
2008-10-30fix a few small things found by using sparsekpfleming1-1/+2
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@152809 f38db490-d61c-443f-a65b-d21fe96a405b
2008-10-17Merge codec_consistency branch. This should make sample usage much happier.qwell36-1052/+358
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@150729 f38db490-d61c-443f-a65b-d21fe96a405b
2008-10-15When using MALLOC_DEBUG, codec_lpc10 leaks memory, because it matches a librarytilghman1-1/+1
malloc() with an ast_free (which, of course, doesn't match up with known allocated memory, so the free fails). (closes issue #13702) Reported by: eliel Patches: codec_lpc10_lpcini.c uploaded by eliel (license 64) git-svn-id: http://svn.digium.com/svn/asterisk/trunk@149637 f38db490-d61c-443f-a65b-d21fe96a405b
2008-09-12Create a new config file status, CONFIG_STATUS_FILEINVALID for differentiatingtilghman9-27/+9
when a file is invalid from when a file is missing. This is most important when we have two configuration files. Consider the following example: Old system: sip.conf users.conf Old result New result ======== ========== ========== ========== Missing Missing SIP doesn't load SIP doesn't load Missing OK SIP doesn't load SIP doesn't load Missing Invalid SIP doesn't load SIP doesn't load OK Missing SIP loads SIP loads OK OK SIP loads SIP loads OK Invalid SIP loads incompletely SIP doesn't load Invalid Missing SIP doesn't load SIP doesn't load Invalid OK SIP doesn't load SIP doesn't load Invalid Invalid SIP doesn't load SIP doesn't load So in the case when users.conf doesn't load because there's a typo that disrupts the syntax, we may only partially load users, instead of failing with an error, which may cause some calls not to get processed. Worse yet, the old system would do this with no indication that anything was even wrong. (closes issue #10690) Reported by: dtyoo Patches: 20080716__bug10690.diff.txt uploaded by Corydon76 (license 14) git-svn-id: http://svn.digium.com/svn/asterisk/trunk@142992 f38db490-d61c-443f-a65b-d21fe96a405b
2008-09-02Update instructions for getting libresamplerussell1-1/+2
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@140566 f38db490-d61c-443f-a65b-d21fe96a405b
2008-08-20Remove extraneous debugging messages.sruffell1-2/+0
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@139154 f38db490-d61c-443f-a65b-d21fe96a405b
2008-08-20Fix bug where the samples were not accurate when in G723 mode, which wouldsruffell1-5/+8
cause the timestamp field of the RTP header to be invalid. git-svn-id: http://svn.digium.com/svn/asterisk/trunk@139153 f38db490-d61c-443f-a65b-d21fe96a405b
2008-08-09More RSW merges. This should do it for the channels/ dir.seanbright1-4/+4
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@136917 f38db490-d61c-443f-a65b-d21fe96a405b
2008-08-07Updating codec_dahdi to the new transcoder interface.sruffell1-108/+80
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@136676 f38db490-d61c-443f-a65b-d21fe96a405b
2008-08-07More merges from resolve-shadow warnings:seanbright2-5/+5
utils/ codecs/ and a change I missed from formats/ git-svn-id: http://svn.digium.com/svn/asterisk/trunk@136408 f38db490-d61c-443f-a65b-d21fe96a405b
2008-07-21Remove libresample from the Asterisk source tree. It is now available in itsrussell1-1/+8
own repository, and must be installed like any other library for Asterisk to use. The two modules that require it are codec_resample and app_jack. To install libresample: $ svn co http://svn.digium.com/svn/libresample/trunk libresample $ cd libresample $ ./configure $ make $ sudo make install This code is currently in our own repository because the build system did not include the appropriate targets for building a dynamic library or for installing the library. git-svn-id: http://svn.digium.com/svn/asterisk/trunk@132390 f38db490-d61c-443f-a65b-d21fe96a405b
2008-07-21Enable higher quality resampling, as it doesn't have a noticeable performancerussell1-1/+1
impact on my machine .. git-svn-id: http://svn.digium.com/svn/asterisk/trunk@132388 f38db490-d61c-443f-a65b-d21fe96a405b
2008-07-11Janitor patch to change uses of sizeof to ARRAY_LENbbryant3-6/+6
(closes issue #13054) Reported by: pabelanger Patches: ARRAY_LEN.patch2 uploaded by pabelanger (license 224) Tested by: seanbright git-svn-id: http://svn.digium.com/svn/asterisk/trunk@130129 f38db490-d61c-443f-a65b-d21fe96a405b
2008-06-26Convert casts to unions, to fix alignment issues on Solaristilghman11-24/+24
(closes issue #12932) Reported by: snuffy Patches: bug_12932_20080627.diff uploaded by snuffy (license 35) git-svn-id: http://svn.digium.com/svn/asterisk/trunk@125386 f38db490-d61c-443f-a65b-d21fe96a405b
2008-06-25Merged revisions 125132 via svnmerge from kpfleming1-1/+2
https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r125132 | kpfleming | 2008-06-25 17:21:30 -0500 (Wed, 25 Jun 2008) | 10 lines allow tonezone to live in a different place than DAHDI/Zaptel, since dahdi-tools and dahdi-linux are now separate packages and can be installed in different places don't include tonezone.h in dahdi_compat.h, because only a couple of modules need it get app_rpt building again after the DAHDI changes (closes issue #12911) Reported by: tzafrir ........ git-svn-id: http://svn.digium.com/svn/asterisk/trunk@125138 f38db490-d61c-443f-a65b-d21fe96a405b
2008-06-12Goodbye Zaptel, hello DAHDI. Removes Zaptel driver support with DAHDI. ↵jpeeler1-67/+66
Configuration file and dialplan backwards compatability has been put in place where appropiate. Release announcement to follow. git-svn-id: http://svn.digium.com/svn/asterisk/trunk@122234 f38db490-d61c-443f-a65b-d21fe96a405b
2008-06-10Revision 117802 changed frame.data to frame.data.ptr however codec_ilbc.c ↵juggie1-4/+4
was not updated. This resolves that oversight. git-svn-id: http://svn.digium.com/svn/asterisk/trunk@121599 f38db490-d61c-443f-a65b-d21fe96a405b
2008-05-22Fix a few places where frame data was used directly.qwell1-4/+4
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@117828 f38db490-d61c-443f-a65b-d21fe96a405b
2008-05-22- revert change to ast_queue_hangup and create ast_queue_hangup_with_causemvanbaak10-44/+44
- make data member of the ast_frame struct a named union instead of a void Recently the ast_queue_hangup function got a new parameter, the hangupcause Feedback came in that this is no good and that instead a new function should be created. This I did. The hangupcause was stored in the seqno member of the ast_frame struct. This is not very elegant, and since there's already a data member that one should be used. Problem is, this member was a void *. Now it's a named union so it can hold a pointer, an uint32 and there's a padding in case someone wants to store another type in there in the future. This commit is so massive, because all ast_frame.data uses have to be altered to ast_frame.data.data Thanks russellb and kpfleming for the feedback. (closes issue #12674) Reported by: mvanbaak git-svn-id: http://svn.digium.com/svn/asterisk/trunk@117802 f38db490-d61c-443f-a65b-d21fe96a405b
2008-05-05Merged revisions 115327 via svnmerge from file1-0/+1
https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r115327 | file | 2008-05-05 19:10:05 -0300 (Mon, 05 May 2008) | 2 lines Make sure that either the main speex library contains preprocess functions or that speexdsp does. If both fail then speex stuff can not be built. ........ git-svn-id: http://svn.digium.com/svn/asterisk/trunk@115328 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-28Merged revisions 111856 via svnmerge from qwell1-3/+3
https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r111856 | qwell | 2008-03-28 16:45:35 -0500 (Fri, 28 Mar 2008) | 12 lines Allow gsm to compile correctly on x86 with gcc4 optimizations. (closes issue #11243) Reported by: whiskerp Patches: 11243-maybe-asm.diff uploaded by qwell (license 4) Tested by: Seggy (IRC) Note: While I did write this patch, I would not have found this if fossil had not reported and fixed issue #12253. A huge thanks to him for helping to (indirectly) find the problem here. ........ git-svn-id: http://svn.digium.com/svn/asterisk/trunk@111857 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-26Merged revisions 110880 via svnmerge from kpfleming50-7359/+5
https://origsvn.digium.com/svn/asterisk/branches/1.4 ................ r110880 | kpfleming | 2008-03-26 09:42:35 -0700 (Wed, 26 Mar 2008) | 10 lines Merged revisions 110869 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.2 ........ r110869 | kpfleming | 2008-03-26 08:53:46 -0700 (Wed, 26 Mar 2008) | 2 lines due to licensing restrictions, we cannot distribute the source code for iLBC encoding and decoding... so remove it, and add instructions on how the user can obtain it themselves ........ ................ git-svn-id: http://svn.digium.com/svn/asterisk/trunk@110881 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-21Merged revisions 110474 via svnmerge from qwell1-3/+3
https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r110474 | qwell | 2008-03-21 09:32:52 -0500 (Fri, 21 Mar 2008) | 7 lines Don't attempt to do optimizations of gsm on mips platforms either. (closes issue #12270) Reported by: zandbelt Patches: 026-gsm-mips.patch uploaded by zandbelt (license 33) ........ git-svn-id: http://svn.digium.com/svn/asterisk/trunk@110475 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-20Use the correct buffer for g722tolin16_sample. This shouldn't have caused anyrussell1-3/+3
problems, but Qwell noticed the typo here. git-svn-id: http://svn.digium.com/svn/asterisk/trunk@110339 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-18Merged revisions 109648 via svnmerge from qwell1-1/+1
https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r109648 | qwell | 2008-03-18 14:23:44 -0500 (Tue, 18 Mar 2008) | 7 lines Allow codecs that use log2comp (g726) to compile correctly on x86 with gcc4 optimizations. (closes issue #12253) Reported by: fossil Patches: log2comp.patch uploaded by fossil (license 140) ........ git-svn-id: http://svn.digium.com/svn/asterisk/trunk@109651 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-11Merged revisions 107464 via svnmerge from kpfleming1-1/+1
https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r107464 | kpfleming | 2008-03-11 09:53:03 -0500 (Tue, 11 Mar 2008) | 2 lines fix various other problems found by gcc 4.3 ........ git-svn-id: http://svn.digium.com/svn/asterisk/trunk@107466 f38db490-d61c-443f-a65b-d21fe96a405b
2008-03-07Merge changes from team/russell/g722-sillyness ...russell1-2/+6
Fix a number of other places where the number of samples in a G722 frame was not properly handled because of various reasons. main/rtp.c: - When a G722 frame is read from the smoother, the number of samples in the frame must be divided by 2 before being sent out over the network. Even though G722 is 16 kHz, an error in some previous spec has made it so that we have to list the number of samples such as if it was 8 kHz. main/file.c: - When scheduling the next time to expect a frame, take into account that the format of the file we're reading from may not be 8 kHz. codecs/codec_g722.c: - When converting from G722 to slinear, g722_decode() expects its samples parameter to be in the silly (real samples / 2) format. Make it so. - When converting from slinear to G722, properly set the number of samples in the frame to be the number of bytes of output * 2. formats/format_pcm.c: - This format module handles G722, among a number of other formats. However, the read() and seek() functions did not account for the fact that G722 has 2 samples per byte. (closes issue #12130, reported by rickross, patched by me) git-svn-id: http://svn.digium.com/svn/asterisk/trunk@106501 f38db490-d61c-443f-a65b-d21fe96a405b
2008-01-16Merged revisions 98951 via svnmerge from file1-0/+1
https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r98951 | file | 2008-01-15 21:13:27 -0400 (Tue, 15 Jan 2008) | 4 lines Add autoconf logic for speexdsp. Later versions use a separate library for some things so we need to use it if present in codec_speex. (closes issue #11693) Reported by: yzg ........ git-svn-id: http://svn.digium.com/svn/asterisk/trunk@98952 f38db490-d61c-443f-a65b-d21fe96a405b
2008-01-15Merged revisions 98943 via svnmerge from russell1-17/+18
https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r98943 | russell | 2008-01-15 17:26:52 -0600 (Tue, 15 Jan 2008) | 25 lines Commit a fix for some memory access errors pointed out by the valgrind2.txt output on issue #11698. The issue here is that it is possible for an instance of a translator to get destroyed while the frame allocated as a part of the translator is still being processed. Specifically, this is possible anywhere between a call to ast_read() and ast_frame_free(), which is _a lot_ of places in the code. The reason this happens is that the channel might get masqueraded during this time. During a masquerade, existing translation paths get destroyed. So, this patch fixes the issue in an API and ABI compatible way. (This one is for you, paravoid!) It changes an int in ast_frame to be used as flag bits. The 1 bit is still used to indicate that the frame contains timing information. Also, a second flag has been added to indicate that the frame came from a translator. When a frame with this flag gets released and has this flag, a function is called in translate.c to let it know that this frame is doing being processed. At this point, the flag gets cleared. Also, if the translator was requested to be destroyed while its internal frame still had this flag set, its destruction has been deffered until it finds out that the frame is no longer being processed. Admittedly, this feels like a hack. But, it does fix the issue, and I was not able to think of a better solution ... ........ git-svn-id: http://svn.digium.com/svn/asterisk/trunk@98944 f38db490-d61c-443f-a65b-d21fe96a405b
2008-01-11Kevin noted that the thing that I _actually_ changed here was that I convertedrussell1-3/+3
a value from a double, to a float, back to a double. Sure enough, when I changed my interim variable back to a double, it still blows up. Switching all of these to a float fixes the problem. This seems like a compiler bug where a double passed as an argument isn't getting properly aligned, so I'll have to see if I can replicate it with a small test program. (related to issue #11725) git-svn-id: http://svn.digium.com/svn/asterisk/trunk@98308 f38db490-d61c-443f-a65b-d21fe96a405b
2008-01-11Fix a bus error that happened when asterisk was built with optimizations on russell1-2/+6
with platforms that explode on unaligned access. I'm not exactly sure why this fixes it, but it fixed it on the machine I was testing on. If it makes sense to you, feel free to enlighten me. :) (closes issue #11725, patched by me) git-svn-id: http://svn.digium.com/svn/asterisk/trunk@98270 f38db490-d61c-443f-a65b-d21fe96a405b