-
/asterisk/trunk/lib/python/asterisk/asterisk.py 6482
Diff: https://reviewboard.asterisk.org/r/4464/diff/
Testing
---
No more shutdown timeouts from tests capable of graceful shutdown.
Thanks,
Corey Farrell
--
_
-- Bandwidth
with REF_DEBUG enabled under valgrind. No reference
leaks or improper memory access. Though this does not test for races, I don't
know of an automated way to do that.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation
be a problem if the
object were required to be immediately weakened after creation.
Corey Farrell wrote:
I've transfered the actual creation of weak proxy objects from ao2_weaken
to ao2_alloc (with an option).
I've changed my mind. Creation of ao2_weakproxy objects requires it's own
On Feb. 20, 2015, 12:03 p.m., Corey Farrell wrote:
/branches/11/include/asterisk/utils.h, lines 946-948
https://reviewboard.asterisk.org/r/4370/diff/2/?file=71539#file71539line946
I feel that configure should create a #define for the type of nested
procedure supported
/4370/#comment25074
I missed this before / thought Josh pointed it out. This one should be
AC_MSG_ERROR as well.
/branches/11/configure.ac
https://reviewboard.asterisk.org/r/4370/#comment25073
These lines are unneeded as we've failed.
- Corey Farrell
On Feb. 20, 2015, 9:35 p.m
/ parenthesis.
Diffs
-
/branches/11/include/asterisk/lock.h 431994
Diff: https://reviewboard.asterisk.org/r/4436/diff/
Testing
---
Looked at the source. I don't know what compiler would match the #if
conditions.
Thanks,
Corey Farrell
to parse.
Diffs
-
/trunk/main/asterisk.c 431991
Diff: https://reviewboard.asterisk.org/r/4435/diff/
Testing
---
Some testsuite runs.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api
/diff/
Testing
---
Build, basic startup/shutdown test.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit
://reviewboard.asterisk.org/r/4370/#comment25068
#error?
- Corey Farrell
On Feb. 19, 2015, 11:09 p.m., Matt Jordan wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4370
.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
---
Looked at the source. I don't know what compiler would match the #if
conditions.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE
., Corey Farrell wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4435/
---
(Updated Feb. 19, 2015, 4:43 p.m.)
Review
, tests/feature_blonde_transfer
Only tests/callparking still has 1 leaked reference (it was more).
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
---
On Feb. 16, 2015, 1:58 p.m., Corey Farrell wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4428
-
/branches/13/main/loader.c 431876
/branches/13/main/codec.c 431876
/branches/13/main/bucket.c 431876
Diff: https://reviewboard.asterisk.org/r/4428/diff/
Testing
---
Build, basic startup/shutdown test.
Thanks,
Corey Farrell
:
tests/apps/directed_pickup/pickup_chan, tests/callparking,
tests/channels/iax2/acl_call, tests/channels/iax2/basic-call,
tests/feature_attended_transfer, tests/feature_blonde_transfer
Only tests/callparking still has 1 leaked reference (it was more).
Thanks,
Corey Farrell
/callparking still has 1 leaked reference (it was more).
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http
of Asterisk timed out, causing reference leaks to be
reported. These tests now shutdown gracefully and have no reference leaks.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com
for tests that cannot run.
Diffs
-
/asterisk/trunk/runtests.py 6377
Diff: https://reviewboard.asterisk.org/r/4410/diff/
Testing
---
Verified correct output for a test that cannot run due to minversion.
Thanks,
Corey Farrell
p.m., Corey Farrell wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4411/
---
(Updated Feb. 9, 2015, 12:50 p.m
to
bridge_builtin_features.c is omitted the test fails due to that one reference
leak.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update
,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
---
Verified correct output for a test that cannot run due to minversion.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4141/#review13861
---
On Nov. 2, 2014, 2:13 a.m., Corey Farrell wrote
to
bridge_builtin_features.c is omitted the test fails due to that one reference
leak.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit
that this cannot race, since any
potential for a race would be due to very exact timing.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE
/4108/#review13933
---
On Oct. 26, 2014, 7:10 a.m., Corey Farrell wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4108
On Jan. 28, 2015, 7:21 a.m., Corey Farrell wrote:
If we assume that there are always unknown security vulnerabilities, I
think it is worth completely removing Server: Asterisk/version.
Another option would be trimming to major version only - Server:
Asterisk/13. Otherwise any
will agree with this but feel that it needs to be
considered.
- Corey Farrell
On Jan. 27, 2015, 7:16 p.m., Ashley Sanders wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4374
. To reply, visit:
https://reviewboard.asterisk.org/r/4186/#review13806
---
On Nov. 18, 2014, 1:36 p.m., Corey Farrell wrote:
---
This is an automatically generated e-mail. To reply
://reviewboard.asterisk.org/r/4186/diff/
Testing
---
Full testsuite against 12. I had 17 failures, but that is normal on my system.
I re-ran the 17 tests without this patch, they still failed.
Thanks,
Corey Farrell
like to see this committed
ASAP so further improvements can be made against it.
- Corey Farrell
On Oct. 16, 2014, 5:23 p.m., Scott Griepentrog wrote:
---
This is an automatically generated e-mail. To reply, visit:
https
.
Diffs
-
/branches/12/include/asterisk/stringfields.h 427735
Diff: https://reviewboard.asterisk.org/r/4186/diff/
Testing
---
Full testsuite against 12. I had 17 failures, but that is normal on my system.
I re-ran the 17 tests without this patch, they still failed.
Thanks,
Corey
On Nov. 18, 2014, 9:18 a.m., Corey Farrell wrote:
Please discard my findings, they are minor. I'd like to see this committed
ASAP so further improvements can be made against it.
Scott Griepentrog wrote:
I like your idea of adding an option to pass valgrind options, but it
needs
---
On Nov. 18, 2014, 9:25 a.m., Corey Farrell wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4186
://reviewboard.asterisk.org/r/4186/diff/
Testing
---
Full testsuite against 12. I had 17 failures, but that is normal on my system.
I re-ran the 17 tests without this patch, they still failed.
Thanks,
Corey Farrell
testsuite against 12. I had 17 failures, but that is normal on my system.
I re-ran the 17 tests without this patch, they still failed.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4194/#review13809
---
Ship it!
Ship It!
- Corey Farrell
On Nov. 18, 2014, 3:03
/diff/
Testing (updated)
---
Full testsuite against 12. I had 17 failures, but that is normal on my system.
I re-ran the 17 tests without this patch, they still failed.
Thanks,
Corey Farrell
--
_
-- Bandwidth
427685
Diff: https://reviewboard.asterisk.org/r/4160/diff/
Testing
---
tests/channels/SIP against 11
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4182/#review13789
---
Ship it!
Ship It!
- Corey Farrell
On Nov. 14, 2014, 6:03
far I've only verified this change with the unit test brought to
my attention by Richard. I'll be running the full testsuite ASAP, but wanted
to allow this code to be seen now.
Thanks,
Corey Farrell
--
_
-- Bandwidth
.
Diffs
-
/asterisk/trunk/tests/bridge/bridge_action/bridge_action.py 5920
Diff: https://reviewboard.asterisk.org/r/4166/diff/
Testing
---
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http
On Nov. 13, 2014, 11:30 a.m., Mark Michelson wrote:
With the fix being made to the leaked bridge in Asterisk, is this change
still required? Does hanging up self.channels[1] not result in
self.channels[3] and the bridge being destroyed as expected?
Corey Farrell wrote:
Still
/r/4182/#comment24262
Does this actually initialize 256 bytes of '\0', or just initialize the
first byte?
/branches/13/main/asterisk.c
https://reviewboard.asterisk.org/r/4182/#comment24261
Space around '-'.
Also why was the return removed?
- Corey Farrell
On Nov. 14, 2014
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4170/#review13743
---
On Nov. 11, 2014, 5:41 p.m., Corey Farrell wrote:
---
This is an automatically
before read() when
it's actually uninitialized data.
I don't have a specific problem with increasing the buffer from 80 to 256, but
I think we need to fix the parser so it doesn't crash if given 256 or more
bytes. I think this issue applies to 11+.
- Corey Farrell
On Nov. 13, 2014, 3:31 p.m
, visit:
https://reviewboard.asterisk.org/r/4166/#review13745
---
On Nov. 11, 2014, 3:37 p.m., Corey Farrell wrote:
---
This is an automatically generated e-mail. To reply, visit
On Nov. 13, 2014, 11:30 a.m., Mark Michelson wrote:
With the fix being made to the leaked bridge in Asterisk, is this change
still required? Does hanging up self.channels[1] not result in
self.channels[3] and the bridge being destroyed as expected?
Corey Farrell wrote:
Still
with these options now works in Linux Mint 17 (gcc 4.8.2-19ubuntu1).
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options
://reviewboard.asterisk.org/r/4160/diff/
Testing (updated)
---
tests/channels/SIP against 11
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
/
Testing
---
Compiled, visually inspected. Ran all of tests/channel/SIP in testsuite.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE
, 2014, 1:25 a.m., Corey Farrell wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4160/
---
(Updated Nov. 10, 2014, 1:25 a.m
Diff: https://reviewboard.asterisk.org/r/4166/diff/
Testing
---
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options
/tests/rest_api/applications/subscribe-bridge/subscribe_bridge.py
5920
Diff: https://reviewboard.asterisk.org/r/4170/diff/
Testing
---
Against 13
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http
to svn:ignore for the
following folders:
apps/confbridge/
codecs/ilbc/
codecs/speex/
tests/
- Corey Farrell
On Nov. 8, 2014, 1:29 p.m., Corey Farrell wrote:
---
This is an automatically generated e-mail. To reply, visit:
https
.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
this fix low risk.
The change is slightly different in 12+, as p-refer-refer_call only exists in
11.
Diffs
-
/branches/11/channels/chan_sip.c 427666
Diff: https://reviewboard.asterisk.org/r/4160/diff/
Testing
---
Compiled, visual inspection.
Thanks,
Corey Farrell
On Nov. 6, 2014, 6:04 a.m., wdoekes wrote:
Isn't the better fix to disable coverage for the shadow compilation?
Corey Farrell wrote:
Probably. I have updated code but I won't have time to test until this
weekend. I'll update the review once I've had a chance to do some builds
/diff/
Testing
---
No. Visually inspected changes, they are straight forward.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE
/diff/
Testing
---
Build with these options now works in Linux Mint 17 (gcc 4.8.2-19ubuntu1).
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
).
Diffs
-
/branches/11/main/manager.c 427380
Diff: https://reviewboard.asterisk.org/r/4153/diff/
Testing
---
tests/manager/config/basic against 13.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided
/lab_stringfields_leak.c
to verify that old pools are now freed.
Full testsuite against Asterisk 13.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
-
/branches/13/res/res_hep.c 427298
Diff: https://reviewboard.asterisk.org/r/4150/diff/
Testing
---
Tested by Zane Conkle.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com
4.8.2-19ubuntu1).
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo
/queue_baseline/run_1/ast1/var/log/asterisk/refs.txt
Diffs
-
/branches/11/channels/chan_console.c 427380
Diff: https://reviewboard.asterisk.org/r/4152/diff/
Testing
---
No. Visually inspected changes, they are straight forward.
Thanks,
Corey Farrell
, they are straight forward.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4151/#review13701
---
On Nov. 6, 2014, 5:06 a.m., Corey Farrell wrote
/basic against 13.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo
.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3603/#review13685
---
On Oct. 30, 2014, 8:06 p.m., Corey Farrell wrote
the echo
protocol, but ast_websocket_remove_protocol is used to remove it. This causes
an extra call to ast_module_unref.
Diffs
-
/branches/12/res/res_http_websocket.c 426831
Diff: https://reviewboard.asterisk.org/r/4140/diff/
Testing
---
Found/tested with r4141.
Thanks,
Corey
it but that test is
disabled per ASTERISK-19565.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http
/file.c 427255
Diff: https://reviewboard.asterisk.org/r/4149/diff/
Testing
---
Verified the issue and fix with tests/apps/voicemail/play_message + r4141.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided
://issues.asterisk.org/jira/browse/ASTERISK-24491
Repository: Asterisk
Description
---
Add missing unref to hepv3_send_packet.
Diffs
-
/branches/13/res/res_hep.c 427298
Diff: https://reviewboard.asterisk.org/r/4150/diff/
Testing
---
Tested by Zane Conkle.
Thanks,
Corey
Asterisk 13.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4114/#review13628
---
On Nov. 3, 2014, 8:29 a.m., Corey Farrell wrote:
---
This is an automatically generated e
Asterisk 13.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
Asterisk 13.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4114/#review13658
---
On Nov. 3, 2014, 12:55 p.m., Corey Farrell wrote:
---
This is an automatically generated e
Diff: https://reviewboard.asterisk.org/r/4141/diff/
Testing
---
Using tests/manager/originate with REF_DEBUG enabled. When the change to
bridge_builtin_features.c is omitted the test fails due to that one reference
leak.
Thanks,
Corey Farrell
-
/branches/12/res/res_http_websocket.c 426831
Diff: https://reviewboard.asterisk.org/r/4140/diff/
Testing
---
Found/tested with r4141.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api
If it's possible for apps_registry to be NULL then it's possible for the
condition to race. Maybe we need to use AO2_GLOBAL_OBJ_STATIC? Could this
issue apply to the other global containers in this module (app_controls,
app_bridges, app_bridges_moh, app_bridges_playback)?
-- Forwarded
---
tests/funcs/func_jitterbuffer against 13 no longer leaks.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http
,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
no longer leaks - tested against 13.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com
/
Testing
---
Against 13, tests passed with no leaks reported. Verified no other active test
uses syncami. Only tests/channels/SIP/nat_supertest used it but that test is
disabled per ASTERISK-19565.
Thanks,
Corey Farrell
/r/4144/diff/
Testing
---
Yes
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4138/#review13647
---
On Oct. 31, 2014, 1:26 a.m., Corey Farrell wrote
ast_writestream is slightly over 64k.
Diffs (updated)
-
/branches/11/main/app.c 426569
/branches/11/apps/app_voicemail.c 426569
Diff: https://reviewboard.asterisk.org/r/4138/diff/
Testing
---
tests/apps/voicemail/check_voicemail_forward_with_prepend/ no longer leaks.
Thanks,
Corey
/func_jitterbuffer.c.
Diffs (updated)
-
/branches/11/main/abstract_jb.c 426593
/branches/11/funcs/func_jitterbuffer.c 426593
Diff: https://reviewboard.asterisk.org/r/4128/diff/
Testing
---
tests/funcs/func_jitterbuffer against 13 no longer leaks.
Thanks,
Corey Farrell
from 300 to 30, use reset_timeout
instead.
Diffs
-
/asterisk/trunk/tests/pbx/dialplan_reload/run-test 5803
Diff: https://reviewboard.asterisk.org/r/4122/diff/
Testing
---
Yes
Thanks,
Corey Farrell
I'm confused about how this can be? Unless I'm missing something a
module dependency doesn't effect how the test runs, just if it runs.
If chan_sip is not required then shouldn't we remove sip.conf from
each of these tests config dirs?
-- Forwarded message --
From: SVN commits to
].format in
ast_audiohook_detach_list.
Diffs
-
/branches/13/main/audiohook.c 426528
Diff: https://reviewboard.asterisk.org/r/4124/diff/
Testing
---
Yes
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation
/trunk/lib/python/asterisk/ari.py 5796
Diff: https://reviewboard.asterisk.org/r/4121/diff/
Testing
---
Using r4038
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk
/
Testing
---
All tests/queues/set_penalty no longer leaks any references (verifies first
added queue_unref).
I'm unsure if the second added queue_unref has been tested, but seems like it
is needed.
Thanks,
Corey Farrell
-
/branches/11/funcs/func_jitterbuffer.c 426593
Diff: https://reviewboard.asterisk.org/r/4128/diff/
Testing
---
tests/funcs/func_jitterbuffer against 13 no longer leaks.
Thanks,
Corey Farrell
--
_
-- Bandwidth and Colocation
/func_jitterbuffer.c 426593
Diff: https://reviewboard.asterisk.org/r/3603/diff/
Testing (updated)
---
Verified the scenario outlined in ASTERISK-22409 no longer experiences audio
quality loss.
Thanks,
Corey Farrell
Hello everyone,
Review 3063 [1] is posted to address audio quality failure experienced
when certain masquerades occur on a channel with func_jitterbuffer
active. The goal of this change is to address situations where a
masquerade occurs due to Local channel optimization, and one or both
of the
/func_jitterbuffer.
- Corey
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4128/#review13639
---
On Oct. 30, 2014, 8:06 p.m., Corey Farrell
201 - 300 of 672 matches
Mail list logo
