----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4196/ -----------------------------------------------------------
Review request for Asterisk Developers and Joshua Colp. Repository: Asterisk Description ------- In https://reviewboard.asterisk.org/r/4167, two bugs were introduced: * Prior to entering the features retry loop, the buffer that holds the collected digits is wiped. However, this inadvertently wipes out the first collected digit on the first pass through, which is obtained in ast_stream_and_wait. This caused all of the features tests to fail. * If ast_app_dtget returns a hangup (-1), the loop would retry incorrectly. If we detect a hangup, we have to stop trying the feature. This patch fixes both issues. Diffs ----- /branches/13/main/bridge_basic.c 428255 Diff: https://reviewboard.asterisk.org/r/4196/diff/ Testing ------- Tested with the atxfer_nominal test. Prior to this patch it would fail; with this patch it completes successfully. Thanks, Matt Jordan
-- _____________________________________________________________________ -- 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
